[Mesa-dev] [PATCH 02/29] i965: Pull out INTEL_DEBUG handling into new intel_debug.[ch] files.
Kenneth Graunke
kenneth at whitecape.org
Mon Sep 30 20:55:48 PDT 2013
On 09/30/2013 05:11 PM, Ian Romanick wrote:
> On 09/27/2013 04:45 PM, Kenneth Graunke wrote:
>> Now that there isn't an intel_context structure, the split between
>> brw_context.[ch] and intel_context.[ch] is rather awkward and arbitrary.
>> Removing intel_context.[ch] seems desirable, but not everything really
>> belongs in brw_context.[ch], either.
>>
>> Moving INTEL_DEBUG handling into separate intel_debug.[ch] files should
>> make them relatively easy to find.
>
> I'm not sure I have a preference, but it seems like this should be
> brw_debug.[ch].
I'm happy to use either name. I chose intel_debug.[ch] to match the
name of the environment variable. But brw_debug.[ch] probably does make
more sense.
Do other people have a preference?
More information about the mesa-dev
mailing list