> Of course! But I gave that comment for a different reason. The EmergencyEvaluator is the last resort for recovery in the interactive system. We should carefully choose and design the amount of features to keep it as robust as possible. The less features, the better.

I agree :-) However, I found the #clear feature quite essential because sometimes, I am presented error messages here that take more than the available screen space.

(Btw, could we maybe always use Display boundingBox as the frame for the EE?)

> More fancy fallbacks for Morphic are the MVC debugger or the Squeak Shell. :-) Just install them as your parent project.

Yes, I installed SqueakShell and it looks great :-) (And will look even greater when I manage to implement a CommandLineUIManager :-))
By the way, its Debugger integration still depends on the old ErrorRecursion classvar ...

Best,
Christoph

________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Montag, 16. Dezember 2019 19:56:30
An: John Pfersich via Squeak-dev
Betreff: Re: [squeak-dev] The Inbox: System-ct.1128.mcz

Hi Christoph.

> Minimality might be a general goal for sustainable software development

Of course! But I gave that comment for a different reason. The EmergencyEvaluator is the last resort for recovery in the interactive system. We should carefully choose and design the amount of features to keep it as robust as possible. The less features, the better.

More fancy fallbacks for Morphic are the MVC debugger or the Squeak Shell. :-) Just install them as your parent project.

Best,
Marcel

Am 16.12.2019 19:51:43 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:

> note that the implementation of Transcripter should be as minimal as possible. That new "clear" command is fine, I guess. :-)

Minimality might be a general goal for sustainable software development :) I just wanted to avoid to repeat myself.

Best,
Christoph

________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Montag, 16. Dezember 2019 19:37:15
An: John Pfersich via Squeak-dev
Betreff: Re: [squeak-dev] The Inbox: System-ct.1128.mcz

Hi Christoph,

note that the implementation of Transcripter should be as minimal as possible. That new "clear" command is fine, I guess. :-)

> In TranscriptStream, #showln: prints the newline first as well

I added that to TranscriptStream to make sure that even the first call to #showln: is on an extra line, regardless of its previous contents.

Best,
Marcel

Am 15.12.2019 12:15:18 schrieb commits at source.squeak.org <commits at source.squeak.org>:

A new version of System was added to project The Inbox:
http://source.squeak.org/inbox/System-ct.1128.mcz

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

Name: System-ct.1128
Author: ct
Time: 14 December 2019, 3:14:14.108612 pm
UUID: 89698e0d-cee3-2041-afa8-96674d632912
Ancestors: System-mt.1127

Extends Transcripter with #clear command and refactors readEvalPrint loop

- Adds #showln: method

=============== Diff against System-mt.1127 ===============

Item was changed:
----- Method: Transcripter>>readEvalPrint (in category 'command line') -----
readEvalPrint
| line okToRevert |
okToRevert := true.
+ [#(quit exit done) includes: (line := self request: '>')]
+ whileFalse: [ line
+ caseOf: {
+ [#revert] -> [okToRevert
+ ifTrue: [RecentMessages default revertMostRecent.
+ self showln: 'reverted: ' , RecentMessages default mostRecent.
+ okToRevert := false]
+ ifFalse: [self showln: 'Only one level of revert currently supported']].
+ [#clear] -> [self clear]. }
+ otherwise: [
+ self showln: ([Compiler evaluate: line] ifError: [:err :ex | err])] ]!
- [#('quit' 'exit' 'done' ) includes: (line := self request: '>')]
- whileFalse:
- [line = 'revert'
- ifTrue: [okToRevert
- ifTrue: [RecentMessages default revertMostRecent.
- self cr; show: 'reverted: ' , RecentMessages default mostRecent.
- okToRevert := false]
- ifFalse: [self cr; show: 'Only one level of revert currently supported']]
- ifFalse: [self cr; show: ([Compiler evaluate: line] ifError: [:err :ex | err])]]!

Item was added:
+ ----- Method: Transcripter>>showln: (in category 'accessing') -----
+ showln: anObject
+ self
+ cr;
+ show: anObject!


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