On 09/11/2014 04:04 PM, Steve Beattie wrote: > On Wed, Sep 03, 2014 at 12:30:18PM -0700, Steve Beattie wrote: >> In profile.h, flagvals is declared to be class, but then in the Profile >> class, the flags field declares it as a struct. This patch makes the >> field declaration type consistent. >> >> Signed-off-by: Steve Beattie <[email protected]> yep Acked-by: John Johansen <[email protected]>
>> --- >> parser/profile.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> Index: b/parser/profile.h >> =================================================================== >> --- a/parser/profile.h >> +++ b/parser/profile.h >> @@ -128,7 +128,7 @@ public: >> >> Profile *parent; >> >> - struct flagvals flags; >> + class flagvals flags; >> struct capabilities caps; >> struct network net; > > Actually, I think the correct stylistic fix is to not use the class > keyword there, like so: > > Signed-off-by: Steve Beattie <[email protected]> > --- > parser/profile.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: b/parser/profile.h > =================================================================== > --- a/parser/profile.h > +++ b/parser/profile.h > @@ -128,7 +128,7 @@ public: > > Profile *parent; > > - struct flagvals flags; > + flagvals flags; > struct capabilities caps; > struct network net; > > > > > -- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
