[apparmor] [2 patches] utils: write prior segments processing

[Christian Boltz]

Hello,

Am Mittwoch, 26. November 2014 schrieb Steve Beattie:
> While trying to help Christian diagnose some problems I was having
> with his capabilities classes patches, I wrote the following patches.
> 
> The first was discovered while writing the second: the assignment for
> setting segments['include'] = True is wrong, it occurs inside the
> 'if not segments['include'] and True in segments.values():' block,
> whereas it needs to always get set outside of that if test.

Acked-by: Christian Boltz <apparmor at cboltz.de>

> The second pulls out all the common processing for writing out each
> of the prior segments that need to be written before writing the
> current segment into a function called 'write_prior_segments',
> reducing a bunch of ugly duplication.

Nice cleanup!

Acked-by: Christian Boltz <apparmor at cboltz.de>
even if I can foresee a merge conflict with the [3/3] part of my 
capability rule class patchset ;-)

Please commit those patches soon so that I can send out a new version of 
the capability rule class patchset.


Regards,

Christian Boltz
-- 
I will hand over the release job to Mr. Bash, he is more reliable :/
[Adrian Schroeter in opensuse]