Jason Ekstrand <ja...@jlekstrand.net> writes: >> + texImages[0] = _mesa_select_tex_image(ctx, texObj, texObj->Target, >> level); >> > > Do you want this inside an else block?
I think it's quite a common idiom in Mesa to handle the special cases as a series of if-statements with a return upfront before handling the normal case at the end without an else. But yes, in this instance it does look a bit odd because the there is only one special case and the normal case is very short, so I don't mind changing it. Regards, - Neil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
