11213 tem: partially support CSI [ 38 and 48 sequences with smaller depths

Review Request #1981 — Created June 7, 2019 and submitted June 29, 2019, 5:37 a.m.

Information
Owner:	tsoome
Repository:	illumos-gate
Branch:
Bugs:	11213
Depends On:
Commit:	b720a25...
Reviewers
Groups:	general
People:

Description

11213 tem: partially support CSI [ 38 and 48 sequences with smaller depths

Testing Done

Issues

0
0
1
0
All Issues: 1

Description	From	Last Updated
Is the depth comparison superfluous (w.r.t. line 1189)?	seeemef@mac.com	June 28, 2019, 7:08 p.m.

usr/src/uts/common/io/tem_safe.c (Diff revision 1)

May I ask the motivation for bool [tem_select_color()] v boolean_t (here)?

tsoome June 25, 2019, 11:02 a.m.

The loader tem is using (mostly) stdbool.h - the transition is not yet complete. The kernel tem is built with boolean_t, I think we could switch it to bool as there should not be any external dependencies, but it has not been high priority. I know it is a bit of mess...