[PATCH 14/25] xfs: remove xfs_bmap_add_extent

[Alex Elder]

On Wed, 2011-08-24 at 02:04 -0400, Christoph Hellwig wrote:
> plain text document attachment (xfs-bmapi-split-xfs_bmap_extent-2)
> There is no real need to the xfs_bmap_add_extent, as the callers know what
> kind of extents they need to it.  Removing it means duplicating the
> extents to btree conversion logic in three places, but overall it's still
> a lot less, and much simpler code.
> 
> Signed-off-by: Christoph Hellwig <hch at lst.de>

Not really a *lot* less code, but it is an improvement.

Simple question below (coding style) but this looks good.

Reviewed-by: Alex Elder <aelder at sgi.com>


. . .

>  
> +#ifdef DEBUG
> +STATIC void
> +xfs_bmap_check_leaf_extents(
> +	struct xfs_btree_cur	*cur,
> +	struct xfs_inode	*ip,
> +	int			whichfork);
> +#else
> +#define xfs_bmap_check_leaf_extents(cur, ip, whichfork)		do { } while (0)
> +#endif

Why do you use "do {} while (0)" rather than just an empty right
hand side?