Re: [Gnustep-cvs] r29139 - in /libs/gui/trunk: ChangeLog Source/NSSliderCell.m

[Eric Wasylishen]

Hi Fred,

On 2009-12-19, at 2:29 PM, Fred Kiefer wrote:

> The change to setMinValue: and setMaxValue: surely are correct and the
> addition of setObjectValue: is just great. What I don't understand is
> why the method setDoubleValue:,  setFloatValue: and setIntValue: had to
> be implemented. The default behaviour (calling setObjectValue:) should
> just do the right thing.

Ah, you're right, setDoubleValue:,  setFloatValue: and setIntValue: shouldn't
be necessary.

> Sadly this is not true for setStringValue:, where the current GNUstep
> implementation on NSCell has a short cut which doesn't call
> setObjectValue:. This should be corrected instead of having a separate
> implementation here.

Sounds good, I will look at fixing -[NSCell setStringValue:], and then we can
remove -[NSSliderCell setStringValue:]. 

> What should be the default value for isContinuous when the cell is just
> created? When we get that right we may not have to set that value when
> dearchiving, as it will get archived as action mask by the super class.
> 

I just checked in F-Script on Snow Leopard:

> NSSliderCell new isContinuous
YES
> NSActionCell new isContinuous
NO
> NSCell new isContinuous
NO

So maybe we can just add [self setContinuous: YES] to NSSliderCell's -init,
and remove my changes to NSSliderCell dearchiving?

> As for the commit access a simple mail to Adam should be sufficient to
> get that. A line in a commit comment wont help. Who is supposed to read
> that?
I'll send an email to Adam.

Thanks,
Eric