[apparmor] [PATCH] [parsers] allow for nested alternations expressions
Seth Arnold
seth.arnold at canonical.com
Tue Nov 5 01:44:52 UTC 2013
On Mon, Nov 04, 2013 at 05:28:22PM -0800, John Johansen wrote:
> > + //MY_REGEX_TEST("\\", "\\", ePatternBasic);
> > + MY_REGEX_TEST("\\\\", "\\\\", ePatternBasic);
> > + //MY_REGEX_TEST("\\blort", "\\blort", ePatternBasic);
> > + MY_REGEX_TEST("\\\\blort", "\\\\blort", ePatternBasic);
> > + //MY_REGEX_TEST("blort\\", "blort\\", ePatternBasic);
> why are these 3 commented out?
Since I'm no fan of commented-out code I very nearly commented on them :)
but then I realized that they are there to document what the next line
_actually_ tests, since the first backslashes are there to escape C
quoting rules. Their presence put me in the right state of mind to read
the rest of the rules. :)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20131104/b0a81640/attachment.pgp>
More information about the AppArmor
mailing list