Pharo-dev

By thread

pharo-dev@lists.pharo.org

November 2017

846 messages

[Pharo 7.0-dev] Build #323: 20549-CompiledMethod-isTestMethod-should-invert-the-query-clauses

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #323 was: SUCCESS. The Pull Request #523 was integrated: "20549-CompiledMethod-isTestMethod-should-invert-the-query-clauses" Pull request url: https://github.com/pharo-project/pharo/pull/523 Issue Url: https://pharo.fogbugz.com/f/cases/20549 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Nov. 24, 2017

Re: [Pharo-dev] [PHARO 7.0A] NEXT ENHANCEMENTS

by Tudor Girba

Well said! Doru > On Nov 23, 2017, at 8:53 PM, Torsten Bergmann <astares(a)gmx.de> wrote: > > What is not visible in the list are the many enhancements happening also on Calypso, Bloc, > Grease and many many others. > > Nonetheless the core image still needs love from more people. Remember even the smallest change > can make a difference and shape the future. Also "a contribution per day keeps bad systems away". > Even one contribution per week or month. > > So jump in - the water is just fine! > > Bye > T. (aka astares) > > Gesendet: Donnerstag, 23. November 2017 um 12:37 Uhr > Von: "phil(a)highoctane.be" <phil(a)highoctane.be> > An: "Pharo Development List" <pharo-dev(a)lists.pharo.org> > Betreff: Re: [Pharo-dev] [PHARO 7.0A] NEXT ENHANCEMENTS > Yes, very. > > Phil > > On Thu, Nov 23, 2017 at 12:26 PM, Alexandre Bergel <alexandre.bergel(a)me.com> wrote: > Impressive!! > > > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > On Nov 23, 2017, at 7:12 AM, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: > > eport period: 5 November 2017 to 23 November 2017 > > * 20739-Remove-dead-stream-primitives > https://pharo.fogbugz.com/f/cases/20739/Remove-dead-stream-primitives > > Issue URL: https://pharo.fogbugz.com/f/cases/20739 > PR URL: https://github.com/pharo-project/pharo/pull/524 > Diff URL: https://github.com/pharo-project/pharo/pull/524/files > > Thanks to clementbera > > * 20645-Windows-rebuildFieldAccessors-problem > Fixing the creation of annonymous classes for FFITypeArray. > It was affecting all the platforms. > > Issue: https://pharo.fogbugz.com/f/cases/20645/Windows-rebuildFieldAccessors-probl… > > Issue URL: https://pharo.fogbugz.com/f/cases/20645 > PR URL: https://github.com/pharo-project/pharo/pull/521 > Diff URL: https://github.com/pharo-project/pharo/pull/521/files > > Thanks to tesonep > > * 20735-ComposablePresenter-classowneron-is-missing-the-initialization-call > https://pharo.fogbugz.com/f/cases/20735/ComposablePresenter-class-owner-on-… > > Issue URL: https://pharo.fogbugz.com/f/cases/20735 > PR URL: https://github.com/pharo-project/pharo/pull/517 > Diff URL: https://github.com/pharo-project/pharo/pull/517/files > > Thanks to pavel-krivanek > > * Fix the sign of FloatNegativeZero > Like any other zero, the sign of Float negativeZero should be zeroâ¦ > Introduce signBit to differentiate Float negativeZero from Float zero. > > This is issue > https://pharo.fogbugz.com/f/cases/19629/0-0-sign-answers-1-should-be-zero > > Note that this PR does not implement isSignMinus which is not > strictly necessary. > If itâs for ISO 10967 compliance, then there will be other > functions missing. > Maybe open a new issue for this one? > > Issue URL: https://pharo.fogbugz.com/f/cases/Fix the sign of > FloatNegativeZero > PR URL: https://github.com/pharo-project/pharo/pull/515 > Diff URL: https://github.com/pharo-project/pharo/pull/515/files > > Thanks to nicolas-cellier-aka-nice > > * 20717-Duplicities-of-themes-in-the-list-2 > https://pharo.fogbugz.com/f/cases/20717/Duplicities-of-themes-in-the-list > > alternative version by Torsten > > Issue URL: https://pharo.fogbugz.com/f/cases/20717 > PR URL: https://github.com/pharo-project/pharo/pull/499 > Diff URL: https://github.com/pharo-project/pharo/pull/499/files > > Thanks to pavel-krivanek > > * 20730 Unused temps in FileReferenceTest, FLHeaderSerializationTest, > FLHookedSubstitutionTest, FT2GlyphRenderer, ... > Fix unused temps in > > FileReferenceTest>>#testRename > FileReferenceTest>>#testDeleteAllChildren > FileReferenceTest>>#testHasChildren > FileReferenceTest>>#testHasDirectories > FileReferenceTest>>#testHasFiles > FLHeaderSerializationTest>>#testAdditionalObjects > FLHookedSubstitutionTest>>#testAvoidRecursion > FT2GlyphRenderer>>#loadSurfaceTransform > FT2GlyphRenderer>>#loadUnicode: > FileDialogWindowTest>>#testIssue6406 > FileReferenceTest>>#testDeleteAll > > Issue URL: https://pharo.fogbugz.com/f/cases/20730 Unused temps > in FileReferenceTest, FLHeaderSerializationTest, > FLHookedSubstitutionTest, FT2GlyphRenderer, ... > PR URL: https://github.com/pharo-project/pharo/pull/510 > Diff URL: https://github.com/pharo-project/pharo/pull/510/files > > Thanks to astares > > * 20731 Unused temps in HDCoverageReport and HDLintReport > fix unused temps > > HDCoverageReport>>#generatePackage:method:on: > HDLintReport>>#generateClass:source:on: > > > https://pharo.fogbugz.com/f/cases/20731/Unused-temps-in-HDCoverageReport-an… > > Issue URL: https://pharo.fogbugz.com/f/cases/20731 Unused temps > in HDCoverageReport and HDLintReport > PR URL: https://github.com/pharo-project/pharo/pull/511 > Diff URL: https://github.com/pharo-project/pharo/pull/511/files > > Thanks to astares > > * 20732 Unused temps in HistoryNodeTest > fix unused temps > > HistoryNodeTest>>#testTwoConsecutiveCloseGroup > HistoryNodeTest>>#testTwoGroups > > https://pharo.fogbugz.com/f/cases/20732/Unused-temps-in-HistoryNodeTest > > Issue URL: https://pharo.fogbugz.com/f/cases/20732 Unused temps > in HistoryNodeTest > PR URL: https://github.com/pharo-project/pharo/pull/512 > Diff URL: https://github.com/pharo-project/pharo/pull/512/files > > Thanks to astares > > * 20733-update-bootstrap-emulation > https://pharo.fogbugz.com/f/cases/20733/update-bootstrap-emulation > > Issue URL: https://pharo.fogbugz.com/f/cases/20733 > PR URL: https://github.com/pharo-project/pharo/pull/516 > Diff URL: https://github.com/pharo-project/pharo/pull/516/files > > Thanks to pavel-krivanek > > * 20384-Converted-rules-to-Renraku-architecture-2 > Let's try this one, it should work > > Issue URL: https://pharo.fogbugz.com/f/cases/20384 > PR URL: https://github.com/pharo-project/pharo/pull/445 > Diff URL: https://github.com/pharo-project/pharo/pull/445/files > > Thanks to Uko > > * 20727-WeakAnnouncerTesttestWeakDoubleAnnouncer-should-be-long > https://pharo.fogbugz.com/f/cases/20727/WeakAnnouncerTest-testWeakDoubleAnn… > > increse timeouts and decrease amount of explicit GC calls > > Issue URL: https://pharo.fogbugz.com/f/cases/20727 > PR URL: https://github.com/pharo-project/pharo/pull/509 > Diff URL: https://github.com/pharo-project/pharo/pull/509/files > > Thanks to pavel-krivanek > > * Avoid costly & useless LargeInteger arithmetics when > raisedToFraction will be inexact > > This example was raised in > https://pharo.fogbugz.com/f/cases/20432/vm-crash-when-using-rairedTo-with-f… > (2009/2000) raisedTo: (3958333/100000) > > The fix is importing the changes from > http://source.squeak.org/trunk/Kernel-nice.1111.diff > > Issue URL: https://pharo.fogbugz.com/f/cases/Avoid costly & > useless LargeInteger arithmetics when raisedToFraction will be inexact > PR URL: https://github.com/pharo-project/pharo/pull/506 > Diff URL: https://github.com/pharo-project/pharo/pull/506/files > > Thanks to nicolas-cellier-aka-nice > > * 20723 Unused temps in AthensDemoMorph, Alien, AthensBalloonEngine, > AthensBezier3Scene, AthensCairoPathBuilder > Fix unused temps in > > AthensDemoMorph>>#figure9: > Alien class>>#newCString: > AthensBalloonEngine>>#drawImage:transform: > AthensBezier3Scene>>#renderOn: > AthensCairoPathBuilder>>#calcCenter:end:angle: > AthensDemoMorph>>#figure10: > AthensDemoMorph>>#figure2: > AthensDemoMorph>>#figure3: > AthensDemoMorph>>#figure6: > > Issue URL: https://pharo.fogbugz.com/f/cases/20723 Unused temps > in AthensDemoMorph, Alien, AthensBalloonEngine, AthensBezier3Scene, > AthensCairoPathBuilder > PR URL: https://github.com/pharo-project/pharo/pull/503 > Diff URL: https://github.com/pharo-project/pharo/pull/503/files > > Thanks to astares > > * 20724 Unused temps in EditorFindReplaceDialogWindow>>#update: > fix unused temp > > Issue URL: https://pharo.fogbugz.com/f/cases/20724 Unused temps > in EditorFindReplaceDialogWindow>>#update: > PR URL: https://github.com/pharo-project/pharo/pull/504 > Diff URL: https://github.com/pharo-project/pharo/pull/504/files > > Thanks to astares > > * 20725 Unused temps in DosTimestampTest, Decompiler, DiskStore, > DosTimestampTest, DateTest > Fix unused temps in > > DosTimestampTest>>#testFromDateAndTime > Decompiler>>#statementsTo: > DiskStore>>#rename:to: > DosTimestampTest>>#testAsDateAndTime > DateTest>>#testPrintFormat > > Issue URL: https://pharo.fogbugz.com/f/cases/20725 Unused temps > in DosTimestampTest, Decompiler, DiskStore, DosTimestampTest, DateTest > PR URL: https://github.com/pharo-project/pharo/pull/505 > Diff URL: https://github.com/pharo-project/pharo/pull/505/files > > Thanks to astares > > * 20722 Unused temps in KMShortcutTest, KMDispatcher, KMKeymapTest > Fix unused temps in > > KMShortcutTest>>#testComplexChainMatches > KMDispatcher>>#dispatch: > KMKeymapTest>>#testExecute > > https://pharo.fogbugz.com/f/cases/20722/Unused-temps-in-KMShortcutTest-KMDi… > > Issue URL: https://pharo.fogbugz.com/f/cases/20722 Unused temps > in KMShortcutTest, KMDispatcher, KMKeymapTest > PR URL: https://github.com/pharo-project/pharo/pull/502 > Diff URL: https://github.com/pharo-project/pharo/pull/502/files > > Thanks to astares > > * 20720 Unused temps in Object, OCASTCheckerTest and OCASTClosureAnalyzerTest > Fix unused temp vars > > Object>>#split: > OCASTCheckerTest>>#testDoubleRemoteAnidatedBlocks > OCASTCheckerTest>>#testExamplePrimitiveErrorCode > OCASTCheckerTest>>#testExampleSelf > OCASTCheckerTest>>#testExampleSuper > OCASTCheckerTest>>#testExampleThisContext > OCASTCheckerTest>>#testInstanceVar > OCASTCheckerTest>>#testNoRemoteBlockArgument > OCASTCheckerTest>>#testNoRemoteBlockReturn > OCASTCheckerTest>>#testNoRemoteBlockTemp > OCASTCheckerTest>>#testNoRemoteMethodTemp > OCASTCheckerTest>>#testOptimizedBlocksAndSameNameTemps > OCASTCheckerTest>>#testSingleRemoteDifferentBlocksSameArgumentName > OCASTCheckerTest>>#testSingleRemoteMethodArgument > OCASTClosureAnalyzerTest>>#testDoubleRemoteAnidatedBlocks > OCASTClosureAnalyzerTest>>#testExampleBlockArgument > OCASTClosureAnalyzerTest>>#testExampleSimpleBlockLocalWhile > OCASTClosureAnalyzerTest>>#testExampleSimpleBlockNested > OCASTClosureAnalyzerTest>>#testExampleWhileModificationBefore > OCASTClosureAnalyzerTest>>#testNestedBlocksRemoteInBlockCase2 > OCASTClosureAnalyzerTest>>#testNestedBlocksRemoteInBlockCase3 > OCASTClosureAnalyzerTest>>#testNoRemoteBlockArgument > OCASTClosureAnalyzerTest>>#testNoRemoteReadInBlock > OCASTClosureAnalyzerTest>>#testNoRemoteReadNestedBlocks > OCASTClosureAnalyzerTest>>#testOptimizedBlockReadInBlock > OCASTClosureAnalyzerTest>>#testOptimizedBlockWriteInBlock > OCASTClosureAnalyzerTest>>#testOptimizedBlockWriteInNestedBlock > OCASTClosureAnalyzerTest>>#testOptimizedBlockWriteInNestedBlockCase2 > OCASTClosureAnalyzerTest>>#testOptimizedBlockWriteInNestedBlockCase3 > OCASTClosureAnalyzerTest>>#testOptimizedBlockWriteInNestedBlockCase4 > OCASTClosureAnalyzerTest>>#testOptimizedBlockWrittenAfterClosedOverCase1 > OCASTClosureAnalyzerTest>>#testOptimizedBlockWrittenAfterClosedOverCase2 > OCASTClosureAnalyzerTest>>#testSingleRemoteDifferentBlocksSameArgumentName > OCASTClosureAnalyzerTest>>#testSingleRemoteMethodArgument > OCASTClosureAnalyzerTest>>#testSingleRemoteReadNestedBlocks > OCASTClosureAnalyzerTest>>#testWrittenAfterClosedOver > > Issue URL: https://pharo.fogbugz.com/f/cases/20720 Unused temps > in Object, OCASTCheckerTest and OCASTClosureAnalyzerTest > PR URL: https://github.com/pharo-project/pharo/pull/500 > Diff URL: https://github.com/pharo-project/pharo/pull/500/files > > Thanks to astares > > * 20721 Unused temps in ProtoObject>>#pointersToExcept: > Fix unused temps in ProtoObject>>#pointersToExcept: > > Issue URL: https://pharo.fogbugz.com/f/cases/20721 Unused temps > in ProtoObject>>#pointersToExcept: > PR URL: https://github.com/pharo-project/pharo/pull/501 > Diff URL: https://github.com/pharo-project/pharo/pull/501/files > > Thanks to astares > > * 20715 Unused temps in RBRefactoringChangeTests, > RPackageClassesSynchronisationTest, RPackageIncrementalTest, ... > Remove unused temps in > > RBRefactoringChangeTests>>#testPerformAddRemoveClassMethod > RPackageClassesSynchronisationTest>>#testReorganizeClassByAddingExtensionProtocol > RPackageClassesSynchronisationTest>>#testReorganizeClassByAddingNewProtocolDoesNothing > RPackageExtensionMethodsSynchronisationTest>>#testAddMethodInExtensionCategoryNotExistingCreateANewPackage > RPackageExtensionMethodsSynchronisationTest>>#testAddMethodInExtensionCategoryNotExistingCreateANewPackageAndInstallsMethodInIt > RPackageExtensionMethodsSynchronisationTest>>#testModifyMethodByChangingCode > RPackageIncrementalTest>>#testAddClassAfterMethods > RPackageIncrementalTest>>#testAddClassAfterMethodsAtMetaclassToo > RPackageIncrementalTest>>#testMethodAddition > RPackageIncrementalTest>>#testPrivateClassRegisterUnregister > RPackageIncrementalTest>>#testRemoveClassAfterMethods > RPackageIncrementalTest>>#testRemoveClassRemovesExtensions > RPackageIncrementalTest>>#testRemoveExtensionMethodRemovesExtensionsFromRPackage > RPackageMethodsSynchronisationTest>>#testModifyMethodByMovingFromClassicCategoryToClassicCategoryDoesNothing > RPackageMonticelloSynchronisationTest>>#testUnloadMCPackageRemovesRPackage > RPackageMonticelloSynchronisationTest>>#testUnregisterMCPackageKeepsRPackage > RPackageOrganizer>>#systemClassRemovedActionFrom: > RBRefactoringChangeTests>>#testPerformAddRemoveMethod > RBRefactoringChangeTests>>#testPerformAddRemoveMethodInteractively > RPackageClassesSynchronisationTest>>#testRecategorizeClassRegisterTheClassMethodsInTheNewPackage > RPackageClassesSynchronisationTest>>#testRecategorizeClassWithMetaClassMethodsRegisterAllClassMethodsInTheNewPackage > RPackageClassesSynchronisationTest>>#testRenameClassUpdateClassDefinedInTheParentPackage > RPackageClassesSynchronisationTest>>#testRenameClassUpdateClassDefinedSelectorsInTheParentPackage > RPackageClassesSynchronisationTest>>#testRenameClassUpdateClassExtensionSelectorsInTheExtendingPackages > RPackageClassesSynchronisationTest>>#testRenameClassUpdateMetaclassDefinedSelectorsInTheParentPackage > RPackageClassesSynchronisationTest>>#testRenameClassUpdateMetaclassExtensionSelectorsInTheExtendingPackages > RPackageClassesSynchronisationTest>>#testRenameClassUpdateOrganizerClassExtendingPackagesMapping > RPackageClassesSynchronisationTest>>#testRenameClassUpdateOrganizerClassPackageMapping > > Issue URL: https://pharo.fogbugz.com/f/cases/20715 Unused temps > in RBRefactoringChangeTests, RPackageClassesSynchronisationTest, > RPackageIncrementalTest, ... > PR URL: https://github.com/pharo-project/pharo/pull/495 > Diff URL: https://github.com/pharo-project/pharo/pull/495/files > > Thanks to astares > > * 20716 Unused temps in MetacelloConfigurationResource, > MetacelloCoreSymbolicVersionTest, MetacelloPackagesSpecTestCase, > MetacelloMCProj > Remove unused temps from > > MetacelloConfigurationResource>>#project > MetacelloCoreSymbolicVersionTest>>#testBasicBleedingEdgeVersion > MetacelloCoreSymbolicVersionTest>>#testBasicStableVersion > MetacelloCoreSymbolicVersionTest>>#testBleedingEdgeMethod > MetacelloCoreSymbolicVersionTest>>#testStableVersionMethod > MetacelloPackagesSpecTestCase>>#testRemoveGroupB > MetacelloPackagesSpecTestCase>>#testRemoveGroupC > MetacelloMCProjectSpecTestCase>>#testMCProjectMergeSpec > > Issue URL: https://pharo.fogbugz.com/f/cases/20716 Unused temps > in MetacelloConfigurationResource, MetacelloCoreSymbolicVersionTest, > MetacelloPackagesSpecTestCase, MetacelloMCProj > PR URL: https://github.com/pharo-project/pharo/pull/497 > Diff URL: https://github.com/pharo-project/pharo/pull/497/files > > Thanks to astares > > * 20714-atatput--atatifAbsentPut-should-not-use-Dictionary > https://pharo.fogbugz.com/f/cases/20714/at-at-put-at-at-ifAbsentPut-should-… > > use "self species" instead of hardcoded Dictionary > > Issue URL: https://pharo.fogbugz.com/f/cases/20714 > PR URL: https://github.com/pharo-project/pharo/pull/496 > Diff URL: https://github.com/pharo-project/pharo/pull/496/files > > Thanks to pavel-krivanek > > * 20709-do-not-create-pharo-core-symlink-during-the-image-building > https://pharo.fogbugz.com/f/cases/20709/do-not-create-pharo-core-symlink-du… > > Issue URL: https://pharo.fogbugz.com/f/cases/20709 > PR URL: https://github.com/pharo-project/pharo/pull/492 > Diff URL: https://github.com/pharo-project/pharo/pull/492/files > > Thanks to pavel-krivanek > > * 20603-Integration-of-new-bytecode-set-Sista-V1-by-default-green-tests > > Issue URL: https://pharo.fogbugz.com/f/cases/20603 > PR URL: https://github.com/pharo-project/pharo/pull/484 > Diff URL: https://github.com/pharo-project/pharo/pull/484/files > > Thanks to clementbera > > * 20707 Unused temps in WindowsStore, WaitfreeQueue, VersionnerProjectPanel, > Fix unused temp vars in > - WindowsStore>>#checkName:fixErrors: > - WaitfreeQueue>>#size > - VersionnerProjectPanel>>#config: > - VersionnerProjectPackagesPanel>>#removeSelectedPackage > - VSSetDescriptionCommand>>#executeWithMessage: > > > https://pharo.fogbugz.com/f/cases/20707/Unused-temps-in-WindowsStore-Waitfr… > > Issue URL: https://pharo.fogbugz.com/f/cases/20707 Unused temps > in WindowsStore, WaitfreeQueue, VersionnerProjectPanel, > PR URL: https://github.com/pharo-project/pharo/pull/490 > Diff URL: https://github.com/pharo-project/pharo/pull/490/files > > Thanks to astares > > * 20711 Unused temps in RubTextEditor, RubParagraph, > RubMethodEditingExample, ReflectiveMethod, RxMatcher > Remove unused temps in > > ReflectiveMethod>>#recompileAST > RubMethodEditingExample>>#open > RubParagraph>>#selectionRectsFrom:to: > RubSelectorChooserMorph>>#drawCommonPrefixAreasOn: > RubTextEditor>>#addDeleteSelectionUndoRecord > RubTextEditor>>#correctFrom:to:with: > RubTextEditor>>#exchangeWith: > RxMatcher>>#currentState > > Issue URL: https://pharo.fogbugz.com/f/cases/20711 Unused temps > in RubTextEditor, RubParagraph, RubMethodEditingExample, > ReflectiveMethod, RxMatcher > PR URL: https://github.com/pharo-project/pharo/pull/494 > Diff URL: https://github.com/pharo-project/pharo/pull/494/files > > Thanks to astares > > * 20704-remove-garbageCollect-from-PharoClassInstallermigrateClassestousing > https://pharo.fogbugz.com/f/cases/20704/remove-garbageCollect-from-PharoCla… > > remove explicit garbage collection > > Issue URL: https://pharo.fogbugz.com/f/cases/20704 > PR URL: https://github.com/pharo-project/pharo/pull/489 > Diff URL: https://github.com/pharo-project/pharo/pull/489/files > > Thanks to pavel-krivanek > > * 20708 Unused temps in TSequencedConcatenationTest and TextEditor > Remove unused temps in > TextMorph>>#handleKeystroke: > TSequencedConcatenationTest>>#testStreamContents > TSequencedConcatenationTest>>#testStreamContentsProtocol > TextConverter>>#next:putAll:startingAt:toStream: > TextConverter>>#nextPutAll:toStream: > TextEditor>>#addDeleteSelectionUndoRecord > TextEditor>>#addTypeInUndoRecord > TextEditor>>#correctFrom:to:with: > TextEditor>>#dispatchCommandOn:return: > TextEditor>>#dispatchOn: > TextEditor>>#mouseUp: > TextEditor>>#replace:with:and: > TextEditor>>#unapplyAttribute: > > Issue URL: https://pharo.fogbugz.com/f/cases/20708 Unused temps > in TSequencedConcatenationTest and TextEditor > PR URL: https://github.com/pharo-project/pharo/pull/491 > Diff URL: https://github.com/pharo-project/pharo/pull/491/files > > Thanks to astares > > * 20710 Unused temps in StringMorph, SimpleGridExample, > SmalltalkEditor, SHMethodEditingMode, SelectorChooserMorph > Remove unused temps from > > StringMorph>>#drawOnAthensCanvas: > SimpleGridExample>>#treeMorph > SmalltalkEditor>>#internalCallToBrowseIt > SHMethodEditingMode>>#open > SelectorChooserMorph>>#drawCommonPrefixAreasOn: > > Issue URL: https://pharo.fogbugz.com/f/cases/20710 Unused temps > in StringMorph, SimpleGridExample, SmalltalkEditor, > SHMethodEditingMode, SelectorChooserMorph > PR URL: https://github.com/pharo-project/pharo/pull/493 > Diff URL: https://github.com/pharo-project/pharo/pull/493/files > > Thanks to astares > > * 20697-Give-some-love-to-SocketAddress > SocketAddress is hardly used in the system. > It is possible to use it more and add some functionality to it. > Comments and test could be improved as well. > > https://pharo.fogbugz.com/f/cases/20697/Give-some-love-to-SocketAddress > > Issue URL: https://pharo.fogbugz.com/f/cases/20697 > PR URL: https://github.com/pharo-project/pharo/pull/486 > Diff URL: https://github.com/pharo-project/pharo/pull/486/files > > Thanks to svenvc > > * 20702-Add-comments-to-hashMultiply > https://pharo.fogbugz.com/f/cases/20702/Add-comments-to-hashMultiply > > use hashMultiply comments provided by Nicolas Celler with some > modifications > > Issue URL: https://pharo.fogbugz.com/f/cases/20702 > PR URL: https://github.com/pharo-project/pharo/pull/487 > Diff URL: https://github.com/pharo-project/pharo/pull/487/files > > Thanks to pavel-krivanek > > * 20703-typo-in-WelcomeHelp > https://pharo.fogbugz.com/f/cases/20703/typo-in-WelcomeHelp > > fix typo > > Issue URL: https://pharo.fogbugz.com/f/cases/20703 > PR URL: https://github.com/pharo-project/pharo/pull/488 > Diff URL: https://github.com/pharo-project/pharo/pull/488/files > > Thanks to pavel-krivanek > > * 20647-SortFunction-should-be-composable > Make SortFunction composable > > https://pharo.fogbugz.com/f/cases/20647/SortFunction-should-be-composable > > Note: impossible to use iceberg to do the commit > incomprehensible interface / unexpected failures / unclear > error messages > > Issue URL: https://pharo.fogbugz.com/f/cases/20647 > PR URL: https://github.com/pharo-project/pharo/pull/440 > Diff URL: https://github.com/pharo-project/pharo/pull/440/files > > Thanks to nicolas-cellier-aka-nice > > * 19942-Glamour-Morphic-Renderer-Spotter-is-only-dead-code > Glamour-Morphic-Renderer-Spotter is only dead code? > > https://pharo.fogbugz.com/f/cases/19942/Glamour-Morphic-Renderer-Spotter-is… > > Issue URL: https://pharo.fogbugz.com/f/cases/19942 > PR URL: https://github.com/pharo-project/pharo/pull/480 > Diff URL: https://github.com/pharo-project/pharo/pull/480/files > > Thanks to MarcusDenker > > * 20696-enable-bootstrapping-from-outside-the-repository > https://pharo.fogbugz.com/f/cases/20696/enable-bootstrapping-from-outside-t… > > Issue URL: https://pharo.fogbugz.com/f/cases/20696 > PR URL: https://github.com/pharo-project/pharo/pull/485 > Diff URL: https://github.com/pharo-project/pharo/pull/485/files > > Thanks to pavel-krivanek > > * 20689-Make-Pharo-bootstrap-independent-on-source-repository > https://pharo.fogbugz.com/f/cases/20689/Make-Pharo-bootstrap-independent-on… > > Issue URL: https://pharo.fogbugz.com/f/cases/20689 > PR URL: https://github.com/pharo-project/pharo/pull/479 > Diff URL: https://github.com/pharo-project/pharo/pull/479/files > > Thanks to pavel-krivanek > > * 20693-Incorrect-basename-of-empty-relative-path > 20693 Incorrect basename of empty relative path > > - Modify Path and subclasses to return the current directory > for an empty relative path. > - Add PathTest>>testBasenameNoParent to confirm correct behaviour. > > Issue URL: https://pharo.fogbugz.com/f/cases/20693 > PR URL: https://github.com/pharo-project/pharo/pull/481 > Diff URL: https://github.com/pharo-project/pharo/pull/481/files > > Thanks to akgrant43 > > * 20678-selecting-of-a-method-version-causes-DNU > Adding this message as a way of having polymorphism with the > different versions of Message Browser > > Issue https://pharo.fogbugz.com/f/cases/20678/selecting-of-a-method-version-cause… > > Issue URL: https://pharo.fogbugz.com/f/cases/20678 > PR URL: https://github.com/pharo-project/pharo/pull/482 > Diff URL: https://github.com/pharo-project/pharo/pull/482/files > > Thanks to tesonep > > * 20686-disabling-of-FreeType-fonts-should-set-bitmap-Source-Pro-fonts > https://pharo.fogbugz.com/f/cases/20686/disabling-of-FreeType-fonts-should-… > > set bitmap source pro fonts after disabling of FreeType > > Issue URL: https://pharo.fogbugz.com/f/cases/20686 > PR URL: https://github.com/pharo-project/pharo/pull/476 > Diff URL: https://github.com/pharo-project/pharo/pull/476/files > > Thanks to pavel-krivanek > > * 19291-suspendedEventHandler-property-seems-unused > suspendedEventHandler property seems unused > > https://pharo.fogbugz.com/f/cases/19291/suspendedEventHandler-property-seem… > > Issue URL: https://pharo.fogbugz.com/f/cases/19291 > PR URL: https://github.com/pharo-project/pharo/pull/466 > Diff URL: https://github.com/pharo-project/pharo/pull/466/files > > Thanks to MarcusDenker > > * 20685-Do-not-use-hardcoded-URLs-in-system-baselines > https://pharo.fogbugz.com/f/cases/20685/Do-not-use-hardcoded-URLs-in-system… > > use repository URL of the baseline package for loading of > other baselines from the same repository > > Issue URL: https://pharo.fogbugz.com/f/cases/20685 > PR URL: https://github.com/pharo-project/pharo/pull/475 > Diff URL: https://github.com/pharo-project/pharo/pull/475/files > > Thanks to pavel-krivanek > > * 20166-wrong-cliprect-on-transformed-athens-canvas > https://pharo.fogbugz.com/f/cases/20166/wrong-cliprect-on-transformed-athen… > > fix proposed by Nicolai Hess > > Issue URL: https://pharo.fogbugz.com/f/cases/20166 > PR URL: https://github.com/pharo-project/pharo/pull/477 > Diff URL: https://github.com/pharo-project/pharo/pull/477/files > > Thanks to pavel-krivanek > > * 20264-Date-gtInspectorDetailsIn-is-incorrectly-using-a-Block-for-the-title-so-selection-is-not-remembered > Fix GT Inspector usage: title's should not be blocks but pure > Strings for them to be remembered as selected in the UI > > Improve several GT Inspector views in small details (especially Socket) > > Touched Date, Time, DateAndTime, Duration, Socket, ZnMessage, > ZnHeaders and ZnMimePart views. > > https://pharo.fogbugz.com/f/cases/20264/Date-gtInspectorDetailsIn-is-incorr… > > Issue URL: https://pharo.fogbugz.com/f/cases/20264 > PR URL: https://github.com/pharo-project/pharo/pull/471 > Diff URL: https://github.com/pharo-project/pharo/pull/471/files > > Thanks to svenvc > > * 20679-Reloading-of-OSWindow-SDL2-causes-Tonel-parsing-error > https://pharo.fogbugz.com/f/cases/20679/Reloading-of-OSWindow-SDL2-causes-T… > > fix broken UTF8 character > > Issue URL: https://pharo.fogbugz.com/f/cases/20679 > PR URL: https://github.com/pharo-project/pharo/pull/473 > Diff URL: https://github.com/pharo-project/pharo/pull/473/files > > Thanks to pavel-krivanek > > * 20676 Categorize uncategorized methods in classes starting with K > Categorize uncategorized methods > > No change in behavior, only categorization > > > https://pharo.fogbugz.com/f/cases/20676/Categorize-uncategorized-methods-in… > > Issue URL: https://pharo.fogbugz.com/f/cases/20676 Categorize > uncategorized methods in classes starting with K > PR URL: https://github.com/pharo-project/pharo/pull/470 > Diff URL: https://github.com/pharo-project/pharo/pull/470/files > > Thanks to astares > > * 20674 Categorize uncategorized methods in classes starting with J > Categorize uncategorized methods in > - JobProgressBarMorph > - JoinSection > - JoinSide > > Issue URL: https://pharo.fogbugz.com/f/cases/20674 Categorize > uncategorized methods in classes starting with J > PR URL: https://github.com/pharo-project/pharo/pull/468 > Diff URL: https://github.com/pharo-project/pharo/pull/468/files > > Thanks to astares > > * 20677-Improve-system-dependencies-tests > https://pharo.fogbugz.com/f/cases/20677/Improve-system-dependencies-testss > > clean system dependencies testing > > Issue URL: https://pharo.fogbugz.com/f/cases/20677 > PR URL: https://github.com/pharo-project/pharo/pull/472 > Diff URL: https://github.com/pharo-project/pharo/pull/472/files > > Thanks to pavel-krivanek > > * 20673 Categorize uncategorized methods in classes starting with H > Categorize uncategorized methods in > - HEFraction > - HEScaledDecimal > > Only categorization, no change in behavior > > Issue URL: https://pharo.fogbugz.com/f/cases/20673 Categorize > uncategorized methods in classes starting with H > PR URL: https://github.com/pharo-project/pharo/pull/467 > Diff URL: https://github.com/pharo-project/pharo/pull/467/files > > Thanks to astares > > * 20675 Categorize uncategorized methods in classes starting with D > Categorize uncategorized method in DatePresenter > > Issue URL: https://pharo.fogbugz.com/f/cases/20675 Categorize > uncategorized methods in classes starting with D > PR URL: https://github.com/pharo-project/pharo/pull/469 > Diff URL: https://github.com/pharo-project/pharo/pull/469/files > > Thanks to astares > > * 20671-Extend-system-dependencies-tests-on-whole-image > https://pharo.fogbugz.com/f/cases/20671/Extend-system-dependencies-tests-on… > > dependency testing for whole image > > Issue URL: https://pharo.fogbugz.com/f/cases/20671 > PR URL: https://github.com/pharo-project/pharo/pull/464 > Diff URL: https://github.com/pharo-project/pharo/pull/464/files > > Thanks to pavel-krivanek > > * 20670-unused-var-in-generateDefaultFileOutResultsName > unused var in #generateDefaultFileOutResultsName > > https://pharo.fogbugz.com/f/cases/20670/unused-var-in-generateDefaultFileOu… > > Issue URL: https://pharo.fogbugz.com/f/cases/20670 > PR URL: https://github.com/pharo-project/pharo/pull/465 > Diff URL: https://github.com/pharo-project/pharo/pull/465/files > > Thanks to MarcusDenker > > * 20668-add-breakpoints-menu-to-Nautilus-segment-icons > https://pharo.fogbugz.com/f/cases/20668 > > fix breakpoints placement for wrapping text > > Issue URL: https://pharo.fogbugz.com/f/cases/20668 > PR URL: https://github.com/pharo-project/pharo/pull/463 > Diff URL: https://github.com/pharo-project/pharo/pull/463/files > > Thanks to pavel-krivanek > > * 20055-immediateByteSubclass-is-deadcoe > immediateByteSubclass: is deadcoe > > https://pharo.fogbugz.com/f/cases/20055/immediateByteSubclass-is-deadcoe > > Issue URL: https://pharo.fogbugz.com/f/cases/20055 > PR URL: https://github.com/pharo-project/pharo/pull/462 > Diff URL: https://github.com/pharo-project/pharo/pull/462/files > > Thanks to MarcusDenker > > * 20263-Debugger-DoesNotUnderstandDebugAction-is-missing-a-defaultHelp-method-so-hover-help-shows-missing > Debugger DoesNotUnderstandDebugAction is missing a > #defaultHelp method so hover help shows missing > > https://pharo.fogbugz.com/f/cases/20263/ > > Issue URL: https://pharo.fogbugz.com/f/cases/20263 > PR URL: https://github.com/pharo-project/pharo/pull/460 > Diff URL: https://github.com/pharo-project/pharo/pull/460/files > > Thanks to MarcusDenker > > * 17654-SpecPreDebugWindow-and-subclasses-relies-on-doesNotUnderstand-delegation-to-find-the-stackPane > SpecPreDebugWindow and subclasses relies on #doesNotUnderstand > delegation to find the stackPane > > https://pharo.fogbugz.com/f/cases/17654 > > Issue URL: https://pharo.fogbugz.com/f/cases/17654 > PR URL: https://github.com/pharo-project/pharo/pull/461 > Diff URL: https://github.com/pharo-project/pharo/pull/461/files > > Thanks to MarcusDenker > > * 20363-ThemeIconsdownloadTo-has-an-argument-which-is-never-referenced-in-code > fix for > https://pharo.fogbugz.com/f/cases/20363/ThemeIcons-downloadTo-has-an-argume… > > Issue URL: https://pharo.fogbugz.com/f/cases/20363 > PR URL: https://github.com/pharo-project/pharo/pull/453 > Diff URL: https://github.com/pharo-project/pharo/pull/453/files > > Thanks to MarcusDenker > > * 20667-Be-able-to-install-breakpoints-by-clicking-on-segment-icons-area > https://pharo.fogbugz.com/f/cases/20667/Be-able-to-install-breakpoints-by-c… > > click on segment icons area adds breakpoint at the beginning of the line > > Issue URL: https://pharo.fogbugz.com/f/cases/20667 > PR URL: https://github.com/pharo-project/pharo/pull/459 > Diff URL: https://github.com/pharo-project/pharo/pull/459/files > > Thanks to pavel-krivanek > > * 20582-atat-messages-call-errorKeyNotFound > Fix for at:at: messages call errorKeyNotFound. > - use errorKeyNotFound: > > https://pharo.fogbugz.com/f/cases/20582/at-at-messages-call-errorKeyNotFound > > Issue URL: https://pharo.fogbugz.com/f/cases/20582 > PR URL: https://github.com/pharo-project/pharo/pull/454 > Diff URL: https://github.com/pharo-project/pharo/pull/454/files > > Thanks to MarcusDenker > > * 20664-set-the-display-size-sooner > https://pharo.fogbugz.com/f/cases/20664/set-the-display-size-sooner > > Issue URL: https://pharo.fogbugz.com/f/cases/20664 > PR URL: https://github.com/pharo-project/pharo/pull/458 > Diff URL: https://github.com/pharo-project/pharo/pull/458/files > > Thanks to pavel-krivanek > > * 20499-Examples-in-GlamourPresentationModel-should-have-no-gtExamplePragma > 20499 > Examples in GlamourPresentationModel should have no <gtExamplePragma> > > https://pharo.fogbugz.com/f/cases/20499 > > Issue URL: https://pharo.fogbugz.com/f/cases/20499 > PR URL: https://github.com/pharo-project/pharo/pull/455 > Diff URL: https://github.com/pharo-project/pharo/pull/455/files > > Thanks to MarcusDenker > > * 20662-displaying-method-versions-in-Nautilus-causes-DNU > https://pharo.fogbugz.com/f/cases/20662/displaying-method-versions-in-Nauti… > > add clearUndoManager to DiffPresenter > > Issue URL: https://pharo.fogbugz.com/f/cases/20662 > PR URL: https://github.com/pharo-project/pharo/pull/457 > Diff URL: https://github.com/pharo-project/pharo/pull/457/files > > Thanks to pavel-krivanek > > * 20661-Fixing-test-from-debugger-should-mark-test-as-green-when-proceed- > Now runCaseForDebug: removes passed test case from errors and failures. > This logic is not inside addPass: method directly to not > affect bug suite result collecting because it can become slow if it > will clean errors and failures for every green test. > Maybe in future we can improve this part. > > https://pharo.fogbugz.com/f/cases/20661/Fixing-test-from-debugger-should-ma… > > Issue URL: https://pharo.fogbugz.com/f/cases/20661 > PR URL: https://github.com/pharo-project/pharo/pull/456 > Diff URL: https://github.com/pharo-project/pharo/pull/456/files > > Thanks to dionisiydk > > * 20611-leftover-logCr-debug-logging-in-GLEmptyBrickarea > remove left over debug code > https://pharo.fogbugz.com/f/cases/20611/leftover-logCr-debug-logging-in-GLE… > > Issue URL: https://pharo.fogbugz.com/f/cases/20611 > PR URL: https://github.com/pharo-project/pharo/pull/452 > Diff URL: https://github.com/pharo-project/pharo/pull/452/files > > Thanks to MarcusDenker > > * 20552-Breakpoints-stay-in-the-breakpoints-browser-even-after-their-method-is-recompiled > Issue URL: > https://pharo.fogbugz.com/f/cases/20552/Breakpoints-stay-in-the-breakpoints… > Breakpoint class now subscribes to the MethodChanged > announcement, and remove from its list the breakpoints from methods > that get recompiled. > Also added a test for this behaviour. > > Issue URL: https://pharo.fogbugz.com/f/cases/20552 > PR URL: https://github.com/pharo-project/pharo/pull/447 > Diff URL: https://github.com/pharo-project/pharo/pull/447/files > > Thanks to dupriezt > > * 20657-Command-B-in-playground-does-not-work-the-save-way-as-printit > https://pharo.fogbugz.com/f/cases/20657 > > try to select line if no AST node found > > Issue URL: https://pharo.fogbugz.com/f/cases/20657 > PR URL: https://github.com/pharo-project/pharo/pull/450 > Diff URL: https://github.com/pharo-project/pharo/pull/450/files > > Thanks to pavel-krivanek > > * 19972-Implementers-code-pane-undo-goes-too-far > Clearing the undo history when changing the selected method in the list. > Also adding a way of clearing this history from the TextPresenter > > Issue URL: https://pharo.fogbugz.com/f/cases/19972 > PR URL: https://github.com/pharo-project/pharo/pull/451 > Diff URL: https://github.com/pharo-project/pharo/pull/451/files > > Thanks to tesonep > > * 20656-Nautilus-comment-panel-got-broken-by-asLayoutFrame-cleans > https://pharo.fogbugz.com/f/cases/20656 > > fix layout frame fraction > > Issue URL: https://pharo.fogbugz.com/f/cases/20656 > PR URL: https://github.com/pharo-project/pharo/pull/449 > Diff URL: https://github.com/pharo-project/pharo/pull/449/files > > Thanks to pavel-krivanek > > * 20650-Release-cleanup-failures > https://pharo.fogbugz.com/f/cases/20650/Release-cleanup-failures > > Issue URL: https://pharo.fogbugz.com/f/cases/20650 > PR URL: https://github.com/pharo-project/pharo/pull/448 > Diff URL: https://github.com/pharo-project/pharo/pull/448/files > > Thanks to pavel-krivanek > > * 20648-Provide-Bitmap-variant-of-Source-Code-fonts > https://pharo.fogbugz.com/f/cases/20648/Provide-Bitmap-variant-of-Source-Co… > > based on PR https://github.com/pharo-project/pharo/pull/439 > > Issue URL: https://pharo.fogbugz.com/f/cases/20648 > PR URL: https://github.com/pharo-project/pharo/pull/441 > Diff URL: https://github.com/pharo-project/pharo/pull/441/files > > Thanks to pavel-krivanek > > * 20644-Load-Bitmap-DejaVu-fonts-from-data-files-instead-of-a-Fuel-file > https://pharo.fogbugz.com/f/cases/20644/Load-Bitmap-DejaVu-fonts-from-data-… > > Issue URL: https://pharo.fogbugz.com/f/cases/20644 > PR URL: https://github.com/pharo-project/pharo/pull/439 > Diff URL: https://github.com/pharo-project/pharo/pull/439/files > > Thanks to pavel-krivanek > > * 20643 Dark and light theme should not be version specific > Rename Pharo3DarkTheme into PharoDarkTheme > Rename Pharo3Theme into PharoLightTheme > and deprecate empty subclasses like Pharo3DarkTheme > and Pharo3Theme (but keep them for compatibility) > > Issue URL: https://pharo.fogbugz.com/f/cases/20643 Dark and > light theme should not be version specific > PR URL: https://github.com/pharo-project/pharo/pull/438 > Diff URL: https://github.com/pharo-project/pharo/pull/438/files > > Thanks to astares > > * Care on dirty package OpalTools > https://pharo.fogbugz.com/f/cases/20642/Care-on-dirty-package-OpalTools > > fix dirty package extension to be in OpalCompiler-Tools > > Issue URL: https://pharo.fogbugz.com/f/cases/Care on dirty > package OpalTools > PR URL: https://github.com/pharo-project/pharo/pull/437 > Diff URL: https://github.com/pharo-project/pharo/pull/437/files > > Thanks to astares > > * 20520 Package "OpalTools" should be "OpalCompiler-Tools" > Rename package OpalTools -> OpalCompiler-Tools > > https://pharo.fogbugz.com/f/cases/20520/Package-OpalTools-should-be-OpalCom… > > Issue URL: https://pharo.fogbugz.com/f/cases/20520 Package > "OpalTools" should be "OpalCompiler > PR URL: https://github.com/pharo-project/pharo/pull/436 > Diff URL: https://github.com/pharo-project/pharo/pull/436/files > > Thanks to astares > > * Stupid use of asLayoutFrame should be eliminated first pass. > Fuck my image just crashed again! ;((((((((((((((((((((((( > > Issue URL: https://pharo.fogbugz.com/f/cases/Stupid use of > asLayoutFrame should be eliminated first pass. > PR URL: https://github.com/pharo-project/pharo/pull/434 > Diff URL: https://github.com/pharo-project/pharo/pull/434/files > > Thanks to Ducasse > > * 20640 Categorize uncategorized methods in classes starting with "P" > Categorize methods. > > > So only categorization - no change in behavior > > Issue URL: https://pharo.fogbugz.com/f/cases/20640 Categorize > uncategorized methods in classes starting with "P" > PR URL: https://github.com/pharo-project/pharo/pull/433 > Diff URL: https://github.com/pharo-project/pharo/pull/433/files > > Thanks to astares > -- www.tudorgirba.com www.feenk.com âThe smaller and more pervasive the hardware becomes, the more physical the software gets."

