On 08/23/2013 02:05 PM, Paul Berry wrote:
On 23 August 2013 13:55, Brian Paul <bri...@vmware.com> wrote:
On 08/23/2013 02:39 PM, Matt Turner wrote:
On Fri, Aug 23, 2013 at 1:24 PM, Brian Paul <bri...@vmware.com> wrote:
In that case, let's fix it by initializing limit to 0 at its declaration.
That way we will get a (helpful) compiler warning in the inevitable future
event that we add new values to _mesa_glsl_parser_targets.
I second Paul's advice here.
Let's not add default cases to switch statements that suppose to handle all
possible enum values. Otherwise, we may silence helpful compiler warnings
in the future.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev