nextspace icon indicating copy to clipboard operation
nextspace copied to clipboard

Terminal Crashes.

Open armm77 opened this issue 3 years ago • 2 comments

When you select the unlimited number of lines and set it to the current window, the whole system crashes.

https://user-images.githubusercontent.com/3104815/138931417-82a132dc-706c-47e0-bfc7-08e583f43b2c.mov

armm77 avatar Oct 26 '21 17:10 armm77

Confirmed. Actually this part of Terminal has not been correctly implemented.

trunkmaster avatar Nov 03 '21 23:11 trunkmaster

I can confirm that the VM or my physical equipment no longer shuts down when the event occurs. only a restart of the workspace is presented.

And as an image explains more than 1000 words, I leave a video of the behavior.

https://user-images.githubusercontent.com/3104815/140520102-d89acfba-9898-487f-960e-a2f89dff9865.mov

armm77 avatar Nov 05 '21 13:11 armm77

@armm77 check out new Terminal dynamic scrollback buffer size management.

Selecting "Unlimited" or number of lines preference doesn't mean that Terminal allocates maximum amount of memory for scrollback buffer to hold specified number of lines. Now it allocates screen by screen until maximum allowed number of lines will be reached while incoming data flows (commands in terminal produce output to Terminal).

trunkmaster avatar Feb 28 '24 22:02 trunkmaster

I checked it and the terminal no longer crashes when selecting the option but I see that it has a limit of 256 lines even if you select unlimited.

Screen Shot 2024-02-28 at 5 47 54 PM

armm77 avatar Feb 28 '24 22:02 armm77

have you set it with "Set Window" or "Set Default" button?

trunkmaster avatar Feb 28 '24 22:02 trunkmaster

@trunkmaster, it was my mistake select "Set Window" or "Set Default".

"Set Window" to unlimited and running ls -l /usr/bin twice crashes the application.

armm77 avatar Feb 28 '24 23:02 armm77

Confirmed. In my case it's a 3 times. Also I see some buffer garbage at some point. Will try to find out cause of that...

trunkmaster avatar Feb 28 '24 23:02 trunkmaster

@armm77 now it should work without crash. Check it please.

trunkmaster avatar Mar 01 '24 00:03 trunkmaster

Hi @trunkmaster, I see that it already works as it should, but I see that the configuration is not saved, so it shows in the preference panel that it is selected as unlimited, I close and open it again and it remains set in the preference but it only marks 256 until it is pressed and set again .

https://github.com/trunkmaster/nextspace/assets/3104815/d8444f17-f606-4ca8-ae2d-b2b14e5d83df

armm77 avatar Mar 03 '24 21:03 armm77

@armm77 fixed, thanks.

trunkmaster avatar Mar 03 '24 22:03 trunkmaster

@trunkmaster, thanks for correcting it. I close the incident

armm77 avatar Mar 04 '24 04:03 armm77