In article <[email protected]>,
Michael Lorenz <[email protected]> wrote:
>-=-=-=-=-=-
>
>+struct wsdisplayio_edid_info {
>+      int buffer_size;
>+      int data_size;
>+      uint8_t *edid_data;
>+};

Shouldn't the sizes be size_t or uintXX_t?
Signed sizes lead to bugs.
Shouldn't edit_data be void *?

christos

Reply via email to