Ticket #154 (new defect)

Opened 5 months ago

[PATCH] fixes r163 to work with ragel 6.1, java, and jruby 1.1

Reported by: stepheneb Owned by: why
Priority: critical Milestone: 0.6
Component: ext/hpricot_scan Version:
Keywords: JRuby Cc: stepheneb

Description

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

Updated method to acces 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.

Attachments

fixing_r163_for_ragel_6_1_and_java.diff (6.8 kB) - added by stepheneb 5 months ago.
patch to fix r163 for ragle 6.1 and Java and JRuby 1.1

Change History

Changed 5 months ago by stepheneb

patch to fix r163 for ragle 6.1 and Java and JRuby 1.1

Note: See TracTickets for help on using tickets.