[apparmor] [PATCH 06/10] From f5898cf8e0b2d9b032e349be6f7e4f57a6e12768 Mon Sep 17 00:00:00 2001 From: John Johansen <john.johansen at canonical.com> Date: Sat, 3 Nov 2012 08:19:47 -0700 Subject: [PATCH 06/10] Convert the parser to C++

John Johansen john.johansen at canonical.com
Sun Jul 28 00:12:57 UTC 2013


On 07/25/2013 03:59 PM, Seth Arnold wrote:
> On Sun, Jul 21, 2013 at 10:32:49PM -0700, John Johansen wrote:
>> This conversion is nothing more than what is required to get it to
>> compile. Further improvements will come as the code is refactored.
>>
>> Unfortunately due to C++ not supporting designated initializers, the auto
>> generation of af names needed to be reworked, and "netlink" and "unix"
>> domain socket keywords leaked in. Since these where going to be added in
>> separate patches I have not bothered to do the extra work to replace them
>> with a temporary place holder.
>>
>> Signed-off-by: John Johansen <john.johansen at canonical.com>
> 
> Acked-by: Seth Arnold <seth.arnold at canonical.com>
> 

> C++ sure feels old and busted after reading through this patch. I hope
> the end result is worth it...
> 
well I wouldn't really call this C++ its C that is compatible with the C++
compiler

eventually as we get to moving parts to classes it will be more worth it.





More information about the AppArmor mailing list