[apparmor] [PATCH 1/8] clean up the lexer

Steve Beattie steve at nxnw.org
Thu Sep 12 07:00:17 UTC 2013


On Wed, Sep 11, 2013 at 01:47:40AM -0700, Tyler Hicks wrote:
> From: John Johansen <john.johansen at canonical.com>
> 
> - Make indenting consistent
> - Move common match + fn patterns into a single shared entry with mulitstate
>   headers
> - add names table to convert lexer state #s to state names used in the code
> - Create/use macros for common patterns of DUMP, DEBUG, return ...
>   this fixes a few places where preprocess or DEBUG output was not
>   available
> - update RE patterns for bugs eg. {WS} inside a character class [] does
>   not match whitespace
> 
> all told despite adding code to provide better support to debug and
> preprocessing, the code is about 150 lines shorter, and has few corner
> cases cleaned up.
> 
> Signed-off-by: John Johansen <john.johansen at canonical.com>
> [tyhicks: Added small fix for D-Bus parsing bug]
> Signed-off-by: Tyler Hicks <tyhicks at canonical.com>

Acked-by: Steve Beattie <steve at nxnw.org>

-- 
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20130912/d445f641/attachment.pgp>


More information about the AppArmor mailing list