Small bug in Help fileMatch

[chrisjj]

rawDate = fileMatch('^Date:\s*.*', 'i');

'\s" is nonsense here, and presumably that should read:

Code: Select all

rawDate = fileMatch('^Date:\\s*.*','i');

though it doesn't cause a fail.

And better:

Code: Select all

rawDate = fileMatch('^Date:.*',    'i');

or indeed (as per RFC):

Code: Select all

rawDate = fileMatch('^Date:.*');

[Admin]

Hi, \s is meant to match all spaces after : and before date starts (just in case there are more than one).

[chrisjj]

Hi, \s is meant to match all spaces after : and before date starts (just in case there are more than one).

But it doesn't. It matches any s after : and before date.

[Admin]

Yes, it should be \\s (escaped). I have logged this one to be fixed. Thank you for reporting this.

[chrisjj]

Yes, it should be \\s (escaped). I have logged this one to be fixed.

Great. But note that actually the \\s is redundant - given it is followed by .* . So as I said, better would be:

rawDate = fileMatch('^Date:.*');

whcih also removed the case insensitivity, respecting the fact that Date is case-significant/

[Admin]

Yes, the \s is not really needed for the match since it's zero or more spaces...