general question..

[Ivan Gyurdiev]

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
>
The GL initialization stuff at the end should also go into a subfunction 
imho.
It really looks like the core of the function control flow does this, 
unless I'm misunderstanding:

hdc = get_hdc_somehow() [ offscreen and onscreen choices ]
ctx = create_new_context(hdc)
switch_context(new_ctx)
initialize_gl_stuff_in_new_context(new_ctx)
switch_context(old_ctx)

The rest is unimportant details like making it work properly :)

Ivan