[squeak-dev] swapMouseButtons broken in trunk

Tobias Pape Das.Linux at gmx.de
Fri Mar 4 15:20:26 UTC 2016


On 04.03.2016, at 15:59, karl ramberg <karlramberg at gmail.com> wrote:

> 
> 
> On Fri, Mar 4, 2016 at 3:25 PM, Bert Freudenberg <bert at freudenbergs.de> wrote:
> On 03.03.2016, at 23:28, Chris Cunningham <cunningham.cb at gmail.com> wrote:
>> 
>> On Thu, Mar 3, 2016 at 12:31 PM, karl ramberg <karlramberg at gmail.com> wrote:
>> Nice.
>> BTW: there is a lot of empty categories in the Preferences class.
>> We must remove those in a post script or something.
>> Preferences organization removeEmptyCategories
>> 
>> I'm not sure how to do it ?
>> 
>> Add it in a package preamble ?
>> Does we then need to remove the preamble again ?
>> 
>> If I remember right, the preamble (and postlude?) only get run once - when they are first changed.  After that, they are skipped.
>> This means you want to leave them in and not remove them - otherwise a person later on would update from Trunk and just get the latest version - which doesn't have the preamble - and the work wouldn't be done.
>> 
>> Or am I wrong, and this only happens with Class initialization?
> 
> It happens exactly the same as with class initialization:
> 
> If the definition (of the script or method) changed, it gets executed.
> 
> You cannot rely on it having executed, or it only executing once. It’s better to write it in a way that running it again does no harm.
> 
> We can force execution by issuing an update map, but it’s better to avoid that.
> 
> For the problem at hand (empty method categories) we also could just add a step to the release builder to ensure all categories are clean.
> 
> We could run this for all classes I guess. I don't see much value in keeping empty method categories around...
> 
> Anyone objects ?


"look at class side" …

Best
	-Tobias

> 
> Best,
> Karl
>   
> 
> - Bert -



More information about the Squeak-dev mailing list