qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] qemu-io: Verify read data by patterns


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH] qemu-io: Verify read data by patterns
Date: Thu, 09 Apr 2009 12:54:09 -0500
User-agent: Thunderbird 2.0.0.21 (X11/20090320)

Kevin Wolf wrote:

This is what you get for using tabs. You deserve it. ;-)

Seriously, I found this really annoying when I did the patch because almost everything else in qemu has four spaces and I needed to change my editor settings for just this file. But I still left indentation on four characters, so this line did fit for me.

I can either send a second version of the patch which fixes this line, or we could have a patch which changes the indentation of the whole file to four spaces (as specified in the coding style document). qemu-io might be new enough to not destroy valuable svn blame information with such a patch.

The only reason I merged it as-is is because the code was derived from something else. I didn't want to force a reindentation that would make it harder to keep the code synced against the upstream project.

Regards,

Anthony Liguori




reply via email to

[Prev in Thread] Current Thread [Next in Thread]