August 2010 - Pharo-dev - lists.pharo.org

Re: [Pharo-project] How do you extract a superclass if you can't select multiple classes?

by StÃ©phane Ducasse

> Seriously - I think you've shown that with a good engine (which I'm hoping it is - it seemed that way when I first came across a few bits in Dolphin) - and a bit of knowledge (although you have tons - but I'm hoping we can follow in your footsteps), a few people could make a big impact on what is possible (I certainly don't find it easy to even contemplate writing these things in Eclipse, as the model is very complicated). Yes!!! Some eclipse refactoring were are may be still are because java is really complex compare to smalltalk and we should take advantage of that. >> Have a look the method #refactoring in ORCmdCreateSubclassRefactoring >> and ORCmdCreateSuperclassRefactoring. Maybe you can improve something >> in terms of UI -> Refactoring Engine interaction? > > I will certainly have a look - and maybe with a bit of guidance at ESUG I might be able to make a difference too. > >> There are already numerous refactorings that I "ported" from Eclipse: > > I just took these for granted... I've been "spoiled" while being away, and had forgotten what the original base set was. I LOVE the generate multiple accessors (and the ability to edit that list and remove ones you don't want). I sometimes which it was a bit more tickable - but the alt-x (or right click) on an item works reasonably well. > >> As well as some new Smalltalk specific ones: > > Nice. > > Thanks as always for your contributions on this - its inspiring > > Tim > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 6, 2010

Re: [Pharo-project] How do you extract a superclass if you can't select multiple classes?

by StÃ©phane Ducasse

