rmuir commented on code in PR #14381:
URL: https://github.com/apache/lucene/pull/14381#discussion_r2007499003
##
lucene/core/src/java/org/apache/lucene/util/automaton/RegExp.java:
##
@@ -778,6 +786,53 @@ private int[] toCaseInsensitiveChar(int codepoint) {
}
}
+ /**
+
rmuir commented on PR #14381:
URL: https://github.com/apache/lucene/pull/14381#issuecomment-2743822277
@dweiss thanks for the suggestion there, gazillions of array creations
avoided. so now this thing will only spike cpu during parsing at worst. I
honestly forget you can pass functions to f
rmuir merged PR #14381:
URL: https://github.com/apache/lucene/pull/14381
--
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.apach
rmuir commented on PR #14381:
URL: https://github.com/apache/lucene/pull/14381#issuecomment-2743798573
after fixing the turkish here's the (correct) automaton for `/[a-z]/`: the
only special cases are long-s and kelvin sign as you expect:
 {
}
}
+ /**
+
rmuir opened a new pull request, #14381:
URL: https://github.com/apache/lucene/pull/14381
Add optional flag to support case-insensitive ranges. A minimal DFA is
always created. This works with Unicode but may have a performance cost.
Each codepoint in the range must be iterated, and a
dweiss commented on code in PR #14381:
URL: https://github.com/apache/lucene/pull/14381#discussion_r2006930601
##
lucene/core/src/java/org/apache/lucene/util/automaton/RegExp.java:
##
@@ -778,6 +786,53 @@ private int[] toCaseInsensitiveChar(int codepoint) {
}
}
+ /**
