Patchwork Whitespace clean up.

login
register
mail settings
Submitter Peter Portante
Date March 21, 2012, 6:23 p.m.
Message ID <1332354187-15720-1-git-send-email-peter.portante@redhat.com>
Download mbox | patch
Permalink /patch/148069/
State New
Headers show

Comments

Peter Portante - March 21, 2012, 6:23 p.m.
Signed-off-by: Peter Portante <pportant@redhat.com>
---
 block/raw-posix.c |   28 ++++++++++++++--------------
 1 files changed, 14 insertions(+), 14 deletions(-)
Andreas Färber - March 22, 2012, 2:42 p.m.
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,

Patch

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,