sean
add an entry and assign to camille.
We got confused with igor and it took us a while to find.
I think that the implementation in Analyse was not necessary. (so
probably having 2 and 3 is ok) even if I never saw the difference.
Stef
On 4/8/14 15:08, Sean P. DeNigris wrote:
GitHub wrote
13729 3 menu item to generate inst var accessors
https://pharo.fogbugz.com/f/cases/13729
I left a comment in the issue, but maybe no one is checking there since it's
closed...
>From the original comment:
There is currently 3 menu items to generate iv accessors:
Analyse > Create inst var accessors
Refactoring > Class Refactoring > Generate accessors
Refactoring > Inst Var Refactoring > Accessors
Each one has it's own underlying logic. I keep the second that has a
shortcut associated and remove the two others.
Please put back #3. It is a different use case. #2 is a batch operation
adding accessors for all inst vars. #3 adds accessors for one inst var. When
many inst vars are involved, it is tedious to deselect all the other
accessors...
The main problem was not that there was multiple options, but that they were
poorly organized (see
http://forum.world.st/Nautilus-accessor-over-generation-td4679480.html)
-----
Cheers,
Sean
--
View this message in context:
http://forum.world.st/pharo-project-pharo-core-ea9df7-40138-tp4770903p4771728.html
Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.
