On Fri, Oct 12, 2018 at 02:22:06AM +0200, Philippe Mathieu-Daudé wrote: > Hi Paolo, > > Here are the changes you suggested in > https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg02294.html > > First two simple cleanups while here, > then slowly convert backends then frontends. > > Regards, > > Phil. > > Philippe Mathieu-Daudé (11): > hw/ipmi: Remove unnecessary declarations > target/xtensa: Remove unnecessary declarations > chardev: Simplify IOWatchPoll::fd_can_read as a GSourceFunc > chardev: Assert backend's chr_can_read() is positive > chardev: Let chr_sync_read() use unsigned type > chardev: Let chr_write use unsigned type > chardev: Let IOReadHandler use unsigned type > chardev: Let IOCanReadHandler use unsigned type > chardev: Let qemu_chr_fe_* use unsigned type > chardev: Let qemu_chr_be_* use unsigned type > chardev: FDChardev::max_size be unsigned
s/unsigned/size_t/ in all these commit messages, as it looked like you meant 'unsigned int' with the current commit msg. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|