On Sun, Apr 17, 2016 at 1:22 AM, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> On 16 April 2016 at 22:04, Marek Olšák <mar...@gmail.com> wrote:
>> On Sat, Apr 16, 2016 at 9:40 PM, Rob Clark <robdcl...@gmail.com> wrote:
>>> On Sat, Apr 16, 2016 at 8:50 AM, Marek Olšák <mar...@gmail.com> wrote:
>>>> From: Marek Olšák <marek.ol...@amd.com>
>>>>
>>>> and remove number assignments which are consecutive
>>>> ---
>>>> src/gallium/include/pipe/p_defines.h | 378
>>>> +++++++++++++++++++----------------
>>>> 1 file changed, 205 insertions(+), 173 deletions(-)
>>>>
>>>> diff --git a/src/gallium/include/pipe/p_defines.h
>>>> b/src/gallium/include/pipe/p_defines.h
>>>> index 1aef21d..6bb180d 100644
>>>> --- a/src/gallium/include/pipe/p_defines.h
>>>> +++ b/src/gallium/include/pipe/p_defines.h
>>>> @@ -51,49 +51,56 @@ enum pipe_error
>>>> /* TODO */
>>>> };
>>>>
>>>> +enum {
>>>
>>> so, I would kinda like to use named enums, and then update the state
>>> structs to use 'em (since it would make gcc and gdb grok them
>>> better).. ofc it is a big change and doesn't have to be done in all
>>> one go, but why not give the enum's names in this first step?
>>
>> Enum type names can't be used everywhere. A lot of states are packed
>> (unsigned x:n). Not sure how enums work with that. In any case, let's
>> add the names later if needed.
>>
> Please name the new enums. All of the ones in gallium/include have names.
If somebody wants to name them, it can be done in another patch. Right
now, no names are needed to compile it.
Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
