medley
medley copied to clipboard
prompt for TEdit window now shows #<Input stream on .. in prompt window
an artifact of calls to Teditt with streams?
Odd... we must have different things loaded: for me it puts up "Please specify a region for the editing window" with no name.
I wasn't on the latest code in my loadup -- I see the same thing after rebuilding.
But is this a bug or a feature? If you gave it stream on a parrticular file, wouldn’t you want to know which stream it wants to get a window for? You could have editing sessions open on different streams for the same file, and somehow want to keep track of which is which. But then the bug would be that it doesn’t show the stream address in the title of the window after it has been created.
On Jun 5, 2022, at 3:06 PM, Nick Briggs @.***> wrote:
Odd... we must have different things loaded: for me it puts up "Please specify a region for the editing window" with no name.
— Reply to this email directly, view it on GitHub https://github.com/Interlisp/medley/issues/790#issuecomment-1146891287, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSTUJLKO4E4GH4TLVUFVJDVNUQGJANCNFSM5X5VILZA. You are receiving this because you were assigned.
this (bug) happens when I click "Edit" from the filebrowser. fb medley/rooms/ select OFFICE.TEDIT and cllick "Edit".
also
select ROOMS-INTRO click on See and you get Failed because NIL is not a NUMBER
also tracing UNPACKFILENAME.STRING to track down
OK, I see that. But I don’t think it has to do with the internals of TEDIT or UNPACKFILENAME. And I think I noticed it before and put it on a list.
I made a change to call PFI.MAYBE.SEE.PRETTY for the TEDIT-SEE command, which is what the file browser calls for SEE. (Also the macro ts in lispusers/TEDIT-PF-SEE. Before that I was just doing COPY.TEXT.TO.IMAGE, which just converts the byte sequence (including doing font changes) but doesn’t reformat in any other way.
The PFI does its own reformatting, and I thought that had a better look (maybe more sensitive to the window width, better at comments…).
The problem here is that this file has a bitmap, and PFI.PRINT.BITMAP is screwed up in some way. It seems to avoid this bug when printing to T but not when it is printing to a text stream. E.g (PRETTYFILEINDEX ‘rooms-intro NIL (OPENTEXTSTREAM) T) fails
On Jun 5, 2022, at 3:27 PM, Larry Masinter @.***> wrote:
this (bug) happens when I click "Edit" from the filebrowser. fb medley/rooms/ select OFFICE.TEDIT and cllick "Edit".
also
select ROOMS-INTRO click on See and you get Failed because NIL is not a NUMBER
https://user-images.githubusercontent.com/1116587/172072692-3bb63ebd-0797-4b10-b631-09b8509f833d.png also tracing UNPACKFILENAME.STRING to track down
— Reply to this email directly, view it on GitHub https://github.com/Interlisp/medley/issues/790#issuecomment-1146894476, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSTUJIZM5EQPVFDJPQ6C3LVNUSTRANCNFSM5X5VILZA. You are receiving this because you were assigned.
On the stream address showing up in the Window-region request string, I see that that happens even if you give a file name and not a stream. I'll fix that.
I believe this has been addressed
There's a related problem I wanted to track down: ADD-EXEC (from background menu) prompts with "Exec" instead of a reasonable prompt like "Drag region for Exec window". (When I do "filebrowser" it does ask "Specify Region for FileBrowser". SEdit says "Select region for SEdit Window" where a fuller description (from the title) would be better. A different issue?