[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