Nov. 24, 2017

Re: [Pharo-dev] OpalCompiler evaluate speed

by Marcus Denker

> On 23 Nov 2017, at 23:37, ClÃ©ment Bera <bera.clement(a)gmail.com> wrote: > > Hi Nicolas. > > Just some comments: > > Another thing you can try is to remove the allocation of Opal's IR. It seems people use the IR only through the IRBuilder, so the API can be kept but it can generate directly bytecode instead of IR then bytecode. Removing those allocations would speed things up. It means merging IRFix / IRTranslator / IRBytecodeGenerator somehow and have the IRBuilder API directly call the new resulting merged class. > > Another thing is that when Opal became the default compiler, I evaluated the speed and saw it was slower, but when loading large projects it seemed loading time was dominated by Monticello / source reading / source loading and the compilation time was overall not that significant (< 20% of time). I don't know if this is still the case with GIT. I have problems currently when editing some large methods (it seems in the IDE the method is compiled at each keystroke...) and when doing OpalCompiler recompileAll (which I do often since I edit bytecode sets) but else the performance of Opal seems to be OK. Evaluate performance may be relevant in some cases but I've never found such cases outside of the IDE in production. > Yes, the IR came over from the existing design of the ClosureCompiler for Squeak that Opal is was based on initiallyâ¦ I liked it at first because I was looking into byte code manipulation back then and for that it was quite nice. But for compiling, it adds a layer that is not really needed. If I would do it again I would not use this IR level in the compiler but instead do something simpler and use the BytecodeBuilder directly. But as Clement notes: it is not that much time when doing compilation if you consider the whole use-case of compiling, so this seemed not that of a problem (e.g. for compiling the image, we were happy with a factor 2, I did not benchmark recently, but I think this is what we reached). For the frontend (compile to AST + name analysis), it is fast enough to do syntax highlighting in the browser with it instead of a specialised parser, so that part is quite ok. One other thing that needs cleanup: the API/ OpalCompiler class. This tries to be compatible with the old API (e.g. failBlock) and got quite hacked to death over timeâ¦ we should clean that up. The complexity of #evaluate is partly due to that, I think. Marcus

