On Oct 3, 2013, at 9:22 PM, Marcus Denker <[email protected]> wrote:
> > On Oct 3, 2013, at 9:19 PM, Stéphane Ducasse <[email protected]> > wrote: > >> marcus for the critics next time please keep them else we will not be able >> to decide if we used them or not. >> This is not three little classes that will change something. Especially when >> they are part of a large library. >> > > But we don't use them. Yes but let us the time to think. First we should run the critics on everything then fix the problem. We should reorganize many things. Removing three classes that are well identified is not nice because now if I want to know which rules were not good I will have to load I do not know which version. Stef > >> Stef >> >> >> >>> 30444 >>> ----- >>> >>> 11762 update #cleanUpForProduction >>> https://pharo.fogbugz.com/f/cases/11762 >>> >>> 11764 unload Refactoring-Spelling >>> https://pharo.fogbugz.com/f/cases/11764 >>> >>> 11763 clean up Refactoring-Critics-Unused >>> https://pharo.fogbugz.com/f/cases/11763 >>> >>> 11761 class StrikeFontFixer is undocumented and the only users not called >>> https://pharo.fogbugz.com/f/cases/11761 >>> >>> >>> Diff information: >>> http://smalltalkhub.com/mc/Pharo/Pharo30/main/Tools-MarcusDenker.1288.diff >>> http://smalltalkhub.com/mc/Pharo/Pharo30/main/Refactoring-Critics-MarcusDenker.75.diff >>> http://smalltalkhub.com/mc/Pharo/Pharo30/main/Graphics-Fonts-MarcusDenker.85.diff >>> >> >> >
