Re: [Gnustep-cvs] r26307 - /libs/gui/trunk/Source/NSApplication.m

2008-03-28 Thread Fred Kiefer
Fabien Vallon wrote:
 On Wed, 26 Mar 2008, Fred Kiefer wrote:
 Riccardo Mottola wrote:
 Author: rmottola
 Date: Sun Mar 16 02:02:52 2008
 New Revision: 26307

 URL: http://svn.gna.org/viewcvs/gnustep?rev=26307view=rev
 Log:
 changed terminate type to BOOL since it is one and the value froma delegate 
 got wrongly cast

 Modified:
 libs/gui/trunk/Source/NSApplication.m

 No it is not. The return value of applicationShouldTerminate: is of type
  NSApplicationTerminateReply and has been this since some time. It used
 to be a BOOL in OpenStep. Could you please undo your change that
 prevents the value of NSTerminateLater from being passed on and we may
 check together what did go wrong in your original delegate.
 
 I reported this bug on #gnustep.
 
 You can try this simple example  here : 
 http://www.sonappart.net/gnustep/appdelegatebug.tar.gz
 
 You can not quit the application.
 

Strange, this application works for me even with the old GNUstep code.
Which environment are you on?
To be honest I don't see, how this could have failed even on a 64 bit
system, but then you never know.
Does it work with Richard latest change? That code looks as save as
possible to me.

Fred




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


Re: [Gnustep-cvs] r26307 - /libs/gui/trunk/Source/NSApplication.m

2008-03-27 Thread Fabien Vallon

Hi,

On Wed, 26 Mar 2008, Fred Kiefer wrote:

 Riccardo Mottola wrote:
  Author: rmottola
  Date: Sun Mar 16 02:02:52 2008
  New Revision: 26307
  
  URL: http://svn.gna.org/viewcvs/gnustep?rev=26307view=rev
  Log:
  changed terminate type to BOOL since it is one and the value froma delegate 
  got wrongly cast
  
  Modified:
  libs/gui/trunk/Source/NSApplication.m
  
 
 No it is not. The return value of applicationShouldTerminate: is of type
  NSApplicationTerminateReply and has been this since some time. It used
 to be a BOOL in OpenStep. Could you please undo your change that
 prevents the value of NSTerminateLater from being passed on and we may
 check together what did go wrong in your original delegate.

I reported this bug on #gnustep.

You can try this simple example  here : 
http://www.sonappart.net/gnustep/appdelegatebug.tar.gz

You can not quit the application.

Fabien.


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


Re: [Gnustep-cvs] r26307 - /libs/gui/trunk/Source/NSApplication.m

2008-03-26 Thread Fred Kiefer
Riccardo Mottola wrote:
 Author: rmottola
 Date: Sun Mar 16 02:02:52 2008
 New Revision: 26307
 
 URL: http://svn.gna.org/viewcvs/gnustep?rev=26307view=rev
 Log:
 changed terminate type to BOOL since it is one and the value froma delegate 
 got wrongly cast
 
 Modified:
 libs/gui/trunk/Source/NSApplication.m
 

No it is not. The return value of applicationShouldTerminate: is of type
 NSApplicationTerminateReply and has been this since some time. It used
to be a BOOL in OpenStep. Could you please undo your change that
prevents the value of NSTerminateLater from being passed on and we may
check together what did go wrong in your original delegate.

Fred


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


Re: [Gnustep-cvs] r26307 - /libs/gui/trunk/Source/NSApplication.m

2008-03-26 Thread Gregory John Casamento
The change wasn't to the return type, but to what it's being compared to in the 
method.   We changed the declaration of a local variable to BOOL so that the 
comparison would be done properly.

You want this changed back?
 
Gregory Casamento -- Principal Consultant - OLC, Inc 
# GNUstep Chief Maintainer

- Original Message 
From: Fred Kiefer [EMAIL PROTECTED]
To: Riccardo Mottola [EMAIL PROTECTED]
Cc: GNUstep Developer gnustep-dev@gnu.org
Sent: Wednesday, March 26, 2008 5:03:55 AM
Subject: Re: [Gnustep-cvs] r26307 - /libs/gui/trunk/Source/NSApplication.m

Riccardo Mottola wrote:
 Author: rmottola
 Date: Sun Mar 16 02:02:52 2008
 New Revision: 26307
 
 URL: http://svn.gna.org/viewcvs/gnustep?rev=26307view=rev
 Log:
 changed terminate type to BOOL since it is one and the value froma delegate 
 got wrongly cast
 
 Modified:
 libs/gui/trunk/Source/NSApplication.m
 

No it is not. The return value of applicationShouldTerminate: is of type
 NSApplicationTerminateReply and has been this since some time. It used
to be a BOOL in OpenStep. Could you please undo your change that
prevents the value of NSTerminateLater from being passed on and we may
check together what did go wrong in your original delegate.

Fred


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





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