On 2013-09-11 03:30:46, Steve Beattie wrote: > On Wed, Sep 11, 2013 at 01:42:32AM -0700, Tyler Hicks wrote: > > Removes an unnecessary variable, simplifies and unifies some of the loop > > logic, and removes commented out code. > > > > Signed-off-by: Tyler Hicks <[email protected]> > > Acked-by: Steve Beattie <[email protected]> > > Similar comment as last time, it might help to make clear what's > happening to lift the merging code out into a relatively simple > function of its own.
Good idea. I'm going to place the sorting and the merging sections into two separate functions. But, I'd like to wait until the C++ patch set and other pending patches on the list settle before doing that. I've added it to my todo list. Tyler
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
