Hi, Tom
In general, I like your changes to the windows videodriver, because we get rid of the special GRXMain entry point. It has some rough edges: pressing the close button is catastrophic (at least in W98) and the last app doesnt't get the focus when the GRX app end. But is a good start.
Some suggestions:
- Setmode. Create the window_thread only if mp->extinfo->mode != GR_frametext, and destroy the window_thread (if created) else.
- Detect. The detect function can be called more than one time, so it is necesary to have a static variable to signal it and return TRUE if the work was done. Move the InitialiceCriticalSection to "init" and add DeleteCriticalSection to "reset".
- Reset. Destoy the window_thread if was created.
- WM_CLOSE. We need to maintain the close dialog, people must be warned they are avorting the program. If user says yes, move the message to the queue, and let abort the program in the input driver (the main thread) mouse/w32inp.c.
Greetings, Mariano.