Page MenuHomeFreeBSD

vt(4): Add support for `vidcontrol -C'
ClosedPublic

Authored by jah on May 12 2020, 5:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 6:00 AM
Unknown Object (File)
Sep 29 2024, 1:02 PM
Unknown Object (File)
Sep 28 2024, 7:46 AM
Unknown Object (File)
Sep 25 2024, 1:20 PM
Unknown Object (File)
Sep 22 2024, 1:10 AM
Unknown Object (File)
Sep 18 2024, 2:30 AM
Unknown Object (File)
Sep 18 2024, 12:40 AM
Unknown Object (File)
Sep 16 2024, 11:54 PM
Subscribers

Details

Summary

PR: 224436

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jah requested review of this revision.May 12 2020, 5:21 AM

I will try this out shortly

Have you been able to test it? Works fine for me, but there might be corner cases I'm just not familiar with.

LGTM

I noticed one nit - after clearing history you can press Scroll Lock and scroll up through the cleared history buffer, while we probably want to reset that as well. However, I'd suggest that you commit this now and we can address that in a followup. I also did not check how sc(4) behaves with respect to this.

This revision is now accepted and ready to land.May 28 2020, 4:49 PM

LGTM

I noticed one nit - after clearing history you can press Scroll Lock and scroll up through the cleared history buffer, while we probably want to reset that as well. However, I'd suggest that you commit this now and we can address that in a followup. I also did not check how sc(4) behaves with respect to this.

From what I can tell, it looks like sc(4) just clears the entire buffer using sc_vtb_clear() without making additional adjustments. But I agree that it doesn't seem useful to be able to scroll back through hundreds of lines of nothing. It looks like resetting the scrollback state should be pretty simple; I'll post a follow-on change to do that.

This revision was automatically updated to reflect the committed changes.