Page MenuHomeFreeBSD

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

Authored by jah on May 12 2020, 5:21 AM.
Tags
None
Referenced Files
F81655556: D24815.id.diff
Fri, Apr 19, 1:14 PM
Unknown Object (File)
Fri, Mar 29, 8:30 AM
Unknown Object (File)
Dec 23 2023, 11:47 AM
Unknown Object (File)
Dec 13 2023, 4:07 PM
Unknown Object (File)
Nov 27 2023, 1:56 PM
Unknown Object (File)
Nov 24 2023, 2:22 AM
Unknown Object (File)
Nov 15 2023, 7:35 PM
Unknown Object (File)
Nov 11 2023, 12:53 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.