[Pharo-project] Issue 3948 in pharo: Transcript and ThreadedTranscript needs to be merged
Status: Accepted Owner: marcus.d...@gmail.com Labels: Milestone-1.3 New issue 3948 by marcus.d...@gmail.com: Transcript and ThreadedTranscript needs to be merged http://code.google.com/p/pharo/issues/detail?id=3948 Transcript is now not a Stream subclass anymore. ThreadedTranscript's install method now destroys the Transcript class. Transcript now has a semaphore, so maybe it is thread safe? merge needed.
Comment #1 on issue 3948 by fernando...@gmail.com: Transcript and ThreadedTranscript needs to be merged http://code.google.com/p/pharo/issues/detail?id=3948 Transcript is designed to be thread safe. Regarding the merge, i've replaced every use of ThreadedTranscript with (new) Transcript. Maybe i've omitted something?
Comment #2 on issue 3948 by fernando...@gmail.com: Transcript and ThreadedTranscript needs to be merged http://code.google.com/p/pharo/issues/detail?id=3948 I've replaced the Transcript class with an equal TranscriptModel instance, reverting to the use of a global Transcript, according to Igors suggestion. I also modified a bit the TranscriptMorph, to denote the read-only nature of the text. (Igor suggested to be able to select the text as well, not yet there); and changed all the uses ( in Core) to go through StandardToolSet openTranscript. First load the changeset, then evaluate the contents of the saved workspace. Attachments: TranscriptModelLoad.1.cs 14.0 KB TranscriptPostLoad.ws 404 bytes
Updates: Status: FixProposed Comment #3 on issue 3948 by marcus.d...@gmail.com: Transcript and ThreadedTranscript needs to be merged http://code.google.com/p/pharo/issues/detail?id=3948 (No comment was entered for this change.)
participants (1)
-
pharo@googlecode.com