Nov. 24, 2017

Re: [Pharo-dev] Problem with primitive instVarAt: / instVarAt:put:, what should we do ?

by ClÃ©ment Bera

In the VM primitiveSlotAt: / at:Put: mean access to any field (pointer or non pointer, sorry for the confusion Marcus earlier on) of an object. The primitive in the image can be named fieldAt:/fieldAt:put: if it makes more sense for Pharo. #[0] slotAt: 1 => 0 #(#a) slotAt: 1 => #a 0@2 slotAt: 1 => 0 With Pharo 6 blocks: | t | t := #tmp. {[ t ] slotAt: 3 . [ t ] slotAt: 4} => { 0 "numArgs". #tmp } Right now you can do that with instVarAt: on the contrary to what instVarAt: comment states, and this is wrong. On Fri, Nov 24, 2017 at 10:32 AM, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > > On 24 Nov 2017, at 00:21, ClÃ©ment Bera <bera.clement(a)gmail.com> wrote: > > > > On Fri, Nov 24, 2017 at 12:00 AM, Stephane Ducasse <stepharo.self@gmail. > com> wrote: > >> Did you talk with marcus? >> > > I don't understand the connection between slots and this problem with > primitives. > >> >> > There is no connection, just the same name. > > For Slots aka First Class Instance variables: > > I sometimes think that Slot for the first class instance variables might > not be a good name, but then, > the only alternative would be âinstance variableâ, but that is not that > nice either, as these Slots include > virtual variables (that are computed) or variables that are combine stored > in a hidden base slot (e.g. > used for BooleanSlot or PropertySlot). > > So for now I will keep the Slot termâ¦ > > > Marcus > -- ClÃ©ment BÃ©ra Pharo consortium engineer https://clementbera.wordpress.com/ BÃ¢timent B 40, avenue Halley 59650 Villeneuve d'Ascq

