Re: [Mesa-dev] [PATCH 10/11] intel/tools/error: Use do-while for field iterator loops.

2017-11-12 Thread Lionel Landwerlin
I think I sneaked the exact same fix in f5e5ca1e210c2e0f505ea154ca553275157dda73 (bottom of the patch). Reviewed-by: Lionel Landwerlin On 12/11/17 08:35, Kenneth Graunke wrote: while loops skip the first field of the instruction/structure, which is not what the

[Mesa-dev] [PATCH 10/11] intel/tools/error: Use do-while for field iterator loops.

2017-11-12 Thread Kenneth Graunke
while loops skip the first field of the instruction/structure, which is not what the code intended. It works out because the field we're looking for doesn't happen to be first, but we ought to do it right regardless. Found while writing the next patch, where Kernel Start Pointer is the first