[PATCH] xfs_io: fix comment in bmap.c

[PATCH] xfs_io: fix comment in bmap.c

[Christoph Hellwig]

Based on a patch from Tomasz Majkowski <moosh009@gmail.com> in bugzilla
#785.


Signed-off-by: Christoph Hellwig <hch@lst.de>

Index: xfsprogs-dev/io/bmap.c
===================================================================
--- xfsprogs-dev.orig/io/bmap.c	2009-01-24 15:36:56.780113315 +0100
+++ xfsprogs-dev/io/bmap.c	2009-01-24 15:37:17.875168142 +0100
@@ -140,7 +140,7 @@ bmap_f(
 		}
 	}
 
-	map_size = nflag ? nflag+2 : 32;	/* initial guess - 256 */
+	map_size = nflag ? nflag+2 : 32;	/* initial guess - 32 */
 	map = malloc(map_size*sizeof(*map));
 	if (map == NULL) {
 		fprintf(stderr, _("%s: malloc of %d bytes failed.\n"),
@@ -151,7 +151,7 @@ bmap_f(
 
 
 /*	Try the xfsctl(XFS_IOC_GETBMAPX) for the number of extents specified
- *	by nflag, or the initial guess number of extents (256).
+ *	by nflag, or the initial guess number of extents (32).
  *
  *	If there are more extents than we guessed, use xfsctl
  *	(XFS_IOC_FSGETXATTR[A]) to get the extent count, realloc some more

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs_io: fix comment in bmap.c

[Eric Sandeen]

Christoph Hellwig wrote:
> Based on a patch from Tomasz Majkowski <moosh009@gmail.com> in bugzilla
> #785.
> 

Heh, I can ACK this one :)

-Eric

> Signed-off-by: Christoph Hellwig <hch@lst.de>
> 
> Index: xfsprogs-dev/io/bmap.c
> ===================================================================
> --- xfsprogs-dev.orig/io/bmap.c	2009-01-24 15:36:56.780113315 +0100
> +++ xfsprogs-dev/io/bmap.c	2009-01-24 15:37:17.875168142 +0100
> @@ -140,7 +140,7 @@ bmap_f(
>  		}
>  	}
>  
> -	map_size = nflag ? nflag+2 : 32;	/* initial guess - 256 */
> +	map_size = nflag ? nflag+2 : 32;	/* initial guess - 32 */
>  	map = malloc(map_size*sizeof(*map));
>  	if (map == NULL) {
>  		fprintf(stderr, _("%s: malloc of %d bytes failed.\n"),
> @@ -151,7 +151,7 @@ bmap_f(
>  
>  
>  /*	Try the xfsctl(XFS_IOC_GETBMAPX) for the number of extents specified
> - *	by nflag, or the initial guess number of extents (256).
> + *	by nflag, or the initial guess number of extents (32).
>   *
>   *	If there are more extents than we guessed, use xfsctl
>   *	(XFS_IOC_FSGETXATTR[A]) to get the extent count, realloc some more
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
> 

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs_io: fix comment in bmap.c

[Felix Blyakher]

On Jan 24, 2009, at 8:39 AM, Christoph Hellwig wrote:

> Based on a patch from Tomasz Majkowski <moosh009@gmail.com> in  
> bugzilla
> #785.

Easy one, make comments reflect the code.
Looks good.

Felix

>
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
>
> Index: xfsprogs-dev/io/bmap.c
> ===================================================================
> --- xfsprogs-dev.orig/io/bmap.c	2009-01-24 15:36:56.780113315 +0100
> +++ xfsprogs-dev/io/bmap.c	2009-01-24 15:37:17.875168142 +0100
> @@ -140,7 +140,7 @@ bmap_f(
>  		}
>  	}
>
> -	map_size = nflag ? nflag+2 : 32;	/* initial guess - 256 */
> +	map_size = nflag ? nflag+2 : 32;	/* initial guess - 32 */
>  	map = malloc(map_size*sizeof(*map));
>  	if (map == NULL) {
>  		fprintf(stderr, _("%s: malloc of %d bytes failed.\n"),
> @@ -151,7 +151,7 @@ bmap_f(
>
>
>  /*	Try the xfsctl(XFS_IOC_GETBMAPX) for the number of extents  
> specified
> - *	by nflag, or the initial guess number of extents (256).
> + *	by nflag, or the initial guess number of extents (32).
>   *
>   *	If there are more extents than we guessed, use xfsctl
>   *	(XFS_IOC_FSGETXATTR[A]) to get the extent count, realloc some more
>
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs