Hello, On Wed, 29 Jun 2011 04:23:00 +0000 (UTC) [email protected] (Christos Zoulas) wrote:
> 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. True. > Shouldn't edit_data be void *? The EDID block is a byte array and read as such. have fun Michael
