- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Mysteries of the compiler

by Marcus Denker

--Â Marcus Denker Sent with Airmail On 17 Oct 2014 at 12:46:20, Nicolai Hess (nicolaihess(a)web.de) wrote: I ran some testcases with your changes and I expect some test are failing now, but actually they don't fail but others do. Yes, but I think these tests are wrong.Â (needs to be carefully checked). 2014-10-16 22:06 GMT+02:00 Nicolai Hess <nicolaihess(a)web.de>: 2014-10-16 17:40 GMT+02:00 Marcus Denker <marcus.denker(a)inria.fr>: >> >> While in the write we need to mark: >> >> lookupVariableForWrite: aVariableNode >> >>Â Â Â | var | >> >>Â Â Â var := scope lookupVar: aVariableNode name. >> >>Â Â Â var ifNil: [^var]. >>Â Â Â var isSpecialVariable ifTrue: [ self storeIntoSpecialVariable: aVariableNode ]. >>Â Â Â var isWritable ifFalse: [ self storeIntoReadOnlyVariable: aVariableNode ]. >> >>Â Â Â var isTemp ifTrue: [ >>Â Â Â Â Â Â Â "when in a loop, all writes escape" >>Â Â Â Â Â Â Â scope outerScope isInsideOptimizedLoop ifTrue: [ var markEscapingWrite ]. >>Â Â Â Â Â Â Â "else only escaping when they will end up in different closures" >>Â Â Â Â Â Â Â (var scope outerNotOptimizedScope ~= scope outerNotOptimizedScope) >>Â Â Â Â Â Â Â Â Â Â Â ifTrue: [ var markEscapingWrite]]. >>Â Â Â ^var >> >> >> I checked that all Opal tests that are failing are actually testing wrong, and I double checked >> that all those methods are now compiled like with the old old compilerâ¦ >> Now recompilng the image. > > Hmmâ¦ nopeâ¦. something wrong. > the âouterScopeâ was wrong. So without that, I can recompile the image and your example is compiled without temp vectorsâ¦ Â Â Â Â Marcus good! I'll open an issue on fogbugz and create a slice with your changes. about reordering of tempvars: It would be nice if the order preserves, but I think we can live with it. Although there is an issue with the debugger: 14058 Inconsistent information in debugger I solved that one by using tempvar names instead if the index, but this has another issue (sometimes the last (or most nested) tempvar is always nil). The real error is, EyeDebuggerContextInspector does recognizes if the index changes (somewhere in EyeDebuggerContextInspector>>#updateList it compares the new elements (with new indices) with the old elements list and skips the update)

Oct. 17, 2014

Re: [Pharo-dev] [Pharo-users] [Ann] Phratch 4.0

by Sven Van Caekenberghe

Excellent work. Thank you. On 17 Oct 2014, at 11:48, jannik laval <jannik.laval(a)gmail.com> wrote: > Phratch 4.0 is out of the box ! > > Phratch 4.0 is cleaner, faster and more stable than phratch 3.0. > > After the version 3.0 that was a release for usability and stability, phratch 4.0 is released to prepare the future. > > The new features are: > > - phratch is more modular, eg having a kernel and lots of addons. > - cleaning a lot the source code > - customizable environment > - each block can be made visible or invisible > - each category can be made visible or invisible > - it is possible to add translation for addons with the pragma <phratchTranslation: âfrâ> in the class PhratchTranslator. This make the translation more modular. > - first integration of phratch with the pharo environment. Using the pragma at the class side of any class in Pharo make the class visible and usable in phratch. > - There is a lot of things to do with phratch, you are welcome to contribute ! The documentation for new features will arrive soon. > > See you on phratch.com > > -- > ~~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/ >

Oct. 17, 2014

Re: [Pharo-dev] [pharo-project/pharo-core] cd6a30: 40310

by Esteban Lorenzano

that i donât know. the change is already there or the slice is actually empty. nothing to do with the app, in that case :) Esteban > On 17 Oct 2014, at 12:28, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > >> On 17 Oct 2014, at 12:21, Esteban Lorenzano <estebanlm(a)gmail.com <mailto:estebanlm@gmail.com>> wrote: >> >> that means most probably a merge conflict (still not handled by new integrator app). >> Sorry, I will fix it soon :) >> > > But I wonder why when I merge the not-integrated slices, it claims that there is no change? > > >> Esteban >> >>> On 17 Oct 2014, at 12:19, Marcus Denker <marcus.denker(a)inria.fr <mailto:marcus.denker@inria.fr>> wrote: >>> >>> We will have to revert this. >>> >>> -> the changes are not there >>> -> yet when I merge the Slice it claims there are no changes. >>> -> So we have to revert. >>> >>> On Fri, Oct 17, 2014 at 11:56 AM, GitHub <noreply(a)github.com <mailto:noreply@github.com>> wrote: >>> Branch: refs/heads/4.0 >>> Home: https://github.com/pharo-project/pharo-core <https://github.com/pharo-project/pharo-core> >>> Commit: cd6a3036bde6d73d3e2502aca8caacaac75dc1db >>> https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac… <https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac…> >>> Author: Jenkins Build Server <board(a)pharo-project.org <mailto:board@pharo-project.org>> >>> Date: 2014-10-17 (Fri, 17 Oct 2014) >>> >>> Changed paths: >>> A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script310.st <http://script310.st/> >>> A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40310.st <http://update40310.st/> >>> M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st >>> >>> Log Message: >>> ----------- >>> 40310 >>> 14246 CompiledMethod>>hash can produce clashes >>> https://pharo.fogbugz.com/f/cases/14246 <https://pharo.fogbugz.com/f/cases/14246> >>> >>> 14257 Replace Announcer>>#on:send:to:s senders in NativeBoost-Core >>> https://pharo.fogbugz.com/f/cases/14257 <https://pharo.fogbugz.com/f/cases/14257> >>> >>> 14253 MC dependency warning should name which package is failing to load >>> https://pharo.fogbugz.com/f/cases/14253 <https://pharo.fogbugz.com/f/cases/14253> >>> >>> http://files.pharo.org/image/40/40310.zip <http://files.pharo.org/image/40/40310.zip> >>> >>> >>> >>> >>> >>> -- >>> -- >>> Marcus Denker -- denker(a)acm.org <mailto:denker@acm.org> >>> http://www.marcusdenker.de <http://www.marcusdenker.de/> >> >

Oct. 17, 2014

Re: [Pharo-dev] Mysteries of the compiler

by Nicolai Hess

I ran some testcases with your changes and I expect some test are failing now, but actually they don't fail but others do. 2014-10-16 22:06 GMT+02:00 Nicolai Hess <nicolaihess(a)web.de>: > 2014-10-16 17:40 GMT+02:00 Marcus Denker <marcus.denker(a)inria.fr>: > >> >> >> >> While in the write we need to mark: >> >> >> >> lookupVariableForWrite: aVariableNode >> >> >> >> | var | >> >> >> >> var := scope lookupVar: aVariableNode name. >> >> >> >> var ifNil: [^var]. >> >> var isSpecialVariable ifTrue: [ self storeIntoSpecialVariable: >> aVariableNode ]. >> >> var isWritable ifFalse: [ self storeIntoReadOnlyVariable: >> aVariableNode ]. >> >> >> >> var isTemp ifTrue: [ >> >> "when in a loop, all writes escape" >> >> scope outerScope isInsideOptimizedLoop ifTrue: [ var >> markEscapingWrite ]. >> >> "else only escaping when they will end up in different >> closures" >> >> (var scope outerNotOptimizedScope ~= scope >> outerNotOptimizedScope) >> >> ifTrue: [ var markEscapingWrite]]. >> >> ^var >> >> >> >> >> >> I checked that all Opal tests that are failing are actually testing >> wrong, and I double checked >> >> that all those methods are now compiled like with the old old compilerâ¦ >> >> Now recompilng the image. >> > >> > Hmmâ¦ nopeâ¦. something wrong. >> > >> >> the âouterScopeâ was wrong. >> >> So without that, I can recompile the image and your example is compiled >> without temp vectorsâ¦ >> >> Marcus >> > > good! > I'll open an issue on fogbugz and create a slice with your changes. > > about reordering of tempvars: > It would be nice if the order preserves, but I think we can live with it. > Although there is an issue with the debugger: > 14058 <https://pharo.fogbugz.com/default.asp?14058> Inconsistent > information in debugger > I solved that one by using tempvar names instead if the index, but > this has another issue (sometimes the last (or most nested) tempvar is > always nil). > > The real error is, EyeDebuggerContextInspector does recognizes if the > index changes > (somewhere in EyeDebuggerContextInspector>>#updateList it > compares the new elements (with new indices) with the old elements list > and skips the update) > > > > >

Oct. 17, 2014

Re: [Pharo-dev] Context variables?

by Frank Shearar

Bear in mind that if you do any fancy stack slicing, you will get... *unexpected* behaviour from DynamicVariable. (See http://www.lshift.net/blog/2012/06/27/resumable-exceptions-can-macro-expres… and follow-on links.) frank On 15 October 2014 16:34, Camille Teruel <camille.teruel(a)gmail.com> wrote: > Hi Jeff, > > I think you should look at DynamicVariable and ProcessSpecificVariable > classes. The first one is a read-only while the second is writable. > You have to subclass these class for each variable you want. > > Ex: > DynamicVariable subclass: #MyVar. > MyVar value: 4 during: [ MyVar value ] > > HTH, > Camiile > > > On 15 oct. 2014, at 17:09, J.F. Rick <self(a)je77.com> wrote: > > I remember there was some discussion on the list about support for variables > tied to the context rather than to the instance or class. This seems > particularly useful for a web application where you might want to access the > request and response from the context. What is the proper name for these > kind of variables and where can I read about how to use them? > > Cheers, > > Jeff > > -- > Jochen "Jeff" Rick, Ph.D. > http://www.je77.com/ > Skype ID: jochenrick > >

Oct. 17, 2014

Re: [Pharo-dev] [pharo-project/pharo-core] cd6a30: 40310

by Marcus Denker

> On 17 Oct 2014, at 12:21, Esteban Lorenzano <estebanlm(a)gmail.com> wrote: > > that means most probably a merge conflict (still not handled by new integrator app). > Sorry, I will fix it soon :) > But I wonder why when I merge the not-integrated slices, it claims that there is no change? > Esteban > >> On 17 Oct 2014, at 12:19, Marcus Denker <marcus.denker(a)inria.fr <mailto:marcus.denker@inria.fr>> wrote: >> >> We will have to revert this. >> >> -> the changes are not there >> -> yet when I merge the Slice it claims there are no changes. >> -> So we have to revert. >> >> On Fri, Oct 17, 2014 at 11:56 AM, GitHub <noreply(a)github.com <mailto:noreply@github.com>> wrote: >> Branch: refs/heads/4.0 >> Home: https://github.com/pharo-project/pharo-core <https://github.com/pharo-project/pharo-core> >> Commit: cd6a3036bde6d73d3e2502aca8caacaac75dc1db >> https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac… <https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac…> >> Author: Jenkins Build Server <board(a)pharo-project.org <mailto:board@pharo-project.org>> >> Date: 2014-10-17 (Fri, 17 Oct 2014) >> >> Changed paths: >> A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script310.st <http://script310.st/> >> A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40310.st <http://update40310.st/> >> M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st >> >> Log Message: >> ----------- >> 40310 >> 14246 CompiledMethod>>hash can produce clashes >> https://pharo.fogbugz.com/f/cases/14246 <https://pharo.fogbugz.com/f/cases/14246> >> >> 14257 Replace Announcer>>#on:send:to:s senders in NativeBoost-Core >> https://pharo.fogbugz.com/f/cases/14257 <https://pharo.fogbugz.com/f/cases/14257> >> >> 14253 MC dependency warning should name which package is failing to load >> https://pharo.fogbugz.com/f/cases/14253 <https://pharo.fogbugz.com/f/cases/14253> >> >> http://files.pharo.org/image/40/40310.zip <http://files.pharo.org/image/40/40310.zip> >> >> >> >> >> >> -- >> -- >> Marcus Denker -- denker(a)acm.org <mailto:denker@acm.org> >> http://www.marcusdenker.de <http://www.marcusdenker.de/> >

Oct. 17, 2014

Re: [Pharo-dev] [pharo-project/pharo-core] cd6a30: 40310

by Esteban Lorenzano

that means most probably a merge conflict (still not handled by new integrator app). Sorry, I will fix it soon :) Esteban > On 17 Oct 2014, at 12:19, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > We will have to revert this. > > -> the changes are not there > -> yet when I merge the Slice it claims there are no changes. > -> So we have to revert. > > On Fri, Oct 17, 2014 at 11:56 AM, GitHub <noreply(a)github.com <mailto:noreply@github.com>> wrote: > Branch: refs/heads/4.0 > Home: https://github.com/pharo-project/pharo-core <https://github.com/pharo-project/pharo-core> > Commit: cd6a3036bde6d73d3e2502aca8caacaac75dc1db > https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac… <https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac…> > Author: Jenkins Build Server <board(a)pharo-project.org <mailto:board@pharo-project.org>> > Date: 2014-10-17 (Fri, 17 Oct 2014) > > Changed paths: > A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script310.st <http://script310.st/> > A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40310.st <http://update40310.st/> > M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st > > Log Message: > ----------- > 40310 > 14246 CompiledMethod>>hash can produce clashes > https://pharo.fogbugz.com/f/cases/14246 <https://pharo.fogbugz.com/f/cases/14246> > > 14257 Replace Announcer>>#on:send:to:s senders in NativeBoost-Core > https://pharo.fogbugz.com/f/cases/14257 <https://pharo.fogbugz.com/f/cases/14257> > > 14253 MC dependency warning should name which package is failing to load > https://pharo.fogbugz.com/f/cases/14253 <https://pharo.fogbugz.com/f/cases/14253> > > http://files.pharo.org/image/40/40310.zip <http://files.pharo.org/image/40/40310.zip> > > > > > > -- > -- > Marcus Denker -- denker(a)acm.org <mailto:denker@acm.org> > http://www.marcusdenker.de <http://www.marcusdenker.de/>

Oct. 17, 2014

Re: [Pharo-dev] [pharo-project/pharo-core] cd6a30: 40310

by Marcus Denker

We will have to revert this. -> the changes are not there -> yet when I merge the Slice it claims there are no changes. -> So we have to revert. On Fri, Oct 17, 2014 at 11:56 AM, GitHub <noreply(a)github.com> wrote: > Branch: refs/heads/4.0 > Home: https://github.com/pharo-project/pharo-core > Commit: cd6a3036bde6d73d3e2502aca8caacaac75dc1db > > https://github.com/pharo-project/pharo-core/commit/cd6a3036bde6d73d3e2502ac… > Author: Jenkins Build Server <board(a)pharo-project.org> > Date: 2014-10-17 (Fri, 17 Oct 2014) > > Changed paths: > A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/ > script310.st > A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/ > update40310.st > M > ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st > > Log Message: > ----------- > 40310 > 14246 CompiledMethod>>hash can produce clashes > https://pharo.fogbugz.com/f/cases/14246 > > 14257 Replace Announcer>>#on:send:to:s senders in NativeBoost-Core > https://pharo.fogbugz.com/f/cases/14257 > > 14253 MC dependency warning should name which package is failing to load > https://pharo.fogbugz.com/f/cases/14253 > > http://files.pharo.org/image/40/40310.zip > > > -- -- Marcus Denker -- denker(a)acm.org http://www.marcusdenker.de

Oct. 17, 2014

Re: [Pharo-dev] CompiledMethod>>hash can produce clashes

by Marcus Denker

--Â Marcus Denker Sent with Airmail On 17 Oct 2014 at 11:45:23, Max Leske (maxleske(a)gmail.com) wrote: > On 17.10.2014, at 11:39, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > > > On 17 Oct 2014, at 11:12, Tudor Girba <tudor(a)tudorgirba.com> wrote: > >> Exactly. So, the problem with Set is not in hash at all, but in equality. Of course, we can still enhance hash, but we should first focus on equality. >> >> And I am also of the opinion that equality should take the name of the selector and even the name of the class into account. > > From a modelling standpoint it sounds as if one object (CompiledMethod) is used for two different things which results in the conflicting ideas about implementing equality and hashing. A CompiledMethod should hold a CompiledCode object while adding the selector and class. The CompiledCode object could then be equivalent or even be optionally shared among similar methods (like all those implementing ^self). > > Just an external observation / idea. Yes, pretty much what I was thinking. I donât know what the consequences would be for the VM though... Yes, I think, too, that many of the problems with CompiledMethod come from the fact that we âabuseâ a low level object in a context where people want a higher level concept. Marcus

Oct. 17, 2014

Re: [Pharo-dev] [Pharo-users] [Ann] Phratch 4.0

by Tudor Girba

This is a great addition to the Pharo ecosystem. Thank you very much! Doru On Fri, Oct 17, 2014 at 11:48 AM, jannik laval <jannik.laval(a)gmail.com> wrote: > Phratch 4.0 is out of the box ! > > Phratch 4.0 is cleaner, faster and more stable than phratch 3.0. > > After the version 3.0 that was a release for usability and stability, > phratch 4.0 is released to prepare the future. > > The new features are: > > - phratch is more modular, eg having a kernel and lots of addons. > - cleaning a lot the source code > - customizable environment > - each block can be made visible or invisible > - each category can be made visible or invisible > - it is possible to add translation for addons with the pragma > <phratchTranslation: âfrâ> in the class PhratchTranslator. This make the > translation more modular. > - first integration of phratch with the pharo environment. Using the > pragma at the class side of any class in Pharo make the class visible and > usable in phratch. > - There is a lot of things to do with phratch, you are welcome to > contribute ! The documentation for new features will arrive soon. > > See you on phratch.com > > -- > > ~~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/ > -- www.tudorgirba.com "Every thing has its own flow"

Oct. 17, 2014