Michael Hanni | 1 Aug 2003 01:30

Re: [RFC] NSTabView* rewrite (patch)

Hi,

--- Alexander Malmberg <alexander <at> malmberg.org> wrote:
> Adam Fedor wrote:
> [snip]
> >
> > This could use the base method:
> > 
> > return [_items makeImmutableCopyOnFail: YES];
> 
> That wouldn't work. It might make the _items instance immutable, which
> would break things when you try to add or remove items later.

Ok, I'll just return the array without modification.

Thanks,

Michael
nobody | 1 Aug 2003 01:44
Picon

[bug #4568] NSTableView memory leaks

=================== BUG #4568: FULL BUG SNAPSHOT ===================
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=4568&group_id=99

Submitted by: None                    Project: GNUstep                      
Submitted on: Thu 07/31/2003 at 19:44
Category:  Gui/AppKit                 Severity:  5 - Major                  
Bug Group:  Bug                       Resolution:  None                     
Assigned to:  None                    Status:  Open                         

Summary:  NSTableView memory leaks

Original Submission:  These are memory leaks, no?

No Followups Have Been Posted

CC list is empty

File Attachments
****************

-------------------------------------------------------
Date: Thu 07/31/2003 at 19:44  Name: table.patch  Size: 1KB   By: None
Patch
http://savannah.gnu.org/bugs/download.php?group_id=99&amp;bug_id=4568&amp;bug_file_id=568

For detailed info, follow this link:
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=4568&group_id=99

_______________________________________________
  Message sent via/by Savannah
(Continue reading)

David Ayers | 1 Aug 2003 01:47
Picon

[Patch/base] Header reorganization

Commited:

	* Created tag 'pre-header-reorg-20030731'.

	* Headers/Additions/GNUstepBase: New directory.
	* Headers/Additions/GNUstepBase/unicode: Ditto.
	* Headers/Foundation: Ditto.
	* Headers/gnustep/base: Obsoleted.
	* Headers/gnustep/unicode: Ditto.

	* Headers/Additions/GNUstepBase/DistributedObjects.h: Moved here
	from Headers/gnustep/base.
	* Headers/Additions/GNUstepBase/GCObject.h: Ditto.
	* Headers/Additions/GNUstepBase/GNUstep.h: Ditto.
	* Headers/Additions/GNUstepBase/GSCategories.h: Ditto.
	* Headers/Additions/GNUstepBase/GSConfig.h.in: Ditto.
	* Headers/Additions/GNUstepBase/GSFileHandle.h: Ditto.
	* Headers/Additions/GNUstepBase/GSIArray.h: Ditto.
	* Headers/Additions/GNUstepBase/GSIMap.h: Ditto.
	* Headers/Additions/GNUstepBase/GSLocale.h: Ditto.
	* Headers/Additions/GNUstepBase/GSMime.h: Ditto.
	* Headers/Additions/GNUstepBase/GSObjCRuntime.h: Ditto.
	* Headers/Additions/GNUstepBase/GSUnion.h: Ditto.
	* Headers/Additions/GNUstepBase/GSXML.h: Ditto.
	* Headers/Additions/GNUstepBase/Unicode.h: Ditto.
	* Headers/Additions/GNUstepBase/behavior.h: Ditto.
	* Headers/Additions/GNUstepBase/config.h.in: Ditto.
	* Headers/Additions/GNUstepBase/objc-gnu2next.h: Ditto.
	* Headers/Additions/GNUstepBase/preface.h.in: Ditto.
	* Headers/Additions/GNUstepBase/thr-mach.h: Ditto.
(Continue reading)

David Ayers | 1 Aug 2003 01:50
Picon

[Patch/gui] Header reorganization

Commited:

	* Created tag 'pre-header-reorg-20030731'.
	
	* Headers/Additions/GNUstepGUI: New directory.
	* Headers/AppKit: Ditto.
	* Headers/gnustep/gui: Obsoleted.

	* Headers/Additions/GNUstepGUI/GMAppKit.h: Moved here from
	Headers/gnustep/gui.
	* Headers/Additions/GNUstepGUI/GMArchiver.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSDisplayServer.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSFontInfo.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSFusedSilica.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSHbox.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSHelpManagerPanel.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSHorizontalTypesetter.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSInfoPanel.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSLayoutManager.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSLayoutManager_internal.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSMemoryPanel.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSMethodTable.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSNibTemplates.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSPasteboardServer.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSServicesManager.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSTable.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSTextConverter.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSTrackingRect.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSTypesetter.h: Ditto.
	* Headers/Additions/GNUstepGUI/GSVbox.h: Ditto.
(Continue reading)

David Ayers | 1 Aug 2003 02:31
Picon

[Patch/back] Header reorganization

Commited:

        * Created tag 'pre-header-reorg-20030731'.

        * Source/art/GNUmakefile: Do not make headers publilc.
        * Source/gsc/GNUmakefile: Ditto.
        * Source/x11/GNUmakefile: Ditto.
        * Source/xdps/GNUmakefile: Ditto.
        * Source/xlib/GNUmakefile: Ditto.

        * Headers/win32/*.h: Update includes for new header structure.
        * Headers/winlib/*.h: Ditto.
        * Headers/x11/XGServer.h: Ditto.
        * Headers/xlib/*.h: Ditto.

        * Source/art/ftfont.m: Ditto.
        * Source/gsc/*.m: Ditto.
        * Source/x11/*.m: Ditto.
        * Source/xdps/*.m: Ditto.
        * Source/xlib/*.m: Ditto.

        * Tools/gpbs.m: Ditto.

Cheers,
David
Adam Fedor | 1 Aug 2003 05:09

Re: NSLog() / NSLogv() annoyance


On Thursday, July 31, 2003, at 10:54 AM, Chris B. Vetter wrote:

> On 30 Jul 2003 21:47:29 -0600
> Adam Fedor <fedor <at> doc.com> wrote:
> [...]
>>> Can't figure out where errno is set to 2, though I suspect it's
>>> coming from _NSLog_standard_printf_handler() trying to write to
>>> syslog(?).
>> I traced this down (and down, etc) - it actually happens when NSLog
>> gets a user default, and NSUserDefaults gets set up and looks for
>> .GNUsteprc, eventually getting to:
>>   +[GSAttrDictionary attributesAt: traverseLink:]
>> which does a 'stat' on the file which doesn't exist. Anyway, what
>> should GNUstep do in this case? Reset errno to 0? The documentation
>> says the value isn't significant unless a library function returns an
>> error, so it really is undefined in this case.
>
> That's interesting, because .GNUsteprc does exist, though it's empty.
>
Well it's looking for the one in the System root.

>
> Instead, it could/should "back up" the original value of errno, call
> stat(2) and do some kind of evaluation if necessary, then restore the
> original?
>
>

So NSLog should back-up the original? It's kind-of crazy, but probably 
(Continue reading)

Richard Frith-Macdonald | 1 Aug 2003 08:10
Picon

Re: NSLog() / NSLogv() annoyance


On Friday, August 1, 2003, at 04:09 AM, Adam Fedor wrote:

>
> On Thursday, July 31, 2003, at 10:54 AM, Chris B. Vetter wrote:
>
>> On 30 Jul 2003 21:47:29 -0600
>> Adam Fedor <fedor <at> doc.com> wrote:
>> [...]
>>>> Can't figure out where errno is set to 2, though I suspect it's
>>>> coming from _NSLog_standard_printf_handler() trying to write to
>>>> syslog(?).
>>> I traced this down (and down, etc) - it actually happens when NSLog
>>> gets a user default, and NSUserDefaults gets set up and looks for
>>> .GNUsteprc, eventually getting to:
>>>   +[GSAttrDictionary attributesAt: traverseLink:]
>>> which does a 'stat' on the file which doesn't exist. Anyway, what
>>> should GNUstep do in this case? Reset errno to 0? The documentation
>>> says the value isn't significant unless a library function returns an
>>> error, so it really is undefined in this case.
>>
>> That's interesting, because .GNUsteprc does exist, though it's empty.
>>
> Well it's looking for the one in the System root.
>
>>
>> Instead, it could/should "back up" the original value of errno, call
>> stat(2) and do some kind of evaluation if necessary, then restore the
>> original?
>>
(Continue reading)

Roland Schwingel | 1 Aug 2003 10:11
Picon
Favicon

NSProxy.m: Tiny compile problem on windows

Hi...

When trying to compile 1.7.2 on windows I can't get NSProxy.m compiled, 
because it cannot find UINT_MAX definition. I had to add these to 
NSProxy.m to the include section to get it compiled... Maybe NSProxy.m 
should be changed that way...

#ifdef HAVE_LIMITS_H
#include <limits.h>
#endif

Thanks,

Roland
Roland Schwingel | 1 Aug 2003 10:32
Picon
Favicon

NSProcessInfo.m: Problem with **LONG** commandline on linux

Hi...

Last night I ran into a problem with NSProcessInfo.m (from 1.7.2) on 
linux which caused GNUstep to crash... It happened to me to get a 4561 
Bytes long commandline to run a tool (toolname + 43 Arguments)... Ok I 
know this isnt very common but maybe NSProcessInfo shouldn't crash 
handling it. What exactly happens is still a bit mysterious... But when 
entering _gnu_process_args() (called by +[NSProcessInfo initialize] 
there is at present a check

  if (argv != 0)
    {
          _gnu_arg_zero = (char*)malloc(strlen(argv[0]) + 1);
          strcpy(_gnu_arg_zero, argv[0]);
    }
  else
   ....

Well in my case argv was a pointer but argv[0] was NULL, so strlen() 
crashes...I changed it to look that way

  if (argv != 0)
    {
        if (argv[0] != NULL)
        {
          _gnu_arg_zero = (char*)malloc(strlen(argv[0]) + 1);
          strcpy(_gnu_arg_zero, argv[0]);
      }
    }
  else
(Continue reading)

Richard Frith-Macdonald | 1 Aug 2003 19:34
Picon

Fwd: NSLog() / NSLogv() annoyance

Forgot to cc this.

Begin forwarded message:

> From: Richard Frith-Macdonald <richard <at> brainstorm.co.uk>
> Date: Fri Aug 1, 2003  6:32:52 PM Europe/London
> To: "Chris B. Vetter" <chrisv <at> web4inc.com>
> Subject: Re: NSLog() / NSLogv() annoyance
>
>
> On Friday, August 1, 2003, at 05:55 PM, Chris B. Vetter wrote:
>
>> On Fri, 1 Aug 2003 15:46:05 +0100
>> Richard Frith-Macdonald <richard <at> brainstorm.co.uk> wrote:
>> [...]
>>> I know you know this but ... it's easy to overlook when you are used
>>> to working with objects ... in C/ObjC, function arguments are passed
>>> by value rather than by reference.
>>> This means that if (the value of) errno is passed as an argument to
>>> NSLog, the value logged will be the original value passed 
>>> irrespective
>>> of any changes to the contents of the errno
>>> variable which might be made inside NSLog itsself.
>>> Any other behavior is a compiler bug.
>>
>> Yes, but check an earlier mail I sent. If you do a system call, then 
>> use
>> NSLog() to print some info regarding the call, and then check errno,
>> errno is set to 2, which is a bad thing (IMHO) since the error 
>> referred
(Continue reading)


Gmane