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

Re: [Pharo-dev] Comments in STON

by Sven Van Caekenberghe

Guille, I understand the problem. The reason is that STONCStyleCommentsSkipStream was/is a quick hack and does not know/understand anything about strings. It seems that the C preprocessor ignores comments in C strings, so we could consider adding that. Basically STONCStyleCommentsSkipStream>>#nextNonCommentChar should know when it is inside a string and then ignore comments. But that means, in the case of STON/JSON, handling both ' (single) and " (double) quoted strings as well as interpret \' and \". All this should be doable, but it is not that easy to do elegantly. I'll see what I can do. Sven > On 16 Jul 2015, at 17:35, Guillermo Polito <guillermopolito(a)gmail.com> wrote: > > Hi Sven! > > I'm finally using the new STON with comments and I found one little problem: my ston file has urls :/ > > { > // comment ok, skipped > /* comment ok, skipped */ > "aURL": "http://ouch.it.will.be.skipped/and/break?my=ston" > } > > That is, doing: > > STON fromStreamWithComments: '{ > // comment ok, skipped > /* comment ok, skipped */ > "aURL": "http://ouch.it.will.be.skipped/and/break?my=ston" > }' readStream > > Will fail with an invalid input error because after skipping comments my STON stream remains like > > STON fromStreamWithComments: '{ > "aURL": "http: > } > > Any thoughts? > Guille > > El dom., 14 de jun. de 2015 a la(s) 3:06 a. m., stepharo <stepharo(a)free.fr> escribiÃ³: > > > Le 10/6/15 20:39, Sven Van Caekenberghe a Ã©crit : > >> On 10 Jun 2015, at 15:13, stepharo <stepharo(a)free.fr> wrote: > >> > >> This is in the entreprise book. I added a subsection on comments. > >> May be I committed but forgot to push. > >> Now I did it. > > And now I see it. > > I made a small change. > super! > > > > >> Stef > >> > >> > >> Le 9/6/15 21:50, Sven Van Caekenberghe a Ã©crit : > >>> Where exactly can I find what you changed ? > >>> I don't see any commit of you on github ... > >>> > >>>> On 09 Jun 2015, at 20:10, stepharo <stepharo(a)free.fr> wrote: > >>>> > >>>> > >>>> > >>>> Le 9/6/15 20:06, stepharo a Ã©crit : > >>>>> I will update the STON chapter :) > >>>> Done :) > >>>> > >>>>> Le 8/6/15 23:39, Sven Van Caekenberghe a Ã©crit : > >>>>>> I would do it like this: use a wrapping stream to skip comments. > >>>>>> > >>>>>> === > >>>>>> Name: STON-Core-SvenVanCaekenberghe.59 > >>>>>> Author: SvenVanCaekenberghe > >>>>>> Time: 8 June 2015, 11:34:32.886167 pm > >>>>>> UUID: 7ab942b5-9d1a-44c3-9d58-da29feb72380 > >>>>>> Ancestors: STON-Core-SvenVanCaekenberghe.58 > >>>>>> > >>>>>> Add experimental STONCStyleCommentsSkipStream and tests > >>>>>> === > >>>>>> Name: STON-Tests-SvenVanCaekenberghe.54 > >>>>>> Author: SvenVanCaekenberghe > >>>>>> Time: 8 June 2015, 11:34:52.918771 pm > >>>>>> UUID: 3dcb2c9c-6efe-4d62-bf36-3bbe91f09d08 > >>>>>> Ancestors: STON-Tests-SvenVanCaekenberghe.53 > >>>>>> > >>>>>> Add experimental STONCStyleCommentsSkipStream and tests > >>>>>> === > >>>>>> > >>>>>> Then you can do as follows: > >>>>>> > >>>>>> STON fromStream: (STONCStyleCommentsSkipStream on: 'Point[/* this is X*/ 1, /* this is Y*/ 2] // Nice huh ?' readStream). > >>>>>> > >>>>>> => 1@2 > >>>>>> > >>>>>> STON fromStream: (STONCStyleCommentsSkipStream on: '// Here is how you create a point: > >>>>>> Point[ > >>>>>> // this is X > >>>>>> 1, > >>>>>> // this is Y > >>>>>> 2 ] > >>>>>> // Nice huh ?' readStream). > >>>>>> > >>>>>> => 1@2 > >>>>>> > >>>>>> This helper class would be useable in other contexts too (like NeoJSON). > >>>>>> > >>>>>> The advantage is that it does not change the STON (or JSON) syntax itself, it just add some functionality for those who want it. > >>>>>> > >>>>>> What do you think ? > >>>>>> > >>>>>> Sven > >>>>>> > >>>>>> PS: this probably needs some more testing > >>>>>> > >>>>>>> On 08 Jun 2015, at 17:16, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > >>>>>>> > >>>>>>>> On 08 Jun 2015, at 17:03, Damien Cassou <damien.cassou(a)inria.fr> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Sven Van Caekenberghe <sven(a)stfx.eu> writes: > >>>>>>>> > >>>>>>>>> I am not so sure we should add that. > >>>>>>>>> > >>>>>>>>> The JSON spec explicitly does not allow comments because of fear of abuse (that the comments would be used to add semantic meaning outside the spec). > >>>>>>>> really? That's surprising. Comments in configuration files are very > >>>>>>>> important. Even more when the configuration files are templates for new > >>>>>>>> users. > >>>>>>> I know some people are using STON in ways that I did not originally anticipate, that is of course cool, but writing configuration files was not the original goal. STON is a version of JSON that is closer to our object model. > >>>>>>> > >>>>>>> Like I said, I will think about it. > >>>>>>> > >>>>>>>> -- > >>>>>>>> Damien Cassou > >>>>>>>> http://damiencassou.seasidehosting.st > >>>>>>>> > >>>>>>>> "Success is the ability to go from one failure to another without > >>>>>>>> losing enthusiasm." --Winston Churchill > >>>>> > >>> > >> > > > > > >

