[Gnustep-cvs] gnustep/core/gui ChangeLog Headers/AppKit/NSInp...

2005-10-19 Thread Fred Kiefer 
CVSROOT:/cvsroot/gnustep
Module name:gnustep
Branch: 
Changes by: Fred Kiefer <[EMAIL PROTECTED]> 05/10/19 23:54:27

Modified files:
core/gui   : ChangeLog 
core/gui/Headers/AppKit: NSInputManager.h 
core/gui/Source: GSTitleView.m NSColor.m NSImageCell.m 
 NSMenuView.m NSResponder.m NSSavePanel.m 
 NSTabView.m NSTableHeaderView.m 

Log message:
Removed gcc 4.02 compiler warnings.

CVSWeb URLs:
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/ChangeLog.diff?tr1=1.2572&tr2=1.2573&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Headers/AppKit/NSInputManager.h.diff?tr1=1.3&tr2=1.4&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/GSTitleView.m.diff?tr1=1.11&tr2=1.12&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSColor.m.diff?tr1=1.55&tr2=1.56&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSImageCell.m.diff?tr1=1.27&tr2=1.28&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSMenuView.m.diff?tr1=1.105&tr2=1.106&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSResponder.m.diff?tr1=1.41&tr2=1.42&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSSavePanel.m.diff?tr1=1.87&tr2=1.88&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSTabView.m.diff?tr1=1.47&tr2=1.48&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSTableHeaderView.m.diff?tr1=1.30&tr2=1.31&r1=text&r2=text

Re: [Gnustep-cvs] gnustep/core/gui ChangeLog Headers/AppKit/NSInp...

2005-10-20 Thread David Ayers 
Fred Kiefer schrieb:
> CVSROOT:  /cvsroot/gnustep
> Module name:  gnustep
> Branch:   
> Changes by:   Fred Kiefer <[EMAIL PROTECTED]> 05/10/19 23:54:27
> 
> Modified files:
>   core/gui   : ChangeLog 
>   core/gui/Headers/AppKit: NSInputManager.h 
>   core/gui/Source: GSTitleView.m NSColor.m NSImageCell.m 
>NSMenuView.m NSResponder.m NSSavePanel.m 
>NSTabView.m NSTableHeaderView.m 
> 
> Log message:
>   Removed gcc 4.02 compiler warnings.
> 
> CVSWeb URLs:
> http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/ChangeLog.diff?tr1=1.2572&tr2=1.2573&r1=text&r2=text
> http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Headers/AppKit/NSInputManager.h.diff?tr1=1.3&tr2=1.4&r1=text&r2=text
> http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/GSTitleView.m.diff?tr1=1.11&tr2=1.12&r1=text&r2=text

--- gnustep/gnustep/core/gui/Source/GSTitleView.m   2005/05/26 02:52:43 
1.11
+++ gnustep/gnustep/core/gui/Source/GSTitleView.m   2005/10/19 23:54:22 
1.12
@@ -351,7 +351,7 @@
 }

 // We do not need app menu over menu
-- (void) rightMouseDown: (NSEvent*)theEvent
+- (NSMenu *) rightMouseDown: (NSEvent*)theEvent
 {
 }

Doesn't this give you a warning about control reaching the end of a
non-void function?  If it doesn't, then that looks like a gcc regression.

Cheers,
David


___
Gnustep-dev mailing list
Gnustep-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnustep-dev

Re: [Gnustep-cvs] gnustep/core/gui ChangeLog Headers/AppKit/NSInp...

2005-10-20 Thread Fred Kiefer 
David Ayers wrote:

> --- gnustep/gnustep/core/gui/Source/GSTitleView.m 2005/05/26 02:52:43 
> 1.11
> +++ gnustep/gnustep/core/gui/Source/GSTitleView.m 2005/10/19 23:54:22 
> 1.12
> @@ -351,7 +351,7 @@
>  }
> 
>  // We do not need app menu over menu
> -- (void) rightMouseDown: (NSEvent*)theEvent
> +- (NSMenu *) rightMouseDown: (NSEvent*)theEvent
>  {
>  }
> 
> Doesn't this give you a warning about control reaching the end of a
> non-void function?  If it doesn't, then that looks like a gcc regression.
> 
Oops, I did change the wrong method here. I wanted to change the next one.

Thank you for pointing out

Fred


___
Gnustep-dev mailing list
Gnustep-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnustep-dev