- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Spotter changes broke CodePanel

by Stephan Eggermont

On 24-02-16 19:37, Tudor Girba wrote: > Hi, > > Could you provide a fully loadable sample to look at this? Just load NewUI from the configuration browser. Stephan

Feb. 24, 2016

Re: [Pharo-dev] Spotter changes broke CodePanel

by Tudor Girba

Hi, Could you provide a fully loadable sample to look at this? Cheers, Doru > On Feb 24, 2016, at 11:52 AM, Stephan Eggermont <stephan(a)stack.nl> wrote: > > I used to do > > CodePanel class>>spot > GTSpotterMorph new > extent: (World width / 3.25 @ (World height / 2.25)) asIntegerPoint; > spotterModel: ((GTSpotter new) > processorsFilter: (CodePanelProcessorFilter new > configuredWith: [ :aProcessor | aProcessor > actLogic: [ :aMethod :step | > step exit. > aMethod ifNil: [ self new openInWindowLabeled: 'Code panel' ] > ifNotNil: [ |panel| > panel := self new. > panel openInWindowLabeled: 'Code panel'. > panel newCardFor: aMethod.]]; > candidatesLimit: 50 ])); > openCenteredInWorld > > That now (since at least 50596) an interesting Brick error. > Do I need to change my code? > > Stephan > > > <PharoDebug.log> -- www.tudorgirba.com www.feenk.com "Being happy is a matter of choice."

Feb. 24, 2016

Re: [Pharo-dev] [Pharo-users] help needed for testing the order in spotter

by Tudor Girba

Hi, Thanks. Indeed, I agree with you. In fact, this is what we did, and then I was told that people would rather want to discuss beforehand :). So, actually, this was a little experiment. The thing is that we do not want to disturb people, but we need the feedback to improve fast. So, right now it looks like the only real way to get feedback is to push the changes. This has the tendency to create some annoyance, but we are trying to react as fast as we can to limit it. I hope this will be Ok with everyone. Cheers, Doru > On Feb 24, 2016, at 7:29 PM, Christophe Demarey <christophe.demarey(a)inria.fr> wrote: > > Hi Doru, > > I don't see a better way to have beta-testers than having your code on the pharo alpha image ;) > You will have very soon feedback and then you can iterate. > > Cheers, > Christophe > -- www.tudorgirba.com www.feenk.com "Speaking louder won't make the point worthier."

Feb. 24, 2016

ReÂ : Re: [Pharo-users] help needed for testing the order in spotter

by Christophe Demarey

Hi Doru, I don't see a better way to have beta-testers than having your code on the pharo alpha image ;) You will have very soon feedback and then you can iterate. Cheers, Christophe

Feb. 24, 2016

[pharo-project/pharo-core]

by GitHub

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

Feb. 24, 2016

[pharo-project/pharo-core] 4a36b4: 50610

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 4a36b431071a12a261972903db068263009cf5a9 https://github.com/pharo-project/pharo-core/commit/4a36b431071a12a261972903… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-02-24 (Wed, 24 Feb 2016) Changed paths: M Refactoring-Critics.package/RBSentNotImplementedRule.class/definition.st R Refactoring-Critics.package/RBSentNotImplementedRule.class/instance/accessing/implementorsCache.st M Refactoring-Critics.package/RBSentNotImplementedRule.class/instance/accessing/implementorsExists_.st R Refactoring-Critics.package/RBSentNotImplementedRule.class/instance/private/resetImplementorsCache.st R Refactoring-Critics.package/RBSentNotImplementedRule.class/instance/running/checkClass_.st A Rubric.package/RubEditingArea.class/instance/initialize-release/beForSmalltalkCodeInClass_.st A Rubric.package/RubScrolledTextMorph.class/instance/initialize-release/beForSmalltalkCodeInClass_.st M Rubric.package/RubSmalltalkCodeMode.class/definition.st M Rubric.package/RubSmalltalkCodeMode.class/instance/shout/classOrMetaClass_.st A Rubric.package/RubSmalltalkCodeMode.class/instance/shout/updateStyler.st M Rubric.package/RubSmalltalkCodeMode.class/instance/shout/withShoutStyler.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50609.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50610.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50609.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50610.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A SmartSuggestions.package/SugsProcessBrowserContext.class/README.md A SmartSuggestions.package/SugsProcessBrowserContext.class/definition.st A SmartSuggestions.package/SugsProcessBrowserContext.class/instance/refactoring/sourceTextArea.st A SmartSuggestions.package/SugsProcessBrowserContext.class/instance/selection/selectedClass.st A SmartSuggestions.package/SugsProcessBrowserContext.class/instance/selection/selectedMethod.st A SmartSuggestions.package/SugsTranscriptContext.class/README.md A SmartSuggestions.package/SugsTranscriptContext.class/definition.st A SmartSuggestions.package/SugsTranscriptContext.class/instance/refactoring/selectedClass.st A SmartSuggestions.package/SugsTranscriptContext.class/instance/refactoring/selectedMethod.st A SmartSuggestions.package/SugsTranscriptContext.class/instance/refactoring/sourceTextArea.st M SmartSuggestions.package/extension/DiffMorph/instance/sugsContext.st A SmartSuggestions.package/extension/ProcessBrowser/instance/sugsContext.st A SmartSuggestions.package/extension/ThreadSafeTranscript/instance/sugsContext.st M Spec-MorphicAdapters.package/MorphicTextAdapter.class/instance/private/setEditingModeFor_withBehavior_.st M Spec-MorphicAdapters.package/MorphicTextAdapter.class/instance/spec protocol/classOrMetaClass_.st A Tool-ProcessBrowser.package/ProcessBrowser.class/instance/accessing/methodText.st M Tool-ProcessBrowser.package/ProcessBrowser.class/instance/accessing/selectedMethod.st M Tool-ProcessBrowser.package/ProcessBrowser.class/instance/accessing/stackListIndex_.st M Tool-ProcessBrowser.package/ProcessBrowser.class/instance/stack list/changeStackListTo_.st M Tool-ProcessBrowser.package/ProcessBrowser.class/instance/views/build.st Log Message: ----------- 50610 17683 but right button (menu) is broken in process browser https://pharo.fogbugz.com/f/cases/17683 17688 speedup RBSentNotImplementedRule https://pharo.fogbugz.com/f/cases/17688 17065 Transcript has no context menu https://pharo.fogbugz.com/f/cases/17065 17684 highlighting broken in message list (senders off / implementors of) https://pharo.fogbugz.com/f/cases/17684 http://files.pharo.org/image/50/50610.zip

