<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 17, 2014 at 8:58 AM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 01/16/2014 10:13 PM, Mark Mueller wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This series encompases the much discussed specification and renaming of MESA_FORMATs,<br>
which now is packed into 8 patches<br>
<br>
Signed-off-by: Mark Mueller <<a href="mailto:MarkKMueller@gmail.com" target="_blank">MarkKMueller@gmail.com</a>><br>
---<br>
</blockquote>
<br></div>
Well, our other enum typedefs (and structs) all use the gl_ prefix.  But the other enum values don't use MESA_ prefixes so gl_formats are weird that way.  I'm kind on the fence about this change.<br>
<br>
-Brian<br>
<br></blockquote><div><br></div><div>Obviously it's not critical, but the gl_ prefix is confusing because of the weirdness, and Ken recommended a name change thus I took a stab at it. I've left this change in V4 of the series. Would it be more convincing with a different name, like mgl_formats, or mesa_gl_formats?</div>
<div><br></div><div>Mark</div></div></div></div>