osc: 4: handle multiple idx;spec pairs

This commit is contained in:
Daniel Eklöf 2020-04-04 14:27:44 +02:00
parent 10330c2a9d
commit ae1b235eaa
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F

41
osc.c
View file

@ -430,29 +430,23 @@ osc_dispatch(struct terminal *term)
case 4: { case 4: {
/* Set color<idx> */ /* Set color<idx> */
/* First param - the color index */ string--;
assert(*string == ';');
for (const char *s_idx = strtok(string, ";"), *s_color = strtok(NULL, ";");
s_idx != NULL && s_color != NULL;
s_idx = strtok(NULL, ";"), s_color = strtok(NULL, ";"))
{
/* Parse <idx> parameter */
unsigned idx = 0; unsigned idx = 0;
for (; *string != '\0' && *string != ';'; string++) { for (; *s_idx != '\0'; s_idx++) {
char c = *string; char c = *s_idx;
idx *= 10; idx *= 10;
idx += c - '0'; idx += c - '0';
} }
if (idx >= 256)
break;
/* Next follows the color specification. For now, we only support rgb:x/y/z */
if (*string == '\0') {
/* No color specification */
break;
}
assert(*string == ';');
string++;
/* Client queried for current value */ /* Client queried for current value */
if (strlen(string) == 1 && string[0] == '?') { if (strlen(s_color) == 1 && s_color[0] == '?') {
uint32_t color = term->colors.table[idx]; uint32_t color = term->colors.table[idx];
uint8_t r = (color >> 16) & 0xff; uint8_t r = (color >> 16) & 0xff;
uint8_t g = (color >> 8) & 0xff; uint8_t g = (color >> 8) & 0xff;
@ -462,15 +456,22 @@ osc_dispatch(struct terminal *term)
snprintf(reply, sizeof(reply), "\033]4;%u;rgb:%02x/%02x/%02x\033\\", snprintf(reply, sizeof(reply), "\033]4;%u;rgb:%02x/%02x/%02x\033\\",
idx, r, g, b); idx, r, g, b);
term_to_slave(term, reply, strlen(reply)); term_to_slave(term, reply, strlen(reply));
break;
} }
else {
uint32_t color; uint32_t color;
if (string[0] == '#' ? !parse_legacy_color(string, &color) : !parse_rgb(string, &color)) bool color_is_valid = s_color[0] == '#'
break; ? parse_legacy_color(s_color, &color)
: parse_rgb(s_color, &color);
if (!color_is_valid)
continue;
LOG_DBG("change color definition for #%u to %06x", idx, color); LOG_DBG("change color definition for #%u to %06x", idx, color);
term->colors.table[idx] = color; term->colors.table[idx] = color;
}
}
render_refresh(term); render_refresh(term);
break; break;
} }