On Thu, Dec 04 2014, Matt Turner wrote:
> On Thu, Dec 4, 2014 at 2:00 PM, Carl Worth <cwo...@cworth.org> wrote:
>> +   assert(!"Should not get here.");
>> +   return error_type;
>
> Let's just use unreachable("not reached") here.

That's a good idea. I copied this line from the surrounding code in the
same file, (there are about 6 or 7 other instances in this file). So I
think I'll follow up with a separate patch to clean these up. Should I
pass over the whole source tree? Or would a change like that be too
annoying for outstanding patches, etc.?

-Carl

-- 
carl.d.wo...@intel.com

Attachment: pgpowNyfgQzJM.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to