[FIX] UI Bitrot

Brian Rice water at tunes.org
Tue Apr 11 18:41:09 PDT 2006


On Apr 11, 2006, at 4:22 PM, Todd Fleming wrote:

> This patch fixes several UI bugs; it runs again.
>
> Note: Do *not* apply to patch that Timmy posted today; it is based  
> on an unfortunate misunderstanding caused by Bug 2.
>
> Bug 1
> =====
> src/ui/graphics.slate is in AutoLoad, which broke the instructions
> in src/ui/README.  "load: 'src/ui/graphics.slate'" caused the file  
> to load twice.  This is why #parents: misbehaved and the call to  
> #text: failed.
>
> Bug 2
> =====
> Fortunately I keep old sources around to compare with.   
> Unfortunately someone made wholesale cosmetic changes to
> plugins/sdl-windows/windows.slate which made it difficult to compare.
> One of those cosmetic changes introduced a bug.
>
> Someone changed:
>   Windows eventTable add:
>       (each key) ->
>       (Windows EventTableEntry
>           newPrototype: (events atSlotNamed: (each value at: 0))
>           constructors: (each value at: 1))].
>
> To:
>   Windows eventTable at: each key put:
>     (Windows EventTableEntry
>        newPrototype: (events atSlotNamed: each value first)
>        constructors: each value first)].
>
> This should be:
>   Windows eventTable at: each key put:
>     (Windows EventTableEntry
>        newPrototype: (events atSlotNamed: each value first)
>        constructors: each value second)].
>
> This caused constructors to be initialized with a (wrong) symbol
> instead of a list.

Erg. That would be my fault. Sorry about that. I've applied the patch  
and pushed it into the site repositories - thanks.

--
-Brian
http://tunes.org/~water/brice.vcf

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
Url : /archives/slate/attachments/20060411/68ed630f/PGP.pgp


More information about the Slate mailing list