Hi Andy,
On 23/04/2011 22:04, Andy Seaborne wrote:
On 07/04/11 17:59, Brian McBride wrote:
[...]
In com.hp.hpl.jena.iri.impl.AbsLexer
Replace
[[
final protected void rule(int rule) {
parser.matchedRule(range,rule,yytext());
}
]]
with
[[
private static final boolean DEBUG = false;
final protected void rule(int rule) {
if (DEBUG) {
parser.matchedRule(range,rule,yytext());
}
}
]]
I think this is in auto-generated code from fragment.jflex.
If so, the change will get lost if it isn't made there.
I don't think so. The AbsLexer code doesn't look like it has been
generated by JFlex.
Mind you, nor does LexerFragment.java.
Would it be better to tie it to Parser.DEBUG (and make that final) so
one setting affects both?
Yes it would. I've made that change in my copy which will be included
if/when I submit a patch to Jira.
Brian
Andy
