[MPlayer-dev-eng] [PATCH] Support locale under Windows console

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Thu Mar 1 12:54:17 CET 2007


Hello,
On Thu, Mar 01, 2007 at 07:00:51PM +0800, Zuxy Meng wrote:
> 2007/3/1, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> >On Thu, Mar 01, 2007 at 06:10:21PM +0800, Zuxy Meng wrote:
> >> 2007/3/1, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> >> >On Thu, Mar 01, 2007 at 12:39:20PM +0800, Zuxy Meng wrote:
> >> >> -
> >> >> +#include <stdio.h>
> >> >>  #include <windows.h>
> >> >>  #include "keycodes.h"
> >> >>  #include "input/input.h"
> >> >>
> >> >>  int mp_input_win32_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){
> >> >> +  HANDLE hd_stdin = GetStdHandle(STD_INPUT_HANDLE);
> >> >> +  if(!PeekNamedPipe(hd_stdin, NULL, size, &retval, NULL, NULL) ||
> >> >!retval){
> >> >
> >> >
> >> >just put the stdio.h include into the USE_ICONV ifdef below. That's not
> >> >too beautiful but not more ugly than this...
> >>
> >> The purpose to include <stdio.h> was to avoid warnings about
> >> undeclared "snprintf": just don't want to introduce more warnings in
> >> the new code. However, "printf" had been used in old code, too; so
> >> after I did what you suggested, there were still warnings. So we can
> >> either:
> >> 1. Ignore the warning and don't bother <stdio.h> at all.
> >> 2. Rename stdin....
> >
> >3. #undef stdin after the include
> >4. change printf to mp_msg in a separate patch
> >5. add stdio.h and rename stdin in a separate patch
> 
> OK. Let's deal with this header later.

Looks fine to me. Anyone who can test: feel free to apply. Otherwise I
will do it, but that may take some time.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list