rmuir commented on pull request #515: URL: https://github.com/apache/lucene/pull/515#issuecomment-986075965
@dweiss i think it's good win just for getting a simpler grammar. There is probably evil stuff we could do to speed up the monster. Yes, I am slightly tempted to import `org.apache.lucene.util.automaton` into `GenerateJFlexTLDMacros.java`... but I agree with your thoughts on the JFlex issue, it is better to just generate the simple "transparent" grammar of TLDs, despite how slow it makes things. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
