The correct syntax for 
Perl5Compiler.CASE_INSENSITIVE_MASK & Perl5Compiler.READ_ONLY_MASK
is
Perl5Compiler.CASE_INSENSITIVE_MASK | Perl5Compiler.READ_ONLY_MASK

See 
<<https://jakarta.apache.org/oro/api/org/apache/oro/text/regex/Perl5Compiler.html#compile(java.lang.String,
 int)>> (copy the whole string between <<...>>)

Jacques

On Saturday, November 16, 2013 8:03 PM [email protected] <[email protected]> 
wrote:
> Author: adrianc
> Date: Sat Nov 16 19:03:17 2013
> New Revision: 1542551
> 
> URL: http://svn.apache.org/r1542551
> Log:
> Small fixup for me previous commit - Perl5 pattern compiler needs to be told 
> the patterns will be shared among threads.
> 
> Modified:
>    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/PatternFactory.java
> 
> Modified: 
> ofbiz/trunk/framework/base/src/org/ofbiz/base/util/PatternFactory.java
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/PatternFactory.java?rev=1542551&r1=1542550&r2=1542551&view=diff
> ==============================================================================
>  ---
> ofbiz/trunk/framework/base/src/org/ofbiz/base/util/PatternFactory.java 
> (original) +++
> ofbiz/trunk/framework/base/src/org/ofbiz/base/util/PatternFactory.java Sat 
> Nov 16 19:03:17 2013 @@ -45,9 +45,9 @@ public class
>         PatternFactory { if (pattern == null) {
>             Perl5Compiler compiler = new Perl5Compiler();
>             if (caseSensitive) {
> -                pattern = compiler.compile(stringPattern);
> +                pattern = compiler.compile(stringPattern, 
> Perl5Compiler.READ_ONLY_MASK);
>             } else {
> -                pattern = compiler.compile(stringPattern, 
> Perl5Compiler.CASE_INSENSITIVE_MASK);
> +                pattern = compiler.compile(stringPattern, 
> Perl5Compiler.CASE_INSENSITIVE_MASK & Perl5Compiler.READ_ONLY_MASK);
>             }
>             pattern = compiledPerl5Patterns.putIfAbsentAndGet(stringPattern, 
> pattern);
>             if (Debug.verboseOn()) {

Reply via email to