Issue 5198 in pharo: All the methods in convenience-debugging in ToolRegistry should be deprecated.

Status: Accepted
Owner: ----
Labels: Milestone-1.4 Type-Cleanup Importance-High Difficulty-Easy

New issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

for a full description read issue 4913.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Issue 5198: All the methods in convenience-debugging in ToolRegistry should  
be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

This issue is now blocking issue 4913.
See http://code.google.com/p/pharo/issues/detail?id=4913

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

the methods
- basicInspect:
- explore:
- exploreViewHierarchy:

are easy to deprecated and change the clients.
In contrast, the following methods
- inspect:
- inspect:label:
- inspectorClassOf:
- registerInspector:for:
depends on the instance variable inspectorsMap, where you can register
a specific inpsector for a class.
Therefore, what do you do with the previous methods?
if you deprecate them, then where do you put that logic?










_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #3 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

i submitted a slice where i fixed the easy ones.

But someone should review the following methods
- debugError
- debugSyntaxError


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Cc: [hidden email]

Comment #4 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

thanks!
Igor can you have a look?


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #5 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

ok.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #6 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

I'm sorry to ask this but
any news?

alternative:
maybe we could integrate the easy methods and leave the hard part in  
another issue


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #7 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

any news... good question.

This issue waits for someone (who ever) to look at it and double check that  
it makes sense.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #8 on issue 5198 by [hidden email]: All the methods in  
convenience-debugging in ToolRegistry should be deprecated.
http://code.google.com/p/pharo/issues/detail?id=5198

checked with igor.

In 14353


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker