May 2015 - Pharo-dev - lists.pharo.org

[pharo-project/pharo-core] 1694da: 50054

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 1694da0d0359039e191b1d685fbaee530c230c77 https://github.com/pharo-project/pharo-core/commit/1694da0d0359039e191b1d68… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-05-20 (Wed, 20 May 2015) Changed paths: M FileSystem-Smalltalk.package/extension/ClassOrganization/instance/fileEntriesDo_.st A Kernel.package/ClassOrganization.class/instance/accessing/allCategories.st A Kernel.package/ClassOrganization.class/instance/accessing/allProtocols.st A Kernel.package/ClassOrganization.class/instance/accessing/allProtocolsNames.st M Kernel.package/ClassOrganization.class/instance/accessing/categories.st R Kernel.package/ClassOrganization.class/instance/accessing/realCategories.st M KernelTests.package/ClassOrganizationTest.class/instance/tests/testRemoveCategory.st M Monticello-Tests.package/MCSnapshotBrowserTest.class/instance/private/protocolsForClass_.st M Nautilus.package/CategoryWidget.class/instance/private/loadPackagesCategoriesInSystemEnvironmentFor_.st M RPackage-Core.package/RPackage.class/instance/accessing/removeClass_.st M RPackage-Core.package/RPackage.class/instance/private/basicImportClass_.st M RPackage-Core.package/RPackage.class/instance/system compatibility/coreCategoriesForClass_.st M RPackage-Core.package/RPackage.class/instance/system compatibility/extensionCategoriesForClass_.st M Refactoring-Environment.package/RBBrowserEnvironment.class/instance/accessing/protocolsFor_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50053.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50054.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50053.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50054.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Tool-Base.package/AbstractTool.class/class/private/protocolSuggestionsFor_.st M Tool-Base.package/AbstractTool.class/instance/method/categorizeMethods_of_from_.st M Traits.package/TClassDescription.class/instance/filein%2Fout/fileOutChangedMessages_on_moveSource_toFile_.st M Traits.package/TClassDescription.class/instance/filein%2Fout/fileOutOn_moveSource_toFile_.st Log Message: ----------- 50054 15425 Make AST based syntax coloring the default https://pharo.fogbugz.com/f/cases/15425 15563 ClassOrganization>>#categories should not answer "all" category https://pharo.fogbugz.com/f/cases/15563 http://files.pharo.org/image/50/50054.zip

May 20, 2015

FileSystem not deleting symlinks

by Damien Cassou

The FileSystem library uses DeleteVisitor and PostorderGuide (depth-first) to delete a directory and all its children. All its children? No. Symlinks are protected from deletion: DeleteVisitor>>visit: aReference PostorderGuide show: aReference to: self selecting: [ :entry | entry isSymlink not ] I didn't find any unit-test or documentation about why symlinks are excluded from deletion. I don't see any reason. I'm quite sure the author had a reason to do that but I don't know which one. Anybody has an idea? -- Damien Cassou http://damiencassou.seasidehosting.st "Success is the ability to go from one failure to another without losing enthusiasm." --Winston Churchill

May 20, 2015

Re: [Pharo-dev] New Collections for Pharo

by Sergio Fedi

Nice work! I frequently needed the MultiValueDictionary and had it implemented several times in the past. â

May 20, 2015

Re: [Pharo-dev] [Pharo 5] For testing: enabled AST based syntax higlighting

by Marcus Denker

> On 20 May 2015, at 09:35, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > Hi, > > For 50054 I have enabled AST based syntax highlighting for testing. > > The idea is to have it enabled to find out what is missing, if there are bugs. > > As soon as we find issues, we can turn it off again. > > If it works fine for 2 weeks, we can start to clean up Shout and move the Shout parser into > the ECompletion package, which then would be the last user of the special token parser. > (Which we should rewrite to just use the AST, too). > Globals that are not class vars are not yet highlighted correctly. https://pharo.fogbugz.com/f/cases/15573/AST-Syntax-highlight-Class-Vars-are… <https://pharo.fogbugz.com/f/cases/15573/AST-Syntax-highlight-Class-Vars-are…> Will be fixed... Marcus

May 20, 2015

Re: [Pharo-dev] Critics Browser should diff formatted original

by Sean P. DeNigris

Sean P. DeNigris wrote > How about we diff the original source /re-formatted/ against the > transformed source? Issue 15574: Critics: Ignore whitespace changes when diffing transformations https://pharo.fogbugz.com/default.asp?15574 ----- Cheers, Sean -- View this message in context: http://forum.world.st/Critics-Browser-should-diff-formatted-original-tp4827… Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

May 20, 2015

Re: [Pharo-dev] [pharo-project/pharo-core] 2e7d6b: 50053

by Sean P. DeNigris

GitHub wrote > 15523 Code Cruft Rule Only Matches One-Liners > https://pharo.fogbugz.com/f/cases/15523 FYI since there were situations where the lint rule should have matched but did not, "new" violations may appear which are actually not new but just being reported for the first time. ----- Cheers, Sean -- View this message in context: http://forum.world.st/pharo-project-pharo-core-2e7d6b-50053-tp4827551p48275… Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

May 20, 2015

Re: [Pharo-dev] Anyone care to review issue 15528, Add TTranscript and DailyNonInteractiveTranscript ?

by Sven Van Caekenberghe

Thx! > On 20 May 2015, at 11:44, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > DONE. >> On 20 May 2015, at 09:24, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: >> >> Anyone care to review issue 15528 ? >> >> Add TTranscript and DailyNonInteractiveTranscript >> >> https://pharo.fogbugz.com/f/cases/15528/Add-TTranscript-and-DailyNonInterac… >> >> This is pretty straightforward, I added a comment about the warning. >> >> Thx, >> >> Sven >> >> > >

May 20, 2015

Re: [Pharo-dev] Anyone care to review issue 15528, Add TTranscript and DailyNonInteractiveTranscript ?

by Marcus Denker

DONE. > On 20 May 2015, at 09:24, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > > Anyone care to review issue 15528 ? > > Add TTranscript and DailyNonInteractiveTranscript > > https://pharo.fogbugz.com/f/cases/15528/Add-TTranscript-and-DailyNonInterac… > > This is pretty straightforward, I added a comment about the warning. > > Thx, > > Sven > >

May 20, 2015

[Pharo 5] For testing: enabled AST based syntax higlighting

by Marcus Denker

Hi, For 50054 I have enabled AST based syntax highlighting for testing. The idea is to have it enabled to find out what is missing, if there are bugs. As soon as we find issues, we can turn it off again. If it works fine for 2 weeks, we can start to clean up Shout and move the Shout parser into the ECompletion package, which then would be the last user of the special token parser. (Which we should rewrite to just use the AST, too). Marcus

May 20, 2015

Anyone care to review issue 15528, Add TTranscript and DailyNonInteractiveTranscript ?

by Sven Van Caekenberghe

Anyone care to review issue 15528 ? Add TTranscript and DailyNonInteractiveTranscript https://pharo.fogbugz.com/f/cases/15528/Add-TTranscript-and-DailyNonInterac… This is pretty straightforward, I added a comment about the warning. Thx, Sven

May 20, 2015