On Aug 6, 2010, at 3:14 PM, Tim Mackinnon wrote: > Well I'm certainly glad that you cared to get the tests back up to scratch - and it makes our whole community more productive. A big thanks for this! > > I'm just a bit embarrassed that in our community - we keep talking about how we have these great refactoring tools (that were invented in smalltalk) - but it looks like in Pharo (and I guess Squeak) - they've fallen a bit behind in terms of usability. yes this is clear. This is why we should wake up and this is why I thank lukas to have invested and fixed RB and its engine. > My scenario is a very frequent one - and it should be possible to do effortlessly (and I think that the pieces are in place to do it). > > Maybe at ESUG we can rally some energy to help support some of these things (hold an appeal fund... not money - but contributions). I want to help - I just am not sure how to yet - and sometimes this is partly my surprise to discover that some of the things I thought already worked - don't. > > Maybe in the show us your projects demo's it might be worth replaying this scenario and showing how it works in Pharo, and how it works in Eclipse (and maybe another Smalltalk) - and then showing some of the features available in the refactoring engine and trying to get support for improving it. This along with useful keyboard shortcuts are the things that eat away at the awesome productiveness I've noticed in other areas. start small and help this is the best things and once a small thing is done pass to the next one. We will all make progress. Stef > > Tim > > > On 6 Aug 2010, at 13:03, Lukas Renggli wrote: > >> On 6 August 2010 13:04, Tim Mackinnon <tamackinnon(a)gmail.com> wrote: >>> Hmmm - (again if you can please be patient with me)... I really am missing >>> something big here? >>> >>> So in my example class X and Y are subclasses of WAComponent - which has >>> tons of subclasses (it could be even worse, they could subclass Object?). So >>> does this mean that I should create my superclass A first? >>> >>> E.g. I create a new subclass of WAComponent, called A. I then make X and Y >>> subclass A. Now I can do the "Refactor Class | Create Subclass" refactoring, >>> and so I have to call it A2 (ultimately I only want A - but if needs must). >>> Then I do get the prompt that you are referring to and can select both X and >>> Y. Finally it just offers to make me class A2 but unlike what I was trying >>> below it doesn't offer to move up any methods or instance variables???? >>> >>> So this doesn't work? And furthermore it seem much more convoluted than it >>> should be? >> >> I don't know. >> >> Keep in mind that nobody really cares about OmniBrowser, the >> Refactoring Engine, and all the other development tools. A few years >> ago they were just rotting along, most tests were broken and the >> refactoring code base was on outdated code from the last century. I am >> maintaining these tools because they make me more productive, not >> because it is fun. >> >> I would love to use something as sophisticated as Eclipse, but I don't >> have interest to do that myself. So I just continue to fix and enhance >> what works for me. If somebody else provides something better, I am >> happy to give it a try. >> >> Lukas >> >>> I don't understand why Refactor Class | Create Superclass doesn't prompt me >>> for other peer classes (this is what I want - in fact what I really, really >>> want is to select 2 classes in the first place). >>> >>> Maybe I'm being really stupid, or does no-one use this refactoring in Pharo? >>> It would seem that my scenario is a very common one - so I'm struggling with >>> this. And frustratingly it seems that all the horsepower to do this, is >>> built into the tools but they just don't use them in an intuitive way? >>> >>> Tim >>> >>> >>> On 6 Aug 2010, at 11:18, Lukas Renggli wrote: >>> >>>> Select the superclass of your classes X and Y and perform the 'create >>>> subclass' refactoring. A dialog asks you to choose the subclasses of >>>> your new class. >>>> >>>> This old website describes most of the refactorings: >>>> http://st-www.cs.illinois.edu/users/brant/Refactory/Refactorings.html. >>>> There is also a description in the tooltips of the menu actions. >>>> >>>> Lukas >>>> >>>> On 6 August 2010 11:47, Tim Mackinnon <tamackinnon(a)gmail.com> wrote: >>>>> >>>>> Hi Lukas - sorry to keep this thread going, but I don't quite understand >>>>> what you mean? Partly I think the issue is that over the years I've >>>>> become >>>>> very familiar with the Eclipse/IntelliJ refactorings (and how they >>>>> approach >>>>> things) and I've slowly forgotten how they work in smalltalk (although - >>>>> in >>>>> Dolphin they seem to more closely match Eclipse/IntelliJ - adding to my >>>>> confusion in Pharo). >>>>> >>>>> To give some context, I had a class X, then I copied it to create class Y >>>>> - >>>>> made some changes and got something useful working. However X and Y are >>>>> mostly the same - so deserve a superclass. This sounds to me like >>>>> standard >>>>> Refactor Class | Create Superclass refactoring (available in the context >>>>> menu). Is this what you are referring to? (And this is a seaside30.rc >>>>> image). Is this the create class refactoring you are referring to? It >>>>> doesn't sound like it? >>>>> >>>>> The one I am referring to just asks me for a name of a superclass and >>>>> shows >>>>> me the method changes it will make (and I can remove those which I don't >>>>> deem common) - but they only apply to the single class I ran the >>>>> refactoring >>>>> for - say X. This then leaves me with Y which still needs work. I was >>>>> expecting somewhere where it would ask me if there are any other classes >>>>> to >>>>> consider for the refactoring (or better still - let me multi-select >>>>> several >>>>> classes before I do the "create superclass" in the first place - which >>>>> apparently is very hard in Pharo for some reason?). >>>>> >>>>> So is there some other browser, or menu option that I am missing? >>>>> >>>>> Tim >>>>> >>>>> On 6 Aug 2010, at 00:31, Lukas Renggli wrote: >>>>> >>>>>>>> Good point. I've been struggling with multiple selection too e.g. when >>>>>>>> setting the refactoring scope. There's an entry called "selection..." >>>>>>>> but >>>>>>>> since I've never been able to select more than one item from a list, >>>>>>>> selecting "selection..." has the same effect as âcategory" (when in a >>>>>>>> category list of course). >>>>>>> >>>>>>> So is there some way of applying a refactoring to more than one class? >>>>>>> What >>>>>>> is that refactoring scope menu item - is there some way to specify 2 or >>>>>>> more >>>>>>> classes in there? Why is this so hard? >>>>>> >>>>>> The refactoring scope is something completely different. It selects >>>>>> the part of the system that is considered by a refactoring. >>>>>> >>>>>> Did you ever try to go through the whole create-class refactoring? I >>>>>> get a dialog that offers me to select multiple classes. >>>>>> >>>>>> Lukas >>>>>> >>>>>> -- >>>>>> Lukas Renggli >>>>>> www.lukas-renggli.ch >>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> Pharo-project(a)lists.gforge.inria.fr >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>> >>>> >>>> >>>> -- >>>> Lukas Renggli >>>> www.lukas-renggli.ch >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> Pharo-project(a)lists.gforge.inria.fr >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> Pharo-project(a)lists.gforge.inria.fr >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> >> >> -- >> Lukas Renggli >> www.lukas-renggli.ch >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 6, 2010

Re: [Pharo-project] How do you extract a superclass if you can't select multiple classes?

by StÃ©phane Ducasse

On Aug 5, 2010, at 2:52 PM, Tim Mackinnon wrote: > Its a real pain that you can't multiple select things in the pharo UI... Agreed. Nor getting nice trees. > As I try and do things in Pharo and compare the operations to what I can easily do in Eclipse or Intelli-J - I've hit a new one - how do I extract a common superclass from 2 classes that are similar? > > I can refactor create superclass - but that just does it for one class, and then I can't easily make my second class a subclass as Pharo complains about duplicate instance variables. > > It seems like the capability is locked in there - but I keep struggling to work it out (refactoring in Pharo seems more complicated than it should be somehow - and I'm really looking forward to sitting down with someone someday and working out some keystrokes to use - extract method, extract temporary, inline temporary should just be keystrokes). > > Tim Thanks tim we need more clients :) (and producers too).

