[PATCH 3/5] server: Get and set the console ColorTable

[Alexandre Julliard]

Hugh McMaster <hugh.mcmaster at outlook.com> writes:

> On Tuesday, 9 August 2016 12:58 PM, Alexandre Julliard wrote:
>> Hugh McMaster <hugh.mcmaster at outlook.com> writes:
>>> @@ -1568,6 +1568,22 @@ struct console_renderer_event
>>>      short int    font_height;
>>>  @END
>>>  
>>> +
>>> +/* Fill the console ColorTable */
>>> + at REQ(set_console_colortable)
>>> +    obj_handle_t handle;        /* handle to the console */
>>> +    VARARG(color,unsigned_int); /* ColorTable */
>>> + at END
>>> +
>>> +
>>> +/* Fetch the console ColorTable */
>>> + at REQ(get_console_colortable)
>>> +    obj_handle_t handle;        /* handle to the console */
>>> + at REPLY
>>> +    VARARG(color,unsigned_int); /* ColorTable */
>>> + at END
>>
>> It would probably be better to add this to the existing
>> get/set_console_output_info requests.
>
> screen_buffer->color_map[16] has a byte size of 64. Wouldn't that cause problems with the maximum server
> request/reply size in functions such as Get/SetConsoleScreenBufferInfoEx?

It has to be in the VARARG data, but that's what you are doing already.

-- 
Alexandre Julliard
julliard at winehq.org