You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2020/05/28 21:24:27 UTC

[GitHub] [lucene-solr] jpountz commented on pull request #1541: RegExp - add case insensitive matching option

jpountz commented on pull request #1541:
URL: https://github.com/apache/lucene-solr/pull/1541#issuecomment-635616067


   The flags that can be passed to the constructor are about the supported operators. Case insentivity is not an operator so it feels wrong to ask users to configure it this way too? I think it's fine to record this information in the flags _internally_, but maybe we should make the constructor take an additional boolean instead of expecting users to configure case insensitivity via a syntax flag?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org