Aug. 6, 2010

Re: [Pharo-project] Issue 2756: Morphic Cleansing - Remove use of findA:

by StÃ©phane Ducasse

On Aug 6, 2010, at 4:54 AM, Guillermo Polito wrote: > Hi, removing some findA: message sendings Yes yes yes!!! We only need a few of them. > i've found: > > MenuMorph>>doButtonAction > "Do the receiver's inherent button action. Makes sense for the kind of MenuMorph that is a wrapper for a single menu-item -- pass it on the the item" > > (self findA: MenuItemMorph) ifNotNil: [:aMenuItem | aMenuItem doButtonAction] > > > And for me it doesn't make sense :). Even the comment doesn't make sense, because it isn't considering the case where a MenuMorph has multiple MenuItemMorph s. > > What should be done in cases like that? If it's my code, i should be tempted to delete it or maybe to deprecate it. And I'm really upset because it is a very difficult thing to verify not to break too much other things. So, what is the "official policy" about this cases? :P I don't like to leave it as it is... Get one or more scenario where the code behavior normally and rewrite so that it does not use findA: What I suggest is that you start with the easy findA: and publish often so that we integrate them in the system and slowly we learn and fix the ugly. Stef > > Cheers, > Guille > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 6, 2010

Re: [Pharo-project] [ANN] Pharo Metacello Universes

by StÃ©phane Ducasse

> > Removing the packages from the MetacelloRepository as Stephane suggested would just make them platform dependent. I missed your point. I did not say to remve the package form metacello but to copy them so that we do not depend on a server that suddenly disappear from earth. Now let us give a try and learn.

Aug. 6, 2010

Re: [Pharo-project] XML Parser, Monticello and unicode?

by jaayer

---- On Thu, 05 Aug 2010 08:37:34 -0700 Norbert Hartl wrote ---- >I'm trying to port the newest XML Parser from squeaksource to gemstone. In XML-Parser-AlexandreBergel.73 there is a unicode test introduced with a longer unicode xml snippet. From this release on I cannot load or merge anything. > >Besides that the xml snippet looks very strange it loads in pharo but not in gemstone. Unpacking the mcz on the console and examine the content showed that the encoding is indeed weird. I don't know what it is but it is neither ascii nor utf-8. Pharo loaded the snippet into a WideString instance. > >How does monticello handle WideString instances when written to a file? > >Norbert It appears those were added to ensure the parser didn't choke on non-UTF-8 input. However, it probably should choke on such input (except in external unparsed entities), and regardless, if such a test is to be added, the non-UTF8 characters should not appear in the literal source code of a method; this was the source of subtle image and package character encoding issues like the one you encountered. I have removed it.

Aug. 6, 2010

Re: [Pharo-project] How do you extract a superclass if you can't select multiple classes?

by Serge Stinckwich

Le 6 aoÃ»t 2010 Ã 20:14, Tim Mackinnon <tamackinnon(a)gmail.com> a Ã©crit : > Well I'm certainly glad that you cared to get the tests back up to scratch - and it makes our whole community more productive. A big thanks for this! > > I'm just a bit embarrassed that in our community - we keep talking about how we have these great refactoring tools (that were invented in smalltalk) - but it looks like in Pharo (and I guess Squeak) - they've fallen a bit behind in terms of usability. > Yes you are probably are right, but you know things does not appear magicly ;-) Feel free to propose bugs fix and enhancements in order to enhance the current state of tools. -- Serge Stinckwich

Aug. 6, 2010

Re: [Pharo-project] How do you extract a superclass if you can't select multiple classes?

by Tim Mackinnon

