[PATCH] Re: help fixing up SetConsoleCtrlHandler()

Peter Riocreux par+wine_devel at silistix.com
Thu Apr 15 06:58:41 CDT 2004


Here is my first cut. 

Remember that this is my first patch, so go easy on me with he
flamethrowers. I know the indenting isn't quite right.

Probably easiest to review by comparing old and new rather than
looking at old and patch.

Comments, especially regarding more appropriate error values to use,
are welcome.

I think that, despite what Juan Lang advised, I need to detect version
numbers as the FIXME can be triggered by perfecly legal use in
XP/NT/?? but illegal code in ME/98/95. I am surprised that there is
not some global enum-type variable containing a value that tells us
which Windows we are pretending to be. There must be many more
functions than this whose behaviour changed between OSes.

Peter

-------------- next part --------------
A non-text attachment was scrubbed...
Name: console.diff
Type: text/x-patch
Size: 2134 bytes
Desc: SetConsoleCtrlHandler patch
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20040415/5bda57d8/console.bin


More information about the wine-devel mailing list