Changeset 164 for trunk/ext/hpricot_scan/hpricot_scan.java.rl

Timestamp:

04/09/2008 22:32:36 (8 months ago)

Author:

stepheneb

Message:

ragel 6.1 fixes for hpricot_scan.java.rl; update jruby object access

Applied the updates to the ragel code for 6.1 to hpricot_scan.java.rl.

Updated decrated method used to access IRubyObject object:

new IRubyObject[]{runtime.newSymbol("buffer_size")}

becomes:

runtime.getCurrentContext(),new IRubyObject[]{runtime.newSymbol("buffer_size")}

The include statements in the main ragel code didn't
work with paths starting with ext/ so I made them
into relative path references which did work.

Updates to the Rakefile to support easy
compilation of the C ragel source code
using different ragel compilation strategies.
This is the default setting:

DEFAULT_RAGEL_C_CODE_GENERATION = "really_fast goto_driven"

which is -G2

The result is no test failures in C and only
one in Java: test_latin1_entities.

Files:

• trunk/ext/hpricot_scan/hpricot_scan.java.rl (modified) (9 diffs)

trunk/ext/hpricot_scan/hpricot_scan.java.rl

@@ -21 +21 @@
 
        public void ELE(IRubyObject N) {
-         if (tokend > tokstart || text) {
+         if (te > ts || text) {
            IRubyObject raw_string = runtime.getNil();
            ele_open = false; text = false;
-           if (tokstart != -1 && N != cdata && N != sym_text && N != procins && N != comment) { 
-             raw_string = runtime.newString(new String(buf,tokstart,tokend-tokstart));
+           if (ts != -1 && N != cdata && N != sym_text && N != procins && N != comment) { 
+             raw_string = runtime.newString(new String(buf,ts,te-ts));
            } 
            rb_yield_tokens(N, tag[0], attr, raw_string, taint);
@@ -79 +79 @@
              mark = mark_aval;
            }
-           if(mark > tokstart) {
+           if(mark > ts) {
              if(N == tag) {
-               mark_tag  -= tokstart;
+               mark_tag  -= ts;
              } else if(N == akey) {
-               mark_akey -= tokstart;
+               mark_akey -= ts;
              } else if(N == aval) {
-               mark_aval -= tokstart;
+               mark_aval -= ts;
              }
            }
@@ -95 +95 @@
              attr = RubyHash.newHash(runtime);
            }
-           ((RubyHash)attr).aset(K,V);
+           ((RubyHash)attr).op_aset(runtime.getCurrentContext(),K,V);
+           // ((RubyHash)attr).aset(K,V);
          }
        }
@@ -115 +116 @@
            if(ele_open) { 
              ele_open = false; 
-             if(tokstart > -1) { 
-               mark_tag = tokstart; 
+             if(ts > -1) { 
+               mark_tag = ts; 
              } 
            } else {
@@ -184 +185 @@
   }
 
-  include hpricot_common "ext/hpricot_scan/hpricot_common.rl";
+  include hpricot_common "hpricot_common.rl";
 
 }%%
@@ -210 +211 @@
 int cs, act, have = 0, nread = 0, curline = 1, p=-1;
 boolean text = false;
-int tokstart=-1, tokend;
+int ts=-1, te;
+int eof=-1;
 char[] buf;
 Ruby runtime;
@@ -300 +302 @@
     if ( done && ele_open ) {
       ele_open = false;
-      if(tokstart > -1) {
-        mark_tag = tokstart;
-        tokstart = -1;
+      if(ts > -1) {
+        mark_tag = ts;
+        ts = -1;
         text = true;
       }
     }
 
-    if(tokstart == -1) {
+    if(ts == -1) {
       have = 0;
-      /* text nodes have no tokstart because each byte is parsed alone */
+      /* text nodes have no ts because each byte is parsed alone */
       if(mark_tag != -1 && text) {
         if (done) {
@@ -322 +324 @@
       mark_tag = 0;
     } else {
-      have = pe - tokstart;
-      System.arraycopy(buf,tokstart,buf,0,have);
+      have = pe - ts;
+      System.arraycopy(buf,ts,buf,0,have);
       SLIDE(tag);
       SLIDE(akey);
       SLIDE(aval);
-      tokend = (tokend - tokstart);
-      tokstart = 0;
+      te = (te - ts);
+      ts = 0;
     }
   }
@@ -359 +361 @@
 public static void Init_hpricot_scan(Ruby runtime) {
   RubyModule mHpricot = runtime.defineModule("Hpricot");
-  mHpricot.getMetaClass().attr_accessor(new IRubyObject[]{runtime.newSymbol("buffer_size")});
+  mHpricot.getMetaClass().attr_accessor(runtime.getCurrentContext(),new IRubyObject[]{runtime.newSymbol("buffer_size")});
   CallbackFactory fact = runtime.callbackFactory(HpricotScanService.class);
   mHpricot.getMetaClass().defineMethod("scan",fact.getSingletonMethod("__hpricot_scan",IRubyObject.class));