Nov. 24, 2017

Re: [Pharo-dev] Problem with primitive instVarAt: / instVarAt:put:, what should we do ?

by Marcus Denker

> On 24 Nov 2017, at 00:21, ClÃ©ment Bera <bera.clement(a)gmail.com> wrote: > > > > On Fri, Nov 24, 2017 at 12:00 AM, Stephane Ducasse <stepharo.self(a)gmail.com <mailto:stepharo.self@gmail.com>> wrote: > Did you talk with marcus? > > I don't understand the connection between slots and this problem with primitives. > There is no connection, just the same name. For Slots aka First Class Instance variables: I sometimes think that Slot for the first class instance variables might not be a good name, but then, the only alternative would be âinstance variableâ, but that is not that nice either, as these Slots include virtual variables (that are computed) or variables that are combine stored in a hidden base slot (e.g. used for BooleanSlot or PropertySlot). So for now I will keep the Slot termâ¦ Marcus

Nov. 24, 2017

Re: [Pharo-dev] OpalCompiler evaluate speed

by Marcus Denker

> On 22 Nov 2017, at 21:14, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: > > Thanks this is good. > I think that Opal deserves more energy. > We are busy right now. Now marcus will certainly be interested. Yes, sadly I am over capacity. I can not react (or even read) every email send to the mailinglist anymore. I have marked this to check for later. Marcus

