discuss-gnustep
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Gorm don't work


From: Germán Arias
Subject: Re: Gorm don't work
Date: Fri, 25 Jan 2019 12:21:56 -0600

El vie, 25-01-2019 a las 12:47 +0000, David Chisnall escribió:
> 
> On 25/01/2019 12:23, Fred Kiefer wrote:
> > > Am 25.01.2019 um 12:29 schrieb David Chisnall<gnustep@theravensne
> > > st.org>:
> > > 
> > > On 22/01/2019 22:15, Germán Arias wrote:
> > > > 2019-01-22 16:01:48.065 Gorm[5259:5259]
> > > > GormDocument.m:109  Assertion
> > > > failed in GormDocument(instance), method _docWindow.  Unable to
> > > > find
> > > > _window ivar in NSDocument class
> > > 
> > > This looks like it's in the code I changed for compatibility with
> > > the new ABI.  Gorm is now using reflection to find private ivars
> > > in GNUstep, rather than accessing them directly (accessing
> > > private ivars in the new ABI will now cause a link failure
> > > anywhere other than the compilation unit containing the
> > > @implementation).
> > > 
> > > If this assertion fails, it means that the Objective-C compiler /
> > > runtime that you are using is not able to find reflection
> > > metadata for the _window ivar in NSDocument.  This is quite
> > > worrying, because looking up ivars via the reflection APIs worked
> > > even back in the GCC 3.x days.
> > 
> > It is a lot easier than that. You just made a small mistake and
> > used an NSString here instead of a C String. Of course this cannot
> > work, with neither of the compilers. Which just shows that nobody
> > did recompile and use Gorm after your change.
> > I fixed this in git and this specific issue should be gone now.
> > There are a lot of other compiler warnings I get when compiling
> > Gorm and there is also the runtime warning about _popUpItemAction:,
> > but that should not block the usage of Gorm.
> 
> I did recompile it (and have a FreeBSD package of it built from the
> git 
> version including my change installed), but I missed the compiler 
> warning in a sea of other ones.  This is why -Werror is a good idea.
> 
> I ran Gorm to test, but apparently managed not to trigger this code
> path.
> 
> David
> 

Thanks Fred, now works.

Germán




reply via email to

[Prev in Thread] Current Thread [Next in Thread]