Feb. 24, 2016

Re: [Pharo-dev] [Pharo-users] help needed for testing the order in spotter

by Tudor Girba

Hi, > On Feb 24, 2016, at 11:24 AM, Damien Cassou <damien.cassou(a)inria.fr> wrote: > > Tudor Girba <tudor(a)tudorgirba.com> writes: > >> Hi, >> >> We need a bit of help with testing the order in Spotter. We do not have a real solution yet, but the current one should improve the situation. >> >> We need a bit of feedback for this one: >> https://pharo.fogbugz.com/f/cases/17656/Spotter-should-sort-search-results >> >> Essentially, you would need to load the latest GT-Spotter and GT-SpotterExtensions-Core and let us know if it works better for spotting implementors or classes. > > I have the impression this is already in the image. It seems to be > working fine. Next time, it might help if you could just give us a Pharo > expression that loads the patch. Indeed, it is now in the image. Unfortunately, there was no feedback :(. I will do it better next time. I just thought that given the urgency with which issues were reported, people would have been more eager to test, so I hurried up :). Cheers, Doru > -- > Damien Cassou > http://damiencassou.seasidehosting.st > > "Success is the ability to go from one failure to another without > losing enthusiasm." --Winston Churchill -- www.tudorgirba.com www.feenk.com "Quality cannot be an afterthought."

Feb. 24, 2016

Re: [Pharo-dev] GTSpotter displayed shortcut glitch

by Tudor Girba

Hi, > On Feb 24, 2016, at 6:00 PM, stepharo <stepharo(a)free.fr> wrote: > > Hi doru and andrei > > I saw that you are displaying the shortcuts that can be used and this is good. > Now may be I got it wrong but > Cmd + M is not for implementors everywhere else it is Cmd + m > Same for senders > So I checked and Cmd + n is senders but displayed Cmd+N > Cmd+N shows references and this is cool. We used the same logic of displaying the keybindings as it is used in the menu. There if you have Shift, it appears explicitly. For example, for references, we have Shif+Cmd+N. I also think that we should have the letters in smallcaps, but at least now it is uniform. We can decide to change it everywhere, and I would be fine with that. Cheers, Doru > Stef > -- www.tudorgirba.com www.feenk.com "Every thing has its own flow."

Feb. 24, 2016

search query?

by stepharo

Hi in the debugger we get a search and I wonder what it is. I type true, each and nothing happens (well I do not get the methods list). I think that the ghost text is misleading since we should/can only type selectors.

Feb. 24, 2016

Re: [Pharo-dev] [pharo-project/pharo-core] 4ade51: 50609

by stepharo

thanks this one is also important :) Le 24/2/16 18:02, GitHub a Ã©crit : > 50609 > 17687 fix: changing class comments does not work > https://pharo.fogbugz.com/f/cases/17687

Feb. 24, 2016