April 2016 - Pharo-dev - lists.pharo.org

[pharo-project/pharo-core] a263af: 50673

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: a263afdfbce0dd993f80824f2fe504bd6ca11a20 https://github.com/pharo-project/pharo-core/commit/a263afdfbce0dd993f80824f… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-04-01 (Fri, 01 Apr 2016) Changed paths: A Athens-Cairo.package/AthensCairoCanvas.class/instance/private/primSetDashesLengths_count_offset_.st M Athens-Cairo.package/AthensCairoCanvas.class/instance/private/primSetLineCap_.st M Athens-Cairo.package/AthensCairoCanvas.class/instance/private/primSetLineJoin_.st M Athens-Cairo.package/AthensCairoCanvas.class/instance/private/primSetLineWidth_.st A Athens-Cairo.package/AthensCairoCanvas.class/instance/private/primSetSource_.st M Athens-Cairo.package/AthensCairoMatrix.class/instance/transformations/translateBy_.st M Athens-Cairo.package/AthensCairoPatternPaint.class/instance/drawing/athensLoadOn_.st M Athens-Cairo.package/AthensCairoPatternPaint.class/instance/drawing/loadOnCairoCanvas_.st M Athens-Cairo.package/AthensCairoPatternPaint.class/instance/primitives/primSetSourceOn_.st M Athens-Cairo.package/AthensCairoStrokePaint.class/instance/private/prepareForDrawingOn_.st M Athens-Cairo.package/AthensCairoStrokePaint.class/instance/private/setDashesOn_.st R Athens-Cairo.package/AthensCairoSurface.class/class/library path/ffiLibraryName.st R Athens-Cairo.package/TCairoLibrary.class/class/library path/nbLibraryNameOrHandle.st M ConfigurationOfAthens.package/ConfigurationOfAthens.class/instance/symbolic versions/stable_.st A ConfigurationOfAthens.package/ConfigurationOfAthens.class/instance/versions/version391_.st A ConfigurationOfAthens.package/ConfigurationOfAthens.class/instance/versions/version392_.st M Metacello-GitBasedRepository.package/MCGitBasedNetworkRepository.class/class/initialization/initialize.st A Metacello-GitHub.package/MCGitHubRepository.class/class/initialization/initialize.st R Metacello-GitHub.package/MCGitHubRepository.class/class/initialize-release/initialize.st R Metacello-GitHub.package/MCGitHubRepository.class/class/initialize-release/release.st M Metacello-GitHub.package/MCGitHubRepository.class/instance/private/downloadJSONTags.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50672.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50673.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50672.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50673.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50673 17887 Update startup of metacello to use the session manager https://pharo.fogbugz.com/f/cases/17887 17922 Update Athens 3.9.2 https://pharo.fogbugz.com/f/cases/17922 http://files.pharo.org/image/50/50673.zip

April 1, 2016

Re: [Pharo-dev] Catching Exceptions without any notice

by Tudor Girba

Hi, Nice to hear from you Igor. I am glad to see you around here. I do not see how your quote applies to the current case given that the original authors did not leave anywhere, but perhaps it was a joke, and I did not get it. Cheers, Doru > On Apr 1, 2016, at 5:54 AM, Igor Stasenko <siguctua(a)gmail.com> wrote: > > A perfect example of careless programming. > "I'll do it my way, and if it causing any problems, i don't care and i will just ignore them. And it's not my problem anyways, i went to something else already, since this part is works and DONE" > :) > > On 30 March 2016 at 14:33, Nicolai Hess <nicolaihess(a)gmail.com> wrote: > Please don't do this: > > updateHeight > "no need to care about height, when it's logic is not customized" > self layout isHeightCustom ifFalse: [ ^ self ]. > [ self bounds: (self brickBounds withHeight: self customHeight) ] > on: Exception > do: [ "just skip and do nothing" ] > > This makes debugging GLM/Brick ui/layout code with "self haltOnce" impossible. > see > GLMBrickGeometryTrait>>#updateHeight > GLMBrickGeometryTrait>>#updateWidth > > And if you log out the raised exception, you see some calls to > not initialized fonts and a ZeroDevide and some more errors. > The above catch, catches and hides wrong / to late initialized objects. > > > > -- > Best regards, > Igor Stasenko. -- www.tudorgirba.com www.feenk.com "No matter how many recipes we know, we still value a chef."

April 1, 2016

Re: [Pharo-dev] Catching Exceptions without any notice

by Igor Stasenko

A perfect example of careless programming. "I'll do it my way, and if it causing any problems, i don't care and i will just ignore them. And it's not my problem anyways, i went to something else already, since this part is works and DONE" :) On 30 March 2016 at 14:33, Nicolai Hess <nicolaihess(a)gmail.com> wrote: > Please don't do this: > > updateHeight > "no need to care about height, when it's logic is not customized" > self layout isHeightCustom ifFalse: [ ^ self ]. > [ self bounds: (self brickBounds withHeight: self customHeight) ] > on: Exception > do: [ "just skip and do nothing" ] > > This makes debugging GLM/Brick ui/layout code with "self haltOnce" > impossible. > see > GLMBrickGeometryTrait>>#updateHeight > GLMBrickGeometryTrait>>#updateWidth > > And if you log out the raised exception, you see some calls to > not initialized fonts and a ZeroDevide and some more errors. > The above catch, catches and hides wrong / to late initialized objects. > -- Best regards, Igor Stasenko.

April 1, 2016