[MPlayer-dev-eng] [PATCH] getch2() support for OS/2
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Wed Feb 20 11:33:42 CET 2008
Hello,
On Wed, Feb 20, 2008 at 06:28:55PM +0900, KO Myung-Hun wrote:
> Index: osdep/getch2-win.c
> ===================================================================
> --- osdep/getch2-win.c (revision 26012)
> +++ osdep/getch2-win.c (working copy)
> @@ -13,7 +13,7 @@
> // HACK, stdin is used as something else below
> #undef stdin
>
> -int mp_input_win32_slave_cmd_func(int fd,char* dest,int size){
> +int mp_input_slave_cmd_func(int fd,char* dest,int size){
> DWORD retval;
> HANDLE stdin = GetStdHandle(STD_INPUT_HANDLE);
> if(!PeekNamedPipe(stdin, NULL, size, &retval, NULL, NULL) || !retval){
I do not like this renaming at all, this is still code that is full of
Windows-API stuff. Sure it is used for OS/2 as well, but is is still the
Windows API it uses, so the previous name still seems more correct.
Add a comment that it also handles OS/2 if you want.
Or if you really want to rename if make it win32_os2...
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list