gnustep-dev
[Top][All Lists]
Advanced

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

Re: NSButtonImageSource weird code


From: Fred Kiefer
Subject: Re: NSButtonImageSource weird code
Date: Wed, 10 Jul 2013 23:48:11 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6

On 10.07.2013 22:08, Vincent R. wrote:
Le 10.07.2013 21:43, Gregory Casamento a écrit :
Have you made the change you suggest and tested it?

No I just LOOKED at the source code and I found it "weird".
I am not even saying that it doesn't work, not tested anything.
If you are not shocked by the line it means there is nothing.
Oh I think I understand why I find it weird it's because the key is not
yet assigned before the call
of setObject, the setObject will assign the sources->imageName (don't
know if it's clear ...)
Sorry for the noise ->[]

You are correct, the code here is wrong and should actually not even compile. What is far worse is that this code here never gets used anywhere and isn't included in the GNUmakefile. There is another implementation of this class in GSNibLoading.h and this is the one we use.

I need to clean up this code by removing one of these implementations. Most likely the one with the bug :-)

Fred




reply via email to

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