ping http://patchwork.ozlabs.org/patch/255998/
Riku, can you integrate this change? Thank you. Petar ________________________________________ From: Peter Maydell [peter.mayd...@linaro.org] Sent: Tuesday, July 02, 2013 3:44 PM To: Petar Jovanovic Cc: qemu-devel@nongnu.org; Petar Jovanovic; aurel...@aurel32.net; riku.voi...@linaro.org; r...@twiddle.net Subject: Re: [PATCH v3] linux-user: improve target_to_host_sock_type conversion On 1 July 2013 01:44, Petar Jovanovic <petar.jovano...@rt-rk.com> 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 <peter.mayd...@linaro.org> -- PMM
