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 <[email protected]>
> 
> Acked-by: Seth Arnold <[email protected]>
> 

> 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.



-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to