July 16, 2015

Re: [Pharo-dev] libSDL2 not present in full download

by Matthieu Lacaton

> I have problems with saving the image while SDL2 windows are opened. > What OS are you using ? I don't seem to have this issue right now but I've already experienced it before. Do you have a way to reproduce it ? 2015-07-16 19:12 GMT+02:00 ClÃ©ment Bera <bera.clement(a)gmail.com>: > Hello, > > I am using SDL2 for a project and this project is in development. You have > to use the latest VM from files.pharo.org and load the bleeding edge of > OSWindow from its configuration in the latest Pharo 5. > > From my experience, it is usable but there are still a few issues like the > fullscreen mode is not working correctly for me and I have problems with > saving the image while SDL2 windows are opened. > > Best, > > Clement > > 2015-07-16 14:32 GMT+02:00 jannik laval <jannik.laval(a)gmail.com>: > >> Hi pharoers, >> >> I was trying to use OSWindow, but in a "full Download" (Image+VM), the >> plugin SDL2 is not present. >> Does this mean that the full download is based on an old VM ? >> It is the case for Pharo 4 and Pharo 5. >> >> Cheers, >> >> -- >> >> ~~Jannik Laval~~ >> Ãcole des Mines de Douai >> Enseignant-chercheur >> http://www.jannik-laval.eu >> http://www.phratch.com >> http://www.approchealpes.info >> http://car.mines-douai.fr/ >> > >

July 16, 2015

Re: [Pharo-dev] libSDL2 not present in full download

by ClÃ©ment Bera

