Re: [PATCH V3] dtc: fix for_each_*() to skip first object if deleted

[Jon Loeliger <jdl-CYoMK+44s/E@public.gmane.org>]

> On Fri, Oct 05, 2012 at 09:57:41AM -0600, Stephen Warren wrote:
> > From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> > 
> > The previous definition of for_each_*() would always include the very
> > first object within the list, irrespective of whether it was marked
> > deleted, since the deleted flag was not checked on the first object,
> > but only on any "next" object.
> > 
> > Fix for_each_*() to check the deleted flag in the loop body every
> > iteration to correct this.
> > 
> > Incidentally, this change is why commit 45013d8 dtc: "Add ability to
> > delete nodes and properties" only caused two "make checkm" failures;
> > only two tests actually use multiple labels on the same property or
> > node. With this current change applied, but commit 317a5d9 "dtc: zero
> > out new label objects" reverted, "make checkm" fails 29 times; i.e.
> > for every test that uses any labels at all.
> > 
> > Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> Acked-by: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>

Applied.

Thanks,
jdl