On 6 Aug 2010, at 15:59, laurent laffont wrote: > If I understand what you want correctly, can you try to select the > superclass, then refactor class>>create subclass. You give the name > and then you select all subclasses that you want to set as > subclasses of the new class. > > Is it right ? Not quite - that assumes that there is a reasonable common superclass (other than Object or WAComponent) - although you can get to that position with a bit of copy/paste of superclass names. THEN - if you do create subclass, that just creates a superclass for your selected classes but DOESN'T offer to promote any common methods or instance variables, that is then left to the reader to sort out. The operation, Refactor Create Superclass is closer to the operation that I want (and I think Lucas may have fixed it so that when it offers to promote methods and ivars you can remove unwanted promotions) HOWEVER - it only does this for a single class, leaving you to manually fix up your other classes by yourself. So its a combination of the two. This is a common thing (and really easy in the Java world - it takes seconds - and I suspect we have the engine to do this in Smalltalk) that I'm surprised its not there (but totally understand that the community has been stretched on other things - and I'm hoping to rally support. There are some basic things that most developers I have met expect - and this is one of them (along with extract method - which we have - but no keystroke for it, assign temporary - again which we have but no keystroke for it, and inline temporary - which we kind of have but its a bit awkward to use and again there is no keystroke for it). I also find that the code reformatter applied after a refactoring is a bit rough as well (and this is possibly why people hold off refactoring as well). I don't want to knock too much - the building blocks appear to be there, its just about building awareness and enthusiastic volunteers to move things forward (without impacting the other important work going on). Tim

Aug. 6, 2010

Re: [Pharo-project] LaTeX to (gulp) Word and related stuff

by Miguel Enrique CobÃ¡ MartÃnez

El vie, 06-08-2010 a las 10:46 -0400, Schwab,Wilhelm K escribiÃ³: > Hello all, > > >From PBE and a previous post, I see that I am not alone in using LaTeX. I once asked about editors, Lyx, Kile and TextMate were mentioned, along with emacs and vi. To folks who have mastered the latter, you have my respect :) > > TextMate appears to be a Mac-only item, but searches indicated that scribes tries to do the same thing on Linux. I have been using TexMaker and after customizing the quick build to apply latexpdf twice and to NOT view the result on its own (not sure why that was important, but it was), and a separate launcher for the pdf viewer, I'm pretty happy. > > In working on a paper, I inevitably got the "send me a copy so I can edit it with track changes." TTH did a surprisingly nice job of making an HTML document. OpenOffice did an ok job of loading it, but insists on linking the images and AFAICT pointedly refuses to embed them in a simple way. Do any of you have a solution to that problem? I would like to work as much as possible in latex and then export to Word for coauthors. Thoughts/recommendations/warnings? > In linux use latex2rtf, or latex2rtf-doc > Bill > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project -- Miguel CobÃ¡ http://miguel.leugim.com.mx

Aug. 6, 2010

Re: [Pharo-project] How do you extract a superclass if you can't select multiple classes?

by laurent laffont

If I understand what you want correctly, can you try to select the superclass, then refactor class>>create subclass. You give the name and then you select all subclasses that you want to set as subclasses of the new class. Is it right ? Laurent On Fri, Aug 6, 2010 at 11:47 AM, Tim Mackinnon <tamackinnon(a)gmail.com>wrote: > Hi Lukas - sorry to keep this thread going, but I don't quite understand > what you mean? Partly I think the issue is that over the years I've become > very familiar with the Eclipse/IntelliJ refactorings (and how they approach > things) and I've slowly forgotten how they work in smalltalk (although - in > Dolphin they seem to more closely match Eclipse/IntelliJ - adding to my > confusion in Pharo). > > To give some context, I had a class X, then I copied it to create class Y - > made some changes and got something useful working. However X and Y are > mostly the same - so deserve a superclass. This sounds to me like standard > Refactor Class | Create Superclass refactoring (available in the context > menu). Is this what you are referring to? (And this is a seaside30.rc > image). Is this the create class refactoring you are referring to? It > doesn't sound like it? > > The one I am referring to just asks me for a name of a superclass and shows > me the method changes it will make (and I can remove those which I don't > deem common) - but they only apply to the single class I ran the refactoring > for - say X. This then leaves me with Y which still needs work. I was > expecting somewhere where it would ask me if there are any other classes to > consider for the refactoring (or better still - let me multi-select several > classes before I do the "create superclass" in the first place - which > apparently is very hard in Pharo for some reason?). > > So is there some other browser, or menu option that I am missing? > > Tim > > On 6 Aug 2010, at 00:31, Lukas Renggli wrote: > > Good point. I've been struggling with multiple selection too e.g. when >>>> setting the refactoring scope. There's an entry called "selection..." >>>> but >>>> since I've never been able to select more than one item from a list, >>>> selecting "selection..." has the same effect as âcategory" (when in a >>>> category list of course). >>>> >>> >>> So is there some way of applying a refactoring to more than one class? >>> What >>> is that refactoring scope menu item - is there some way to specify 2 or >>> more >>> classes in there? Why is this so hard? >>> >> >> The refactoring scope is something completely different. It selects >> the part of the system that is considered by a refactoring. >> >> Did you ever try to go through the whole create-class refactoring? I >> get a dialog that offers me to select multiple classes. >> >> Lukas >> >> -- >> Lukas Renggli >> www.lukas-renggli.ch >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

Aug. 6, 2010