On 03/ 7/18 01:42 PM, Emil Velikov wrote: > On 7 March 2018 at 20:10, Alan Coopersmith <alan.coopersm...@oracle.com> > wrote: >> That should be effectively equivalent, just less change in indentation for >> the lines in between. >> > You're correct - it's functionally identical although it seems cleaner.
While the patch seems cleaner & simpler this way, I think the version in my original patch results in the final code being cleaner and more straightforward to understand without having to trace where goto's are jumping to, and that's more important long-term than the patch itself. > Regardless of which version you opt for > Reviewed-by: Emil Velikov <emil.veli...@collabora.com> Thanks, -alan- _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel