general question..

Chris Ahrendt celticht32 at aol.com
Mon Jul 21 22:09:44 CDT 2008


Ivan Gyurdiev wrote:
> James Hawkins wrote:
>>> context.c - same except in this case its just a return with noting 
>>> else..
>>> why do a goto why not just do the return?
>>>
>>>     
>>
>> This is probably ok to change.  I can only imagine that the original
>> author thought there might be cleanup needed at some later point.
>>   
> The bigger question is why there is a huge if-else statement, and why 
> this function is so large.
> Huge if-else statement = 2 sub-functions
> 
> Shader dirty constants - do shader internals really belong here ?
> 
> Ivan
probably not this is the sort of discussion I was wanting to start with 
the GOTO comment =)


Chris



More information about the wine-devel mailing list