Re: CVS commit: src/sys/dev/wscons

[christos%astron.com@localhost (Christos Zoulas)]

In article <20110629030617.0D3DC175D0%cvs.netbsd.org@localhost>,
Michael Lorenz <source-changes-d%NetBSD.org@localhost> 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