edn_turbo 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aa435bbcdadfbf654990bf8ee34e6830f5642d68
-  data.tar.gz: 590f17ada5e84d90550d9b99b7b5afb035b0384a
+  metadata.gz: 41f78cfbb6704e1d0a4f7e1336d62609c3c10d18
+  data.tar.gz: 0a8080b1e3356f9e6f743b27670ee9d8ccba4769
 SHA512:
-  metadata.gz: 2be6b790ab0072934c329c81d5e02e479749697521d54b561ccccd3c005e3d855085bb4c8e74c194e09fc540161871b8a921c6c1e35ce50bb4888fb926116677
-  data.tar.gz: 7ecb72bba74f5e7016cad489e480aa74d496953d78218612de60d48ca6493a37fcbb8fc0e296a9fc6a1b9d84928fcb30fbd4f3293b89010783d8d41c31589f0f
+  metadata.gz: b45b5f73a05bee804ca8159d7e6b42ca39fde77545c7ed36a67b5f1197e7ac88e45a2e41a08179eba2f8ad6f7f5ab5bc51ee9f67cbbd12d8dbdbe0a95679df33
+  data.tar.gz: 98f1c888c9eeb40fabd89f48a81a9e8178321393cb6f74e49d078b1ae8e1e1f39be8653c7bda51f37a45811cbcca97a99505439c0afa052da65fe4da25b148fe

data/README.md

@@ -5,10 +5,11 @@ Ruby C-extension for parsing EDN files.
 
 Written based on the functionality of
 [edn](https://github.com/relevance/edn-ruby) but with the goal of
-achieving much faster parsing. Currently supports a subset of the EDN
-spec, primarily that created from dumping ruby object data in EDN
-format. Support for tagged elements, sets, and full unicode is coming
-soon.
+achieving much faster parsing. 
+
+As of v0.2.0, `edn_turbo` requires `edn` to support tagged elements 
+using a similar API and return types. The eventual goal is to bundle 
+`edn_turbo` as an optional parser for `edn`.
 
 Some quick sample runs comparing time output of file reads using `edn`
 and `edn_turbo`:
@@ -52,11 +53,11 @@ Dependencies
   - [rake 10.3.2](http://rake.rubyforge.org)
   - [rake-compiler 0.9.2](http://rake-compiler.rubyforge.org)
   - [rice 1.7.0](http://rice.rubyforge.org)
+  - [edn 1.0.7](https://github.com/relevance/edn-ruby)
 - [icu4c](http://icu-project.org/apiref/icu4c/)
 
 `edn_turbo` uses a ragel-based parser but the generated .cc file is
-bundled so ragel should not need to be installed.
-
+bundled so ragel should not need to be installed. 
 
 Usage
 =====
@@ -68,3 +69,14 @@ Usage
        pp output if output != nil
     end
 ```
+
+Known problems
+==============
+As of v0.2.0:
+
+- Need to support stand-alone `-` and `+` symbols as well as named 
+symbols with a leading `-`. The current set of parsers expect only
+numeric values to optionally lead with a `-` so support for the
+other two cases requires refactoring of the parser logic
+- Need to check handling of discards at the root level. Discards
+within containers looks to work 100% as far as I know.

data/Rakefile

@@ -7,7 +7,6 @@ NAME = 'edn_turbo'
 LIB_DIR = "lib/#{NAME}"
 EXT_BUNDLE = "#{LIB_DIR}/#{NAME}.#{RbConfig::CONFIG['DLEXT']}"
 
-
 EXT_PATH               = "ext/#{NAME}"
 RAGEL_PARSER_SRC       = "edn_parser.rl"
 RAGEL_PARSER_SRC_PATH  = "#{EXT_PATH}/#{RAGEL_PARSER_SRC}"
@@ -23,6 +22,8 @@ task :chmod do
   File.chmod(0775, EXT_BUNDLE)
 end
 
+CLEAN.include(["*.png", "*.gem"])
+
 task :ragel => GEN_CC_PARSER_SRC_PATH
 
 
@@ -34,9 +35,11 @@ file GEN_CC_PARSER_SRC_PATH => RAGEL_PARSER_SRC_PATH do
   end
 end
 
-task :graph do
+
+task :graph, [:machine] do |t, args|
+  args.with_defaults(:machine => 'EDN_value')
   TMPFILE='/tmp/ragel_edn'
-  MACHINE='EDN_value'
+  MACHINE=args[:machine]
 
   # assumes graphviz is installed
   sh "ragel -Vp -S #{MACHINE} -o #{TMPFILE} #{EXT_PATH}/#{RAGEL_PARSER_SRC} && dot -Tpng #{TMPFILE} -o #{MACHINE}.png"