Hello, I am using SDL2 for a project and this project is in development. You have to use the latest VM from files.pharo.org and load the bleeding edge of OSWindow from its configuration in the latest Pharo 5. >From my experience, it is usable but there are still a few issues like the fullscreen mode is not working correctly for me and I have problems with saving the image while SDL2 windows are opened. Best, Clement 2015-07-16 14:32 GMT+02:00 jannik laval <jannik.laval(a)gmail.com>: > Hi pharoers, > > I was trying to use OSWindow, but in a "full Download" (Image+VM), the > plugin SDL2 is not present. > Does this mean that the full download is based on an old VM ? > It is the case for Pharo 4 and Pharo 5. > > Cheers, > > -- > > ~~Jannik Laval~~ > Ãcole des Mines de Douai > Enseignant-chercheur > http://www.jannik-laval.eu > http://www.phratch.com > http://www.approchealpes.info > http://car.mines-douai.fr/ >

July 16, 2015

Re: [Pharo-dev] Comments in STON

by Guillermo Polito

Hi Sven! I'm finally using the new STON with comments and I found one little problem: my ston file has urls :/ { // comment ok, skipped /* comment ok, skipped */ "aURL": "http://ouch.it.will.be.skipped/and/break?my=ston" } That is, doing: STON fromStreamWithComments: '{ // comment ok, skipped /* comment ok, skipped */ "aURL": "http://ouch.it.will.be.skipped/and/break?my=ston" }' readStream Will fail with an invalid input error because after skipping comments my STON stream remains like STON fromStreamWithComments: '{ "aURL": "http: } Any thoughts? Guille El dom., 14 de jun. de 2015 a la(s) 3:06 a. m., stepharo <stepharo(a)free.fr> escribiÃ³: > > > Le 10/6/15 20:39, Sven Van Caekenberghe a Ã©crit : > >> On 10 Jun 2015, at 15:13, stepharo <stepharo(a)free.fr> wrote: > >> > >> This is in the entreprise book. I added a subsection on comments. > >> May be I committed but forgot to push. > >> Now I did it. > > And now I see it. > > I made a small change. > super! > > > > >> Stef > >> > >> > >> Le 9/6/15 21:50, Sven Van Caekenberghe a Ã©crit : > >>> Where exactly can I find what you changed ? > >>> I don't see any commit of you on github ... > >>> > >>>> On 09 Jun 2015, at 20:10, stepharo <stepharo(a)free.fr> wrote: > >>>> > >>>> > >>>> > >>>> Le 9/6/15 20:06, stepharo a Ã©crit : > >>>>> I will update the STON chapter :) > >>>> Done :) > >>>> > >>>>> Le 8/6/15 23:39, Sven Van Caekenberghe a Ã©crit : > >>>>>> I would do it like this: use a wrapping stream to skip comments. > >>>>>> > >>>>>> === > >>>>>> Name: STON-Core-SvenVanCaekenberghe.59 > >>>>>> Author: SvenVanCaekenberghe > >>>>>> Time: 8 June 2015, 11:34:32.886167 pm > >>>>>> UUID: 7ab942b5-9d1a-44c3-9d58-da29feb72380 > >>>>>> Ancestors: STON-Core-SvenVanCaekenberghe.58 > >>>>>> > >>>>>> Add experimental STONCStyleCommentsSkipStream and tests > >>>>>> === > >>>>>> Name: STON-Tests-SvenVanCaekenberghe.54 > >>>>>> Author: SvenVanCaekenberghe > >>>>>> Time: 8 June 2015, 11:34:52.918771 pm > >>>>>> UUID: 3dcb2c9c-6efe-4d62-bf36-3bbe91f09d08 > >>>>>> Ancestors: STON-Tests-SvenVanCaekenberghe.53 > >>>>>> > >>>>>> Add experimental STONCStyleCommentsSkipStream and tests > >>>>>> === > >>>>>> > >>>>>> Then you can do as follows: > >>>>>> > >>>>>> STON fromStream: (STONCStyleCommentsSkipStream on: 'Point[/* this > is X*/ 1, /* this is Y*/ 2] // Nice huh ?' readStream). > >>>>>> > >>>>>> => 1@2 > >>>>>> > >>>>>> STON fromStream: (STONCStyleCommentsSkipStream on: '// Here is how > you create a point: > >>>>>> Point[ > >>>>>> // this is X > >>>>>> 1, > >>>>>> // this is Y > >>>>>> 2 ] > >>>>>> // Nice huh ?' readStream). > >>>>>> > >>>>>> => 1@2 > >>>>>> > >>>>>> This helper class would be useable in other contexts too (like > NeoJSON). > >>>>>> > >>>>>> The advantage is that it does not change the STON (or JSON) syntax > itself, it just add some functionality for those who want it. > >>>>>> > >>>>>> What do you think ? > >>>>>> > >>>>>> Sven > >>>>>> > >>>>>> PS: this probably needs some more testing > >>>>>> > >>>>>>> On 08 Jun 2015, at 17:16, Sven Van Caekenberghe <sven(a)stfx.eu> > wrote: > >>>>>>> > >>>>>>>> On 08 Jun 2015, at 17:03, Damien Cassou <damien.cassou(a)inria.fr> > wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Sven Van Caekenberghe <sven(a)stfx.eu> writes: > >>>>>>>> > >>>>>>>>> I am not so sure we should add that. > >>>>>>>>> > >>>>>>>>> The JSON spec explicitly does not allow comments because of fear > of abuse (that the comments would be used to add semantic meaning outside > the spec). > >>>>>>>> really? That's surprising. Comments in configuration files are > very > >>>>>>>> important. Even more when the configuration files are templates > for new > >>>>>>>> users. > >>>>>>> I know some people are using STON in ways that I did not > originally anticipate, that is of course cool, but writing configuration > files was not the original goal. STON is a version of JSON that is closer > to our object model. > >>>>>>> > >>>>>>> Like I said, I will think about it. > >>>>>>> > >>>>>>>> -- > >>>>>>>> Damien Cassou > >>>>>>>> http://damiencassou.seasidehosting.st > >>>>>>>> > >>>>>>>> "Success is the ability to go from one failure to another without > >>>>>>>> losing enthusiasm." --Winston Churchill > >>>>> > >>> > >> > > > > > > >

July 16, 2015

Re: [Pharo-dev] How to combine event listeners in Bloc?

by Stephan Eggermont

Hmm, this is not too bad. But it still is dependent on the layout. BlCardLayout>>dropAccepted: anEvent |wellPanelFlow droppedMorph localPosition | wellPanelFlow := anEvent handler. droppedMorph := anEvent contents asDragWell. localPosition := wellPanelFlow globalPointToLocal: anEvent position. wellPanelFlow hasSubmorphs ifFalse: [ wellPanelFlow addMorph: droppedMorph ]. wellPanelFlow addMorph: droppedMorph asElementNumber: (self indexForInserting: localPosition inList: wellPanelFlow submorphs) BlCardLayout>>indexForInserting: aPoint inList: morphList | cmp1 cmp2 cmp3 | cmp1 := [:rect | aPoint x < rect right]. cmp2 := [:rect | aPoint y < rect bottom]. cmp3 := [:rect | aPoint y < rect top]. morphList keysAndValuesDo: [:index :m | | box | box := m fullBounds translateBy: m position. "Check for inserting before current row" (cmp3 value: box) ifTrue: [^index]. "Check for inserting before current cell" ((cmp1 value: box) and: [cmp2 value: box]) ifTrue: [^index]]. ^morphList size + 1

July 16, 2015

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/50174 Home: https://github.com/pharo-project/pharo-core

July 16, 2015

[pharo-project/pharo-core] 0b9a9b: 50174

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 0b9a9ba2b14eb3fc549d4a666a7138feed9e088b https://github.com/pharo-project/pharo-core/commit/0b9a9ba2b14eb3fc549d4a66… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-07-16 (Thu, 16 Jul 2015) Changed paths: M Collections-Sequenceable.package/SparseLargeTable.class/instance/accessing/zapDefaultOnlyEntries.st R Collections-Sequenceable.package/SparseLargeTable.class/instance/private/findLastNonNilSubTable.st M Monticello.package/MCOrganizationDefinition.class/instance/accessing/description.st A Monticello.package/MCOrganizationDefinition.class/instance/private/fullDescription.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50173.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50174.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50173.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50174.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50174 14616 Package Tags causing merge conflict https://pharo.fogbugz.com/f/cases/14616 15987 SparseLargeTable>>zapDefaultOnlyEntries creates an unused copy https://pharo.fogbugz.com/f/cases/15987 http://files.pharo.org/image/50/50174.zip

July 16, 2015

Re: [Pharo-dev] How to combine event listeners in Bloc?

by Stephan Eggermont

On 16-07-15 11:41, Alain Plantec via Pharo-dev wrote: >i think it is better as specific feature of the pane event listener It feels a bit like feature envy. Too much poking around. BlWellPanelFlowEventListener>>PaneldropAccepted: anEvent |wellPanelFlow droppedMorph localPosition found| wellPanelFlow := anEvent handler. droppedMorph := anEvent contents asDragWell. localPosition := wellPanelFlow globalPointToLocal: anEvent position. wellPanelFlow hasSubmorphs ifFalse: [ wellPanelFlow addMorph: droppedMorph ]. found := wellPanelFlow morphsAt: localPosition. found ifNotNil: [ "We found at least one. The deepest is first, that's the one we want" found := found first. found bounds center x < localPosition x ifTrue: [ "to the right of its center, so after it?" wellPanelFlow addMorph: droppedMorph after: found ] ifFalse: [ wellPanelFlow addMorph: droppedMorph inFrontOf: found ] ] ifNil: [ found := self precedingMorphIn: wellPanelFlow submorphs at: localPosition. found ifNil: [ wellPanelFlow addMorphFront: droppedMorph ] ifNotNil: [ wellPanelFlow addMorph: droppedMorph after: found ] ] BlWellPanelFlowEventListener>>precedingMorphIn: aCollection at: localPosition |precedingMorph| precedingMorph := nil. aCollection do: [ :aMorph | aMorph bounds bottom < localPosition y ifTrue: [ "wrong row" precedingMorph := aMorph ] ifFalse: [ aMorph bounds right < localPosition x ifTrue: [ precedingMorph := aMorph] ifFalse: [ ^aMorph ]] ]. ^precedingMorph

July 16, 2015

libSDL2 not present in full download

by jannik laval

Hi pharoers, I was trying to use OSWindow, but in a "full Download" (Image+VM), the plugin SDL2 is not present. Does this mean that the full download is based on an old VM ? It is the case for Pharo 4 and Pharo 5. Cheers, -- ~~Jannik Laval~~ Ãcole des Mines de Douai Enseignant-chercheur http://www.jannik-laval.eu http://www.phratch.com http://www.approchealpes.info http://car.mines-douai.fr/

July 16, 2015

Re: [Pharo-dev] Pharo 5: browser windows are extremely resource hungry

by Henrik Johansen

> On 16 Jul 2015, at 9:36 , Esteban Lorenzano <estebanlm(a)gmail.com> wrote: > > >> On 16 Jul 2015, at 09:10, Max Leske <maxleske(a)gmail.com> wrote: >> >> Apparently this is a polling bug in the new FastTable. FastTable should not have been activated yet Esteban told me and in the newest build indeed FastTable has been deactivated again and the behaviour I described is no longer reproducible. > > this :) > > Esteban > >> >> Max >> >>> Hi >>> >>> I discovered that in Pharo 5, having two or more browser windows open significantly reduces responsiveness in the image (delayed clicks, ghosting when moving windows etc.). This is especially true if a method has been selected in two or more windows. The ghosting becomes much worse when you drag a window. I checked that the same does not happen in Pharo 4. >>> >>> I originally thought that there was a strong impact on CPU load too, but itâs actually not that bad. >>> >>> Has anybody seen this or even worked on a fix for it? If not, Iâll open a bug report. >>> >>> >>> Cheers, >>> Max >> > > Should've named it FastTableExceptWhenSlow until it's out of beta ;) Cheers, Henry

July 16, 2015