Re: [Qemu-devel] [PATCH v3] linux-user: improve target_to_host_sock_type conversion

[Petar Jovanovic]

ping
http://patchwork.ozlabs.org/patch/255998/

Riku, can you integrate this change?
Thank you.

Petar
________________________________________
From: Peter Maydell address@hidden
Sent: Tuesday, July 02, 2013 3:44 PM
To: Petar Jovanovic
Cc: address@hidden; Petar Jovanovic; address@hidden; address@hidden; 
address@hidden
Subject: Re: [PATCH v3] linux-user: improve target_to_host_sock_type conversion

On 1 July 2013 01:44, Petar Jovanovic <address@hidden> wrote:
> The patch also includes necessary code style changes (tab to spaces) in the
> header file since most of the file has been touched by this change.

This makes it pretty painful to review because of all the "not actually
changing anything" changes. If you want to fix up indent on an entire
file that would be better as a single patch on its own before this
one; then it's easy to check that that has no actual code change,
and the patch with the code changes is smaller and easier to read.

(The general rule of thumb is "if you're just changing a few lines
of indent/braces in lines that the patch touches anyway" that's fine;
larger scale changes go in their own patch if we do them at all.)

Not using the same base as the kernel (hex vs octal) made checking
the values a bit tedious too, but I think they're all correct.

That said,
Reviewed-by: Peter Maydell <address@hidden>

-- PMM