Nov. 24, 2017

Re: [Pharo-dev] OpalCompiler evaluate speed

by Nicolas Cellier

2017-11-23 23:37 GMT+01:00 ClÃ©ment Bera <bera.clement(a)gmail.com>: > Hi Nicolas. > > Just some comments: > > Another thing you can try is to remove the allocation of Opal's IR. It > seems people use the IR only through the IRBuilder, so the API can be kept > but it can generate directly bytecode instead of IR then bytecode. Removing > those allocations would speed things up. It means merging IRFix / > IRTranslator / IRBytecodeGenerator somehow and have the IRBuilder API > directly call the new resulting merged class. > > Yes, I was thinking of this optimization: use IR only when we need advanced tools/instrumentation. Another thing is that when Opal became the default compiler, I evaluated > the speed and saw it was slower, but when loading large projects it seemed > loading time was dominated by Monticello / source reading / source loading > and the compilation time was overall not that significant (< 20% of time). > I don't know if this is still the case with GIT. I have problems currently > when editing some large methods (it seems in the IDE the method is compiled > at each keystroke...) and when doing OpalCompiler recompileAll (which I do > often since I edit bytecode sets) but else the performance of Opal seems to > be OK. Evaluate performance may be relevant in some cases but I've never > found such cases outside of the IDE in production. > > Best ! > > Clement > > Right, my case was generating the strings, so Compiler was the main contributor. For more general case, it should be benched. I was thinking of a tool for checking the example comments "some code example >>> some result". > On Thu, Nov 23, 2017 at 9:41 PM, Nicolas Cellier < > nicolas.cellier.aka.nice(a)gmail.com> wrote: > >> >> >> 2017-11-22 0:31 GMT+01:00 Ben Coman <btc(a)openinworld.com>: >> >>> >>> >>> On 22 November 2017 at 05:49, Nicolas Cellier < >>> nicolas.cellier.aka.nice(a)gmail.com> wrote: >>> >>>> >>>> >>>> 2017-11-21 14:19 GMT+01:00 Nicolas Cellier < >>>> nicolas.cellier.aka.nice(a)gmail.com>: >>>> >>>>> I have an ArbitraryPrecisionFloatTests doing an exhaustive test for >>>>> printing and reevaluating all positve half precision float. >>>>> >>>>> That's about 2^15 or approximately 32k loop which evaluate snippets >>>>> like >>>>> >>>>> (ArbitraryPrecisionFloat readFrom: '1.123' readStream numBits: 10) >>>>> >>>>> The test was naively written with Compiler evaluate: and was using the >>>>> legacy Compiler. >>>>> >>>>> If I rewrite self class compiler evaluate: the test times out. >>>>> Let's see what increase is necessary: >>>>> >>>>> [ ArbitraryPrecisionFloatTest new testPrintAndEvaluate ] >>>>> timeToRun. >>>>> -> 3s with legacy Compiler >>>>> -> 14s with OpalCompiler >>>>> >>>>> It's not unexpected that intermediate representation (IR) reification >>>>> has a cost, but here the 4.5x is a bit too much... >>>>> This test did account for 1/4 of total test duration already (3s out >>>>> of 12s). >>>>> With Opal, the total test duration doubles... (14s out of 23s) >>>>> >>>>> So let's analyze the hot spot with: >>>>> >>>>> MessageTally spyOn: [ ArbitraryPrecisionFloatTest new >>>>> testPrintAndEvaluate ]. >>>>> >>>>> (I didn't use AndreasSystemProfiler becuase outputs seems a bit >>>>> garbbled, no matter since the primitives do not account that much, a >>>>> MessageTally will do the job) >>>>> >>>>> I first see a hot spot which does not seem that necessary: >>>>> >>>>> | |24.6% {3447ms} RBMethodNode(RBProgramNode)>>formattedCode >>>>> >>>>> From the comments I understand that AST-based stuff requires a pattern >>>>> (DoIt) and an explicit return (^), but this expensive formatting seems too >>>>> much for just evaluating. i think that we should change that. >>>>> >>>>> Then comes: >>>>> >>>>> | |20.7% {2902ms} RBMethodNode>>generate: >>>>> >>>>> which is split in two halves, ATS->IR and IR->bytecode >>>>> >>>>> | | |9.3% {1299ms} RBMethodNode>>generateIR >>>>> >>>>> | | | |11.4% {1596ms} IRMethod>>generate: >>>>> >>>>> But then I see this cost a 2nd time which also leave room for progress: >>>>> >>>>> | |10.9% {1529ms} RBMethodNode>>generateIR >>>>> >>>>> | | |12.9% {1814ms} IRMethod>>generate: >>>>> >>>>> The first is in RBMethodNode>>generateWithSource, the second in >>>>> OpalCompiler>>compile >>>>> >>>>> Last comes the parse time (sourceCode -> AST) >>>>> >>>>> | 13.2% {1858ms} OpalCompiler>>parse >>>>> >>>>> Along with semantic analysis >>>>> >>>>> | 6.0% {837ms} OpalCompiler>>doSemanticAnalysis >>>>> >>>>> ----------------------------------- >>>>> >>>>> For comparison, the legacy Compiler decomposes into: >>>>> >>>>> | |61.5% {2223ms} Parser>>parse:class:category:n >>>>> oPattern:context:notifying:ifFail: >>>>> >>>>> which more or less covers parse time + semantic analysis time. >>>>> That means that Opal does a fair work for this stage. >>>>> >>>>> Then, the direct AST->byteCode phase is: >>>>> >>>>> | 16.9% {609ms} MethodNode>>generate >>>>> >>>>> IR costs almost a 5x on this phase, but we know it's the price to pay >>>>> for the additional features that it potentially offers. If only we would do >>>>> it once... >>>>> >>>>> And that's all for the legacy one... >>>>> >>>>> -------------------------------------- >>>>> >>>>> This little exercize shows that a 2x acceleration of OpalCompiler >>>>> evaluate seems achievable: >>>>> - simplify the uselessely expensive formatted code >>>>> - generate bytecodes once, not twice >>>>> >>>>> Then it will be a bit more 2x slower than legacy, which is a better >>>>> trade for yet to come superior features potentially brought by Opal. >>>>> >>>>> It would be interesting to carry same analysis on method compilation >>>>> >>>> >>>> Digging further here is what I find: >>>> >>>> compile sends generate: and answer a CompiledMethod >>>> translate sends compile but throw the CompiledMethod away, and just >>>> answer the AST. >>>> Most senders of translate will also generate: (thus we generate: twice >>>> quite often, loosing a 2x factor in compilation). >>>> >>>> A 2x gain is a huge gain when installing big code bases, especially if >>>> the custom is to throw image away and reconstruct. >>>> No matter if a bot does the job, it does it for twice many watts and at >>>> the end, we're waiting for the bot. >>>> >>>> However, before changing anything, further clarification is required: >>>> translate does one more thing, it catches ReparseAfterSourceEditing and >>>> retry compilation (once). >>>> So my question: are there some cases when generate: will cause >>>> ReparseAfterSourceEditing? >>>> >>> >>> I don't know the full answer about other cases, but I can provide the >>> background why ReparseAfterSourceEditing was added. >>> >>> IIRC, a few years ago with the move to an AST based system, there was a >>> problem with syntax highlighting where >>> the AST referenced its original source which caused highlighting offsets >>> when reference to source modified in the editor. >>> Trying to work backwards from modified source to update all AST elements >>> source-location proved an intractable problem. >>> The workaround I found was to move only in a forward direction >>> regenerating AST from source every keystroke. >>> Performance was acceptable so this became the permanent solution. >>> >>> I don't have access to an image to check, but you should find >>> ReparseAfterSourceEditing raised only in one location near editor #changed: >>> Maybe this should activate only for interactively modified code, and >>> disabled/bypassed for bulk loading code. >>> For testing purposes commenting it out should not harm the system, just >>> produce visual artifacts in syntax highlighting. >>> >>> >>> >>>> That could happen in generation phase if some byte code limit is >>>> exceeded, and an interactive handling corrects code... >>>> I did not see any such condition, but code base is huge... >>>> >>> >>> At worst, the impact should only be a temporary visual artifact. >>> Corrected on the next keystroke. >>> (unless ReparseAfterSourceEditing has been adopted for other than >>> original purpose, but I'd guess not) >>> >>> cheers -ben >>> >>> Hi Ben, >> Thanks for information. >> We must keep ReparseAfterSourceEditing, it does its job. >> >> But it just sounds like we have an inversion: >> >> translate (source code->AST) does call compile (source >> code->AST->bytecode in a CompiledMethod) >> >> I would expect the other way around: if we want to compile, we need to >> translate first. >> If we want to translate, we don't really need to compile, unless there's >> an hidden reason... >> Thus my question: is there an hidden reason? >> > > > > -- > ClÃ©ment BÃ©ra > Pharo consortium engineer > https://clementbera.wordpress.com/ > BÃ¢timent B 40, avenue Halley 59650 > <https://maps.google.com/?q=40,+avenue+Halley+59650%C2%A0Villeneuve+d'Ascq&entry=gmail&source=g>Villeneuve > d > <https://maps.google.com/?q=40,+avenue+Halley+59650%C2%A0Villeneuve+d'Ascq&entry=gmail&source=g> > 'Ascq > <https://maps.google.com/?q=40,+avenue+Halley+59650%C2%A0Villeneuve+d'Ascq&entry=gmail&source=g> >

