BorderStyle>>inset that return the width is not a good method, since there
is a BorderStyle named #inset.

This will just obfuscate.

Here one should use BorderStyle>>width instead

Karl




On Tue, May 12, 2015 at 12:02 PM, karl ramberg <karlramberg at gmail.com>
wrote:

> SimpleBorder>>width
> ^width ifNil:[0]
>
> I'll push this to trunk if there are no protests.
>
> Simple stuff like this should work:
> RectangleMorph new borderStyle: BorderStyle simple; openInWorld
>
> Karl
>
> On Tue, May 12, 2015 at 10:57 AM, karl ramberg <karlramberg at gmail.com>
> wrote:
>
>> Maybe we need
>> SimpleBorder>>width
>> ^width ifNil:[ ^0]
>>
>> On Tue, May 12, 2015 at 10:47 AM, karl ramberg <karlramberg at gmail.com>
>> wrote:
>>
>>> There is also BorderStyle class inset and BorderStyle inset.
>>> One returns a class, one a integer.
>>>
>>> BorderStyle simple inset => nil ?
>>>
>>> Karl
>>>
>>> On Tue, May 12, 2015 at 10:14 AM, marcel.taeumel <Marcel.Taeumel at hpi.de>
>>> wrote:
>>>
>>>> No, even Morph >> #borderStyle assumes that there is an instance of
>>>> BorderStyle returned. So Morphic-DS.916.mcz is fine.
>>>>
>>>> If you take a look at the strack trace and the implementation of
>>>> BorderedMorph >> #borderStyle, you can see that #width:color: is only
>>>> sent
>>>> to the class BorderStyle, which strangely happens to be a ByteSymbol in
>>>> recent VMs.
>>>>
>>>> So it's a VM bug.
>>>>
>>>> Best,
>>>> Marcel
>>>>
>>>>
>>>>
>>>> --
>>>> View this message in context:
>>>> http://forum.world.st/Re-Build-failed-in-Jenkins-SqueakTrunk-1472-tp4825918p4825933.html
>>>> Sent from the Squeak - Dev mailing list archive at Nabble.com.
>>>>
>>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20150512/b143d291/attachment.htm