Keith Whitwell pisze:
> On Sun, 2009-12-13 at 15:27 -0800, Marek Olšák wrote:
>
>> +static INLINE
>> +void util_blitter_save_fragment_sampler_states(
>> + struct blitter_context *blitter,
>> + int num_sampler_states,
>> + void **sampler_states)
>> +{
>> + assert(num_textures <= 32);
>> +
>> + blitter->saved_num_sampler_states = num_sampler_states;
>> + memcpy(blitter->saved_sampler_states, sampler_states,
>> + num_sampler_states * sizeof(void *));
>> +}
>> +
>>
>
> Have you tried compiling with debug enabled? The assert above fails to
> compile. Also, can you use Elements() or similar instead of the
> hard-coded 32?
>
> Maybe we can figure out how to go back to having asserts keep exposing
> their contents to the compiler even on non-debug builds. This used to
> work without problem on linux and helped a lot to avoid these type of
> problems.
>
>
Precisely. Recently I've been thinking about mapping assert() to
__assume() for non-debug builds on windows and MSVC.
http://msdn.microsoft.com/en-us/library/1b3fsfxw%28VS.80%29.aspx
------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
Mesa3d-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