Nov. 24, 2017

Re: [Pharo-dev] Bloc Tutorial Feedback

by Aliaksei Syrel

Hi Sean, That is awesome, thank you! :) - The booklet reached from the GH readme seems out of sync with Bloc. I had > to alter a lot of the animations from the loaded code e.g. > BlTransformAnimation did not exist (I used BlScaleAnimation instead), > animations didn't understand #absolute, ` animation startOn: target` needed > to be changed to `animation target: target; start`, etc. That is the Iceberg issue (https://github.com/pharo-vcs/iceberg/issues/479) Tutorial requires you to load a stable version of Bloc from *pharo6.1* branch. However, iceberg does not do that and instead loads an ancient bloc version from #master branch because you probably have a shared repo of bloc somewhere on your machine. The tutorial's code is up-to date :) Could you try to remove any bloc clones from you computer, take a fresh pharo6.1 image and install Tutorial following the readme? ( https://github.com/pharo-graphics/Tutorials) Cheers, Alex On 24 November 2017 at 08:38, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: > Aliaksei told me that he should do another pass to add animations and > layout chapters. > I imagine that he did some refactorings so this is good that you spotted > them. > > > On Fri, Nov 24, 2017 at 8:36 AM, Stephane Ducasse > <stepharo.self(a)gmail.com> wrote: > > Thanks Sean I will have a look when I get some time. > > > > > > On Fri, Nov 24, 2017 at 4:41 AM, Sean P. DeNigris <sean(a)clipperadams.com> > wrote: > >> I did a native English review pass on the first half and submitted a PR > on > >> GH. > >> > >> A few other items: > >> - When clipping is used to initially draw the card outline as a rounded > >> rect, instead of using the rect as a path, it's not immediately obvious > why > >> this is done. It's only way later when the cross is drawn on the back > of the > >> card that one /may/ realize that it's so card contents (e.g. the cross) > >> don't get drawn outside the rounded bounds. It would be good to explain > this > >> upfront. > >> - The booklet reached from the GH readme seems out of sync with Bloc. I > had > >> to alter a lot of the animations from the loaded code e.g. > >> BlTransformAnimation did not exist (I used BlScaleAnimation instead), > >> animations didn't understand #absolute, ` animation startOn: target` > needed > >> to be changed to `animation target: target; start`, etc. > >> > >> Otherwise a very nice tutorial. Thanks! > >> > >> > >> > >> > >> ----- > >> Cheers, > >> Sean > >> -- > >> Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.ht > ml > >> > >

Nov. 24, 2017

Re: [Pharo-dev] [Vm-dev] FFI Struct Argument Pass By Value Fails on Mac 64 bit

by Ben Coman

On 24 November 2017 at 13:16, Ben Coman <btc(a)openinworld.com> wrote: > > > On 22 November 2017 at 21:59, Ben Coman <btc(a)openinworld.com> wrote: > >> >> >> On 22 November 2017 at 13:38, Todd Blanchard <tblanchard(a)mac.com> wrote: >> >>> >>> I've been trying to track this down for a couple weeks now. >>> >>> I have concluded that structs passed by value to functions on the 64 bit >>> VM are not properly populated. The struct's memory is all zero'd. >>> >>> I found this while trying to work with LibClang and found that functions >>> that fetched code locations from code ranges always returned invalid zero'd >>> locations. After spending some time with lldb I have traced the problem >>> into the native code and found that the argument is not correct. >>> >>> I've carved out the wee bit of clang to reproduce this in a tiny library. >>> >>> The gist of it is below and the entire file is included. Basically the >>> struct passed to the function clang_getRangeStart is zero'd memory >>> regardless of the data I send from the image side. >>> >>> The build command I used on sierra is clang -shared -undefined >>> dynamic_lookup -o microclang.dylib microclang.c >>> >> >> On Ubuntu 16.04 I used... >> $ clang -shared -fPIC -o libmicroclang.so microclang.c >> >> $ clang test.c -L. -l microclang >> test.c:6:53: error: no member named 'begin_int_data' in >> 'CXSourceLocation' >> if(clang_getRangeStart(clang_getArbitraryRange()).begin_int_data == >> 0) >> >> I presume you meant... >> if(clang_getRangeStart(clang_getArbitraryRange()).int_data == 0) >> so correcting and continuing... >> >> $ clang test.c -L. -l microclang >> $ LD_LIBRARY_PATH=. ./a.out >> That failed >> >> So I'm not sure how to proceed. >> I was expecting that would work while Pharo failed. >> >> Now interestingly... >> $ clang test.c microlang.c >> $ ./a.out >> That worked >> >> >> So it seems a similar problem exists outside our FFI. >> >> cheers -ben >> >> P.S. I refactored you code to extract a header file (attached) >> > > The issue is still beyond my ken, but I've made some progress towards > isolating/understanding the issue. > Attached zip exploded here for easy reference... > > > ___microlang.h___ > typedef unsigned uintptr_t; > > typedef struct { > const void *ptr_data[2]; > } CXSourceRange_; > > CXSourceRange_ clang_getArbitraryRange_(); > int clang_getRangeEnd_(CXSourceRange_ range); > > > > ___microclang.c___ > #include "microclang.h" > const char* libraryString = "library_pointer"; > > CXSourceRange_ clang_getArbitraryRange_() > { CXSourceRange_ range = {0}; > range.ptr_data[0] = (void*)libraryString; > return range; > } > > int clang_getRangeEnd_(CXSourceRange_ range) > { // Special decoding for CXSourceLocations for CXLoadedDiagnostics. > if ((uintptr_t)range.ptr_data[0] & 0x1) > { return 0; } > else > { return 1; } > } > > > > > ___test.c___ > #include <stdio.h> > #include "microclang.h" > const char* localString = "local_pointer"; > > void test( CXSourceRange_ range, char *note ) > { int result = clang_getRangeEnd_(range); > if(result == 0) > { printf("That failed (%s)\n", note); } > else > { printf("That worked (%s)\n", note); } > } > > int main() > { CXSourceRange_ range1 = clang_getArbitraryRange_(); > test(range1, "library string"); > > CXSourceRange_ range2 = {0}; > range2.ptr_data[0] = (void*)localString; > test(range2, "local string"); > } > > > > ___Makefile___ > default: clean static shared > > clean: > rm -f *so *App > @echo > > shared: > clang -g -o libmicroclang.so -shared -fPIC microclang.c > clang -g -o sharedApp test.c -L. -lmicroclang > LD_LIBRARY_PATH=. ./sharedApp > @echo > > static: > clang -g -o staticApp test.c microclang.c > ./staticApp > @echo > > > > Now running... > $ make > report > > gives... > ___report___ > rm -f *so *App > > clang -g -o staticApp test.c microclang.c > ./staticApp > That worked (library string) > That worked (local string) > > clang -g -o libmicroclang.so -shared -fPIC microclang.c > clang -g -o sharedApp test.c -L. -lmicroclang > LD_LIBRARY_PATH=. ./sharedApp > That failed (library string) > That worked (local string) > Further simplification dealing *only* with strings (see attached sharedLibString.zip) (also attached is clang3.zip as a step along the way) ___microclang.c___ typedef unsigned uintptr_t; const char* myLibraryString = "library_pointer"; const char * lib_getLibraryString() { return myLibraryString; } int lib_testString( const char *aString ) { unsigned test = (uintptr_t)aString & 0x1; printf("\n test=%d, aString-->%d\n", test, (uintptr_t)aString); if (test) { return 0; } else { return 1; } } ___test.c___ #include <stdio.h> int lib_testString( const char *aString ); const char *lib_getLibraryString(); const char *localString = "local_pointer"; void test( int result, char *note ) { if(result == 0) { printf("That failed (%s)\n", note); } else { printf("That worked (%s)\n", note); } } int main() { const char * libraryString = lib_getLibraryString(); test(lib_testString(libraryString), "library string"); test(lib_testString(localString), "local string"); } $ make > report ___report___ rm -f *so *App clang -g -o staticApp test.c microclang.c ./staticApp test=0, aString-->4196150 That worked (library string) test=0, aString-->4196068 That worked (local string) clang -g -o libmicroclang.so -shared -fPIC microclang.c clang -g -o sharedApp test.c -L. -lmicroclang LD_LIBRARY_PATH=. ./sharedApp test=1, aString-->-792512599 That failed (library string) test=0, aString-->4196484 That worked (local string) cheers -ben

Nov. 24, 2017

Re: [Pharo-dev] [PHARO 7.0A] NEXT ENHANCEMENTS

by Esteban Lorenzano

Nov. 24, 2017

← Newer
1
...
15
16
17
18
19
20
21
...
85
Older →

Jump to page: