- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Pharo-dev Digest, Vol 88, Issue 7
by Richard Uttner Aug. 26, 2020

Aug. 26, 2020

Hi Esteban and Martin, sorry for my contribution to this topic being so late. I fully agree to Martin's suggestion of checking if an inspected object conforms to the rich inspector api before applying it. For dealing with Glorp, I found an easy solution which Pavel will integrate soon in the Glorp port he published some time ago. My approach to find the trigger(s) resolving the proxy was simply hooking #doesNotUnderstand: with a Transcript trace of the current message. So I found out that it were only 3, which I list below along with my implementation in Proxy that stopped any resolving: gtDisplayString Â Â Â self isInstantiated Â Â Â Â Â Â ifTrue: [ ^self class name , ' on ', value gtDisplayString]. Â Â Â ^self class name, ' uninstantiated on ', self query resultClass name gtInspectorVariableNodesIn: aCollection Â Â "Taken from Object, avoiding loop over indexed variables calling #basicSize" Â Â aCollection Â Â Â Â Â Â addAll: ((self class allSlots collect: [ :slot | Â Â Â Â Â Â Â Â Â GTInspectorSlotNode hostObject: self slot: slot ])) readSlot: aSlot Â Â Â "Taken from Object, sent by GTInspectorSlotNode" Â Â Â ^aSlot read: self With this approach, I was not able to trigger proxy resolution by just navigating in the inspector, I had to explicitely send #getValue to the slot. Hope this helps until we have a more general solution! Best regards, Richard Am 08.08.2020 um 18:00 schrieb pharo-dev-request(a)lists.pharo.org: > Message: 1 > Date: Fri, 7 Aug 2020 20:33:50 -0300 > From: Esteban Maringolo<emaringolo(a)gmail.com> > To: Pharo Developers<pharo-dev(a)lists.pharo.org> > Subject: [Pharo-dev] Tools with Proxy support > Message-ID: > <CAJMgPC+5v4eG+qK3m7LjWmYs3H1J=mWopg_acUU9VcCsY5YJ5g(a)mail.gmail.com> > Content-Type: text/plain; charset="UTF-8" > > Hi, > > I'm having a hard time trying to debug some Glorp code in Pharo 8 > because the inspectors and the glamour things are triggering the > instantiation of my Proxy objects. > > I don't know how the inspectors, variable lists, and other tool work, > but I assume there should be a way to reimplement something in Proxy > to avoid causing a MessageNotUnderstood that triggers the > instantiation. > > If there is no way for it (I guess there is, looking at some gt* > selectors in ProtoObject), please consider having a debugPrintString > (and debugPrintOn:) in the upcoming Spec2 based tools.:-) > > Best regards, > > Esteban A. Maringolo > > > > ------------------------------ > > Message: 2 > Date: Fri, 7 Aug 2020 16:59:10 -0700 > From: Martin McClure<martin(a)hand2mouse.com> > To: Pharo Development List<pharo-dev(a)lists.pharo.org>, Esteban > Maringolo<emaringolo(a)gmail.com> > Subject: Re: [Pharo-dev] Tools with Proxy support > Message-ID:<49dde322-a79d-1b16-798e-9679ce08b1bf(a)hand2mouse.com> > Content-Type: text/plain; charset=utf-8; format=flowed > > Hi Esteban, > > I also have trouble with this sort of thing in Pharo -- inspectors > sending messages to objects under inspection that are not understood, or > that do not respond in the way the inspector expects. > > It is possible to build a basic inspector that sends no messages at all > to the inspected object (using the mirror methods), and it would be nice > to have that option in Pharo. > > It's also possible to have a "careful" inspector that uses a mirror > method to determine the class of the inspected object, then uses either > the purely-mirroring inspector or the richer inspector, depending on > whether the object's class inherits from Object. Since the GemStone > clients make use of proxy objects, GemStone's inspectors take this > general approach. They're not perfect either, but problems are > considerably fewer. > > Regards, > -Martin

3 2

[Pharo 9.0] Build #616: Variable-writeInContext-should-return-Value
by ci-pharo-ci-jenkins2＠inria.fr Aug. 24, 2020

Aug. 24, 2020

There is a new Pharo build available! The status of the build #616 was: FAILURE. The Pull Request #7174 was integrated: "Variable-writeInContext-should-return-Value" Pull request url: https://github.com/pharo-project/pharo/pull/7174 Issue Url: https://github.com/pharo-project/pharo/issues/Variable Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #615: Guard-ReservedVariable-name-setter
by ci-pharo-ci-jenkins2＠inria.fr Aug. 24, 2020

Aug. 24, 2020

There is a new Pharo build available! The status of the build #615 was: FAILURE. The Pull Request #7173 was integrated: "Guard-ReservedVariable-name-setter" Pull request url: https://github.com/pharo-project/pharo/pull/7173 Issue Url: https://github.com/pharo-project/pharo/issues/Guard Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #614: refersToLiteral-DoNotLookIntoProperties
by ci-pharo-ci-jenkins2＠inria.fr Aug. 24, 2020

Aug. 24, 2020

There is a new Pharo build available! The status of the build #614 was: FAILURE. The Pull Request #6991 was integrated: "refersToLiteral-DoNotLookIntoProperties" Pull request url: https://github.com/pharo-project/pharo/pull/6991 Issue Url: https://github.com/pharo-project/pharo/issues/refersToLiteral Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #613: 7126-Remove-duplication-between-IconStyler-and-IconStylerRT
by ci-pharo-ci-jenkins2＠inria.fr Aug. 23, 2020

Aug. 23, 2020

There is a new Pharo build available! The status of the build #613 was: FAILURE. The Pull Request #7169 was integrated: "7126-Remove-duplication-between-IconStyler-and-IconStylerRT" Pull request url: https://github.com/pharo-project/pharo/pull/7169 Issue Url: https://github.com/pharo-project/pharo/issues/7126 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #612: 7166-Calypso-Class-definition-mode-is-wrong-should-not-be-a-comment-mode
by ci-pharo-ci-jenkins2＠inria.fr Aug. 23, 2020

Aug. 23, 2020

There is a new Pharo build available! The status of the build #612 was: FAILURE. The Pull Request #7167 was integrated: "7166-Calypso-Class-definition-mode-is-wrong-should-not-be-a-comment-mode" Pull request url: https://github.com/pharo-project/pharo/pull/7167 Issue Url: https://github.com/pharo-project/pharo/issues/7166 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #611: 7128-Selecting-packages-in-Finder-brings-an-error
by ci-pharo-ci-jenkins2＠inria.fr Aug. 22, 2020

Aug. 22, 2020

There is a new Pharo build available! The status of the build #611 was: SUCCESS. The Pull Request #7164 was integrated: "7128-Selecting-packages-in-Finder-brings-an-error" Pull request url: https://github.com/pharo-project/pharo/pull/7164 Issue Url: https://github.com/pharo-project/pharo/issues/7128 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #610: Rename RubWorkspaceBar into RubWorkspaceBarRuler
by ci-pharo-ci-jenkins2＠inria.fr Aug. 22, 2020

Aug. 22, 2020

There is a new Pharo build available! The status of the build #610 was: SUCCESS. The Pull Request #7162 was integrated: "Rename RubWorkspaceBar into RubWorkspaceBarRuler" Pull request url: https://github.com/pharo-project/pharo/pull/7162 Issue Url: https://github.com/pharo-project/pharo/issues/7111 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #609: 7157-RBParser-parseMethod-raises-an-error-RBParseErrorNode-DNU-body
by ci-pharo-ci-jenkins2＠inria.fr Aug. 21, 2020

Aug. 21, 2020

There is a new Pharo build available! The status of the build #609 was: FAILURE. The Pull Request #7161 was integrated: "7157-RBParser-parseMethod-raises-an-error-RBParseErrorNode-DNU-body" Pull request url: https://github.com/pharo-project/pharo/pull/7161 Issue Url: https://github.com/pharo-project/pharo/issues/7157 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 9.0] Build #608: Merge NewTools-0.4.2.5
by ci-pharo-ci-jenkins2＠inria.fr Aug. 21, 2020

Aug. 21, 2020

There is a new Pharo build available! The status of the build #608 was: FAILURE. The Pull Request #7160 was integrated: "Merge NewTools-0.4.2.5" Pull request url: https://github.com/pharo-project/pharo/pull/7160 Issue Url: https://github.com/pharo-project/pharo/issues/NewTools Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0