Hi Marcel,


if the block is part of a do-it method, it is not installed so you will be informed that "no version are installed". We could also open a System Browser on BlockClosure in this case, but that would be inconsistent ...


A method comment could be:

"Opens a System Browser on the method that contains the receiver."

Do you think this comment adds value? :-)


Best,

Christoph

________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Montag, 23. März 2020 11:58:29
An: gettimothy via Squeak-dev
Betreff: Re: [squeak-dev] The Inbox: Tools-ct.958.mcz

There are comments missing. What if the method is a do-it? Please add comments.

Best,
Marcel

Am 16.03.2020 20:47:16 schrieb commits at source.squeak.org <commits at source.squeak.org>:

Christoph Thiede uploaded a new version of Tools to project The Inbox:
http://source.squeak.org/inbox/Tools-ct.958.mcz

==================== Summary ====================

Name: Tools-ct.958
Author: ct
Time: 16 March 2020, 8:47:01.435937 pm
UUID: 7c9173ab-a806-734d-8b7f-fcf1fe6ad124
Ancestors: Tools-mt.955

Proposal: Implement #browse on BlockClosure and CompiledCode again.

thisContext sender arguments last browse.
thisContext sender arguments last method browse.

=============== Diff against Tools-mt.955 ===============

Item was added:
+ ----- Method: BlockClosure>>browse (in category '*Tools-Browser') -----
+ browse
+
+ ^ self method browse!

Item was added:
+ ----- Method: CompiledCode>>browse (in category '*Tools-Browser') -----
+ browse
+
+ ^ self method browse!


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20200323/e7585a06/attachment.html>