* [Qemu-trivial] [PATCH] Whitespace clean up.
@ 2012-03-21 18:23 Peter Portante
2012-03-22 14:42 ` [Qemu-trivial] [Qemu-devel] " Andreas Färber
0 siblings, 1 reply; 2+ messages in thread
From: Peter Portante @ 2012-03-21 18:23 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, Peter Portante
Signed-off-by: Peter Portante <pportant@redhat.com>
---
block/raw-posix.c | 28 ++++++++++++++--------------
1 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/block/raw-posix.c b/block/raw-posix.c
index 2d1bc13..719a590 100644
--- a/block/raw-posix.c
+++ b/block/raw-posix.c
@@ -112,7 +112,7 @@
reopen it to see if the disk has been changed */
#define FD_OPEN_TIMEOUT (1000000000)
-#define MAX_BLOCKSIZE 4096
+#define MAX_BLOCKSIZE 4096
typedef struct BDRVRawState {
int fd;
@@ -494,7 +494,7 @@ again:
#endif
if (!fstat(fd, &sb) && (S_IFCHR & sb.st_mode)) {
#ifdef DIOCGMEDIASIZE
- if (ioctl(fd, DIOCGMEDIASIZE, (off_t *)&size))
+ if (ioctl(fd, DIOCGMEDIASIZE, (off_t *)&size))
#elif defined(DIOCGPART)
{
struct partinfo pi;
@@ -894,7 +894,7 @@ static int hdev_has_zero_init(BlockDriverState *bs)
static BlockDriver bdrv_host_device = {
.format_name = "host_device",
- .protocol_name = "host_device",
+ .protocol_name = "host_device",
.instance_size = sizeof(BDRVRawState),
.bdrv_probe_device = hdev_probe_device,
.bdrv_file_open = hdev_open,
@@ -903,12 +903,12 @@ static BlockDriver bdrv_host_device = {
.create_options = raw_create_options,
.bdrv_has_zero_init = hdev_has_zero_init,
- .bdrv_aio_readv = raw_aio_readv,
- .bdrv_aio_writev = raw_aio_writev,
- .bdrv_aio_flush = raw_aio_flush,
+ .bdrv_aio_readv = raw_aio_readv,
+ .bdrv_aio_writev = raw_aio_writev,
+ .bdrv_aio_flush = raw_aio_flush,
.bdrv_truncate = raw_truncate,
- .bdrv_getlength = raw_getlength,
+ .bdrv_getlength = raw_getlength,
.bdrv_get_allocated_file_size
= raw_get_allocated_file_size,
@@ -1015,7 +1015,7 @@ static BlockDriver bdrv_host_floppy = {
.format_name = "host_floppy",
.protocol_name = "host_floppy",
.instance_size = sizeof(BDRVRawState),
- .bdrv_probe_device = floppy_probe_device,
+ .bdrv_probe_device = floppy_probe_device,
.bdrv_file_open = floppy_open,
.bdrv_close = raw_close,
.bdrv_create = hdev_create,
@@ -1024,10 +1024,10 @@ static BlockDriver bdrv_host_floppy = {
.bdrv_aio_readv = raw_aio_readv,
.bdrv_aio_writev = raw_aio_writev,
- .bdrv_aio_flush = raw_aio_flush,
+ .bdrv_aio_flush = raw_aio_flush,
.bdrv_truncate = raw_truncate,
- .bdrv_getlength = raw_getlength,
+ .bdrv_getlength = raw_getlength,
.bdrv_get_allocated_file_size
= raw_get_allocated_file_size,
@@ -1114,7 +1114,7 @@ static BlockDriver bdrv_host_cdrom = {
.format_name = "host_cdrom",
.protocol_name = "host_cdrom",
.instance_size = sizeof(BDRVRawState),
- .bdrv_probe_device = cdrom_probe_device,
+ .bdrv_probe_device = cdrom_probe_device,
.bdrv_file_open = cdrom_open,
.bdrv_close = raw_close,
.bdrv_create = hdev_create,
@@ -1123,7 +1123,7 @@ static BlockDriver bdrv_host_cdrom = {
.bdrv_aio_readv = raw_aio_readv,
.bdrv_aio_writev = raw_aio_writev,
- .bdrv_aio_flush = raw_aio_flush,
+ .bdrv_aio_flush = raw_aio_flush,
.bdrv_truncate = raw_truncate,
.bdrv_getlength = raw_getlength,
@@ -1233,7 +1233,7 @@ static BlockDriver bdrv_host_cdrom = {
.format_name = "host_cdrom",
.protocol_name = "host_cdrom",
.instance_size = sizeof(BDRVRawState),
- .bdrv_probe_device = cdrom_probe_device,
+ .bdrv_probe_device = cdrom_probe_device,
.bdrv_file_open = cdrom_open,
.bdrv_close = raw_close,
.bdrv_create = hdev_create,
@@ -1242,7 +1242,7 @@ static BlockDriver bdrv_host_cdrom = {
.bdrv_aio_readv = raw_aio_readv,
.bdrv_aio_writev = raw_aio_writev,
- .bdrv_aio_flush = raw_aio_flush,
+ .bdrv_aio_flush = raw_aio_flush,
.bdrv_truncate = raw_truncate,
.bdrv_getlength = raw_getlength,
--
1.7.7.6
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] Whitespace clean up.
2012-03-21 18:23 [Qemu-trivial] [PATCH] Whitespace clean up Peter Portante
@ 2012-03-22 14:42 ` Andreas Färber
0 siblings, 0 replies; 2+ messages in thread
From: Andreas Färber @ 2012-03-22 14:42 UTC (permalink / raw)
To: Peter Portante; +Cc: qemu-trivial, Kevin Wolf, qemu-devel, Peter Portante
For one, while non-functional, this patch should probably be reviewed by
the block maintainer to not interfere with already queued patches.
Then the subject should indicate which part of code it touches, like
"block: " or "block/raw-posix.c: ". If Kevin agrees to merge this patch
I would suggest to amend the commit message saying that this is
converting tabs to spaces.
We don't usually place a period after the subject line (which happens to
not be a sentence) but that's an irrelevant detail. :)
Our policy has been to not apply formatting-only fixes without an
obvious reason. Is there a patch following that depends on this one?
That being said, the code movement looks correct.
Andreas
Am 21.03.2012 19:23, schrieb Peter Portante:
> Signed-off-by: Peter Portante <pportant@redhat.com>
> ---
> block/raw-posix.c | 28 ++++++++++++++--------------
> 1 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/block/raw-posix.c b/block/raw-posix.c
> index 2d1bc13..719a590 100644
> --- a/block/raw-posix.c
> +++ b/block/raw-posix.c
> @@ -112,7 +112,7 @@
> reopen it to see if the disk has been changed */
> #define FD_OPEN_TIMEOUT (1000000000)
>
> -#define MAX_BLOCKSIZE 4096
> +#define MAX_BLOCKSIZE 4096
>
> typedef struct BDRVRawState {
> int fd;
> @@ -494,7 +494,7 @@ again:
> #endif
> if (!fstat(fd, &sb) && (S_IFCHR & sb.st_mode)) {
> #ifdef DIOCGMEDIASIZE
> - if (ioctl(fd, DIOCGMEDIASIZE, (off_t *)&size))
> + if (ioctl(fd, DIOCGMEDIASIZE, (off_t *)&size))
> #elif defined(DIOCGPART)
> {
> struct partinfo pi;
> @@ -894,7 +894,7 @@ static int hdev_has_zero_init(BlockDriverState *bs)
>
> static BlockDriver bdrv_host_device = {
> .format_name = "host_device",
> - .protocol_name = "host_device",
> + .protocol_name = "host_device",
> .instance_size = sizeof(BDRVRawState),
> .bdrv_probe_device = hdev_probe_device,
> .bdrv_file_open = hdev_open,
> @@ -903,12 +903,12 @@ static BlockDriver bdrv_host_device = {
> .create_options = raw_create_options,
> .bdrv_has_zero_init = hdev_has_zero_init,
>
> - .bdrv_aio_readv = raw_aio_readv,
> - .bdrv_aio_writev = raw_aio_writev,
> - .bdrv_aio_flush = raw_aio_flush,
> + .bdrv_aio_readv = raw_aio_readv,
> + .bdrv_aio_writev = raw_aio_writev,
> + .bdrv_aio_flush = raw_aio_flush,
>
> .bdrv_truncate = raw_truncate,
> - .bdrv_getlength = raw_getlength,
> + .bdrv_getlength = raw_getlength,
> .bdrv_get_allocated_file_size
> = raw_get_allocated_file_size,
>
> @@ -1015,7 +1015,7 @@ static BlockDriver bdrv_host_floppy = {
> .format_name = "host_floppy",
> .protocol_name = "host_floppy",
> .instance_size = sizeof(BDRVRawState),
> - .bdrv_probe_device = floppy_probe_device,
> + .bdrv_probe_device = floppy_probe_device,
> .bdrv_file_open = floppy_open,
> .bdrv_close = raw_close,
> .bdrv_create = hdev_create,
> @@ -1024,10 +1024,10 @@ static BlockDriver bdrv_host_floppy = {
>
> .bdrv_aio_readv = raw_aio_readv,
> .bdrv_aio_writev = raw_aio_writev,
> - .bdrv_aio_flush = raw_aio_flush,
> + .bdrv_aio_flush = raw_aio_flush,
>
> .bdrv_truncate = raw_truncate,
> - .bdrv_getlength = raw_getlength,
> + .bdrv_getlength = raw_getlength,
> .bdrv_get_allocated_file_size
> = raw_get_allocated_file_size,
>
> @@ -1114,7 +1114,7 @@ static BlockDriver bdrv_host_cdrom = {
> .format_name = "host_cdrom",
> .protocol_name = "host_cdrom",
> .instance_size = sizeof(BDRVRawState),
> - .bdrv_probe_device = cdrom_probe_device,
> + .bdrv_probe_device = cdrom_probe_device,
> .bdrv_file_open = cdrom_open,
> .bdrv_close = raw_close,
> .bdrv_create = hdev_create,
> @@ -1123,7 +1123,7 @@ static BlockDriver bdrv_host_cdrom = {
>
> .bdrv_aio_readv = raw_aio_readv,
> .bdrv_aio_writev = raw_aio_writev,
> - .bdrv_aio_flush = raw_aio_flush,
> + .bdrv_aio_flush = raw_aio_flush,
>
> .bdrv_truncate = raw_truncate,
> .bdrv_getlength = raw_getlength,
> @@ -1233,7 +1233,7 @@ static BlockDriver bdrv_host_cdrom = {
> .format_name = "host_cdrom",
> .protocol_name = "host_cdrom",
> .instance_size = sizeof(BDRVRawState),
> - .bdrv_probe_device = cdrom_probe_device,
> + .bdrv_probe_device = cdrom_probe_device,
> .bdrv_file_open = cdrom_open,
> .bdrv_close = raw_close,
> .bdrv_create = hdev_create,
> @@ -1242,7 +1242,7 @@ static BlockDriver bdrv_host_cdrom = {
>
> .bdrv_aio_readv = raw_aio_readv,
> .bdrv_aio_writev = raw_aio_writev,
> - .bdrv_aio_flush = raw_aio_flush,
> + .bdrv_aio_flush = raw_aio_flush,
>
> .bdrv_truncate = raw_truncate,
> .bdrv_getlength = raw_getlength,
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-03-22 14:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-21 18:23 [Qemu-trivial] [PATCH] Whitespace clean up Peter Portante
2012-03-22 14:42 ` [Qemu-trivial] [Qemu-devel] " Andreas Färber
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).