On Tue, Apr 10, 2018 at 10:15:45PM -0700, Jason Ekstrand wrote:
> On Tue, Apr 10, 2018 at 12:52 PM, Caio Marcelo de Oliveira Filho <
> caio.olive...@intel.com> wrote:
>
> > The matching code doesn't make real use of the return value. The main
> > function return value is ignored, and while the worker function
> > propagate its return value, the actual callback never returns false.
> >
>
> When I wrote this, I don't think I realized it was only going to have the
> one use. I was mostly following the pattern we'd used for all the other
> foreach functions in NIR. That said, I think this is better.
I guessed that was the case.
> When a block contains multiple lines, we use { } regardless of whether or
> not it's just a single statement.
Got it!
Will send v2 soon.
Thanks,
Caio
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
