On 08 Jun 2011, at 12:07, Simon Pepping wrote:

> On Tue, Jun 07, 2011 at 10:05:24PM +0200, Andreas L. Delmelle wrote:
>> At any rate, the starting '\(' does indeed seem to be an invalid escape 
>> sequence (Shouldn't it be '\\(' if it needs to match a literal bracket?) 
>> It is present as-is, on our end in 
>> src/documentation/intermediate-format-ng/fop-intermediate-format-ng-datatypes.xsd.
>>  I am not an XML Schema expert, but that pattern does seem to be odd. Not 
>> sure why the leading backslash is needed...
>> 
>> That file hasn't changed in over 2 years, though. :-/
> 
> '\(' in the pattern is valid and indicates a literal '('.

Oooh, right! Only in Java String literal patterns do we need the double escape, 
i.e. we would need "\\(" for the regex matching pattern "\(". Since the source 
is XML Schema here, that obviously does not apply...

<snip />
> 
> I committed a fix.

Thanks for taking care of this!

Regards

Andreas
---

Reply via email to