Re: Compiler warnings on Debian Sid kfreebsd-i386

2013-08-12 Thread Ken Sharp
On 12/08/13 20:15, Mislav Blazevic wrote: It seems that xmlBufPtr was renamed to xmlBufferPtr in new libxml. It does seem that way. :-) Or maybe :-(

Re: Compiler warnings on Debian Sid kfreebsd-i386

2013-08-12 Thread Mislav Blazevic
8.0+dfsg1-7+nmu1 --> 2.9.1+dfsg1-3) as a >> result and introduced some new compiler warnings: >> >> /home/ken/wine-git/dlls/**msxml3/node.c: In function >> ‘htmldtd_dumpcontent’: >> /home/ken/wine-git/dlls/**msxml3/node.c:908:9: warning: passing argument >

Compiler warnings on Debian Sid kfreebsd-i386

2013-08-12 Thread Ken Sharp
Following my previous e-mail (http://www.winehq.org/pipermail/wine-devel/2013-August/100754.html) I have since moved from Wheezy to Sid to work around a Debian bug. libxml2 has been updated (2.8.0+dfsg1-7+nmu1 --> 2.9.1+dfsg1-3) as a result and introduced some new compiler warnings: /h

Re: Compiler warnings on Debian kfreebsd-i386

2013-08-10 Thread Ken Sharp
On 08/08/13 21:28, Charles Davis wrote: On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote: Some interesting, some not: /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘f

Re: Compiler warnings on Debian kfreebsd-i386

2013-08-09 Thread Ken Sharp
On 10/08/13 00:01, Charles Davis wrote: Did you run autoreconf like I said? No! I'm useless! I'll get back to you tomorrow. Sorry. :(

Re: Compiler warnings on Debian kfreebsd-i386

2013-08-09 Thread Charles Davis
On Aug 9, 2013, at 11:01 AM, Ken Sharp wrote: > On 08/08/13 21:28, Charles Davis wrote: >> >> On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote: >> >>> Some interesting, some not: >>> >>> /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ >>> defined but not used [-Wunused-f

Re: Compiler warnings on Debian kfreebsd-i386

2013-08-09 Thread Ken Sharp
On 08/08/13 21:28, Charles Davis wrote: On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote: Some interesting, some not: /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘f

Re: Compiler warnings on Debian kfreebsd-i386

2013-08-08 Thread Charles Davis
On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote: > Some interesting, some not: > > /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ > defined but not used [-Wunused-function] > /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: > ‘find_owning_pid’ defined but no

Compiler warnings on Debian kfreebsd-i386

2013-08-08 Thread Ken Sharp
Some interesting, some not: /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/jscri

Re: d3d8/d3d9: fix compiler warnings on non x86/x86_64

2010-12-02 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=7411 Your paranoid android.

Re: d3d8/d3d9: fix compiler warnings on non x86/x86_64

2010-12-02 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=7410 Your paranoid android.

Re: dlls/ddraw/executebuffer.c: silence compiler warnings

2009-05-10 Thread Stefan Dösinger
> executebuffer.c:230: warning: comparison between 'D3DRENDERSTATETYPE' and > 'enum _D3DTRANSFORMSTATETYPE' executebuffer.c:232: warning: comparison > between 'D3DRENDERSTATETYPE' and 'enum _D3DTRANSFORMSTATETYPE' > executebuffer.c:234: warning: comparison between 'D3DRENDERSTATETYPE' and > 'enum

Re: NetBSD compiler warnings patches - review request

2009-04-24 Thread David Laight
On Thu, Apr 23, 2009 at 11:46:38AM -0500, Austin English wrote: > Howdy, > > I recently started retesting NetBSD (5.0 this time), and have filed > bugs for most of the warnings. A few of them I have patches ready for, > but figured I should probably get reviewed first: > > There's a NetBSD bug he

Re: NetBSD compiler warnings patches - review request

2009-04-23 Thread Hans Leidekker
On Thursday 23 April 2009 18:46:38 Austin English wrote: > diff --git a/dlls/winhttp/net.c b/dlls/winhttp/net.c > index 79e6789..3c68ee0 100644 > --- a/dlls/winhttp/net.c > +++ b/dlls/winhttp/net.c > @@ -82,7 +82,7 @@ static CRITICAL_SECTION cs_gethostbyname = { > &critsect_debug, -1, 0, 0, 0, 0

NetBSD compiler warnings patches - review request

2009-04-23 Thread Austin English
Howdy, I recently started retesting NetBSD (5.0 this time), and have filed bugs for most of the warnings. A few of them I have patches ready for, but figured I should probably get reviewed first: First up, bug 18161: gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -D_NTSYSTEM_ -D_RE

Re: ntdll: fix a couple compiler warnings on OpenSolaris

2009-03-03 Thread Francois Gouget
> +wine_set_fs( thread_data->fs ); > +wine_set_gs( thread_data->gs ); > +#endif > + > /* get %fs and %gs at time of the fault */ > #ifdef FS_sig > *fs = LOWORD(FS_sig(sigcontext)); Isn't it a problem to move wine_set_{fs,gs}() just before code that purports to 'get %fs and %gs

Re: gdi32: Fixed a couple of Win64 pointer cast compiler warnings

2009-02-13 Thread Alexandre Julliard
Eddie Leung writes: > diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c > index c8a3c4c..44a9a6f 100644 > --- a/dlls/gdi32/dc.c > +++ b/dlls/gdi32/dc.c > @@ -1433,7 +1433,7 @@ static BOOL WINAPI call_dc_hook16( HDC hdc, WORD code, > DWORD_PTR data, LPARAM lP > args[2] = LOWORD(data); >

Re: widl: initialize _RetVal to avoid compiler warnings

2009-02-06 Thread Rob Shearman
function_get_rettype(func->type)); > -print_proxy( " _RetVal;\n"); > + > +/* Initialize _RetVal properly in order to avoid compiler warnings */ > +if (is_ptr(type_function_get_rettype(func->type)) || > is_array(type_function_get_rettype(func-&g

Re: tools/fnt2bdf: fix a few compiler warnings (1/10)

2008-11-16 Thread Austin English
On Sun, Nov 16, 2008 at 3:21 AM, Marcus Meissner <[EMAIL PROTECTED]> wrote: > On Wed, Nov 12, 2008 at 10:28:37AM +0100, Michael Stefaniuc wrote: >> Austin English wrote: >> > This series of patches fixes most of the new warnings generated in gcc >> > 4.3.2. There's one issue left, but I'm not quite

Re: tools/fnt2bdf: fix a few compiler warnings (1/10)

2008-11-16 Thread Michael Stefaniuc
Marcus Meissner wrote: > On Wed, Nov 12, 2008 at 10:28:37AM +0100, Michael Stefaniuc wrote: >> Austin English wrote: >>> This series of patches fixes most of the new warnings generated in gcc >>> 4.3.2. There's one issue left, but I'm not quite sure what should be >>> done on that one. >>> diff --g

Re: tools/fnt2bdf: fix a few compiler warnings (1/10)

2008-11-16 Thread Marcus Meissner
On Wed, Nov 12, 2008 at 10:28:37AM +0100, Michael Stefaniuc wrote: > Austin English wrote: > > This series of patches fixes most of the new warnings generated in gcc > > 4.3.2. There's one issue left, but I'm not quite sure what should be > > done on that one. > > diff --git a/tools/fnt2bdf.c b/too

Re: tools/fnt2bdf: fix a few compiler warnings (1/10)

2008-11-12 Thread Michael Stefaniuc
Austin English wrote: > This series of patches fixes most of the new warnings generated in gcc > 4.3.2. There's one issue left, but I'm not quite sure what should be > done on that one. > diff --git a/tools/fnt2bdf.c b/tools/fnt2bdf.c > index 1a14d6c..3c3134b 100644 > --- a/tools/fnt2bdf.c > +++ b/

Re: Compiler warnings in gcc 4.3.2

2008-11-11 Thread Austin English
On Tue, Nov 11, 2008 at 9:28 PM, James Hawkins <[EMAIL PROTECTED]> wrote: > On Tue, Nov 11, 2008 at 9:17 PM, Austin English <[EMAIL PROTECTED]> wrote: >> I recently upgraded to intrepid, which has gcc 4.3.2. It gave a few >> new warnings, and I've got patches ready for all but this one. I >> starte

Re: Compiler warnings in gcc 4.3.2

2008-11-11 Thread James Hawkins
On Tue, Nov 11, 2008 at 9:17 PM, Austin English <[EMAIL PROTECTED]> wrote: > I recently upgraded to intrepid, which has gcc 4.3.2. It gave a few > new warnings, and I've got patches ready for all but this one. I > started to write a patch to remove the check, since it's useless, but > wasn't sure i

Compiler warnings in gcc 4.3.2

2008-11-11 Thread Austin English
I recently upgraded to intrepid, which has gcc 4.3.2. It gave a few new warnings, and I've got patches ready for all but this one. I started to write a patch to remove the check, since it's useless, but wasn't sure if it should instead be converted to a signed integer instead. [EMAIL PROTECTED]:~/

Re: dlls/d3d9/tests/visual.c -- address five compiler warnings (RESEND)

2008-01-25 Thread Gerald Pfeifer
On Mon, 14 Jan 2008, Alexandre Julliard wrote: >> @@ -4348,12 +4349,12 @@ static void vshader_version_varying_test >> "vs_3_0 returned color 0x%08x, expected 0x00203366\n", color); >> color = getPixelColor(device, 160, 360); >> ok((color & 0x00ff) >= 0x003c && (color & 0x0

Re: dlls/d3d9/tests/visual.c -- address five compiler warnings (RESEND)

2008-01-14 Thread Alexandre Julliard
Gerald Pfeifer <[EMAIL PROTECTED]> writes: > @@ -4348,12 +4349,12 @@ static void vshader_version_varying_test > "vs_3_0 returned color 0x%08x, expected 0x00203366\n", color); > color = getPixelColor(device, 160, 360); > ok((color & 0x00ff) >= 0x003c && (color & 0x00ff

Re: libs/wine: Avoid compiler warnings about unused parameter(-W)

2007-11-05 Thread Alexandre Julliard
Detlef Riekenberg <[EMAIL PROTECTED]> writes: > Changelog: > libs/wine: Avoid compiler warnings about unused parameter(-W) > > The code-sequence is optimized away. > Another option would be "__attribute__((unused__))" > or removing the function. The best o

Re: libs/wine: Avoid compiler warnings about unused parameter(-W)

2007-11-04 Thread Michael Stefaniuc
Detlef Riekenberg wrote: > Changelog: > libs/wine: Avoid compiler warnings about unused parameter(-W) > > > The code-sequence is optimized away. > Another option would be "__attribute__((unused__))"

Re: libs/wpp: Avoid compiler warnings (-W)

2007-11-04 Thread Dan Hipschman
On Sun, Nov 04, 2007 at 01:04:02PM +0100, Detlef Riekenberg wrote: > > Why is the code only used, when "WANT_NEAR_INDICATION" is defined? > Should we remove the code or enable it always? > > @@ -591,18 +592,17 @@ static void generic_msg(const char *s, c > fprintf(stderr, "%s:%d:%d: %s: ", pp

Re: Janitorial : Use Compiler Warnings ( -Wmissing-declarations ) ( #1, take 2 )

2005-01-11 Thread Alexandre Julliard
Rémi Assailly <[EMAIL PROTECTED]> writes: > diff -u libs/wine/debug.c libs/wine/debug.c > --- libs/wine/debug.c 2005-01-10 20:12:34.748897920 +0100 > +++ libs/wine/debug.c 2005-01-10 19:18:26.0 +0100 > @@ -61,6 +61,9 @@ > return strcmp( name, *chan + 1 ); > } > > +void *__wine_dbg_

Re: Janitorial : Use Compiler Warnings ( -Wmissing-declarations ) ( #1 )

2005-01-10 Thread Alexandre Julliard
Rémi Assailly <[EMAIL PROTECTED]> writes: > Can we create "private" headers ? headers which will be only used in files > where > some declarations are missing ? No, at least not in this case. These are used in generated files that must not depend on other headers. -- Alexandre Julliard [EMAIL

Re: Janitorial : Use Compiler Warnings ( -Wmissing-declarations ) ( #1 )

2005-01-10 Thread Rémi Assailly
Selon Alexandre Julliard <[EMAIL PROTECTED]>: > Rémi Assailly <[EMAIL PROTECTED]> writes: > > > --- include/wine/library.h 2005-01-07 19:41:26.0 +0100 > > +++ include/wine/library.h 2005-01-07 19:49:41.389057824 +0100 > > @@ -42,6 +42,7 @@ > > extern void *wine_dlopen( const char *filen

Re: Janitorial : Use Compiler Warnings ( -Wmissing-declarations ) ( #1 )

2005-01-09 Thread Alexandre Julliard
Rémi Assailly <[EMAIL PROTECTED]> writes: > --- include/wine/library.h2005-01-07 19:41:26.0 +0100 > +++ include/wine/library.h2005-01-07 19:49:41.389057824 +0100 > @@ -42,6 +42,7 @@ > extern void *wine_dlopen( const char *filename, int flag, char *error, int > errorsize ); > ext

compiler warnings

2003-11-27 Thread Ivan Leo Murray-Smith
The current code compiles with a few warnings, in the attachment. BTW you can see how code degenerates rapidly when it's not free software any more, probably 10% of all the output when compiling winex 3.2 is compiler warnings. warnings Description: Binary data

New compiler warnings

2003-10-28 Thread Gerald Pfeifer
My nightly autobuilder on FreeBSD 5.1 noticed the following warning introduced yesterday: signal_i386.c:524: warning: `save_context' defined but not used signal_i386.c:602: warning: `restore_context' defined but not used Also, there are two OpenGL warnings left, after most have been fixed a f