Re: [PATCH 1/2] xfs_repair: Add inline function to get avl tree node

[Eric Sandeen <sandeen@xxxxxxxxxxx>]

On 11/8/11 12:46 PM, Carlos Maiolino wrote:
> dd get_inode_offset() inline function, which will return the offset
> of a specific node in the AVL tree avoiding the need to calculate the
> the offset each time it needs to be used.

might be good to find the open-coded instances of this, and
make them use the helper too.

-Eric

> ---
>  repair/incore.h |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/repair/incore.h b/repair/incore.h
> index ee0e86a..8e311c9 100644
> --- a/repair/incore.h
> +++ b/repair/incore.h
> @@ -311,6 +311,12 @@ void             get_inode_rec(struct xfs_mount *mp, 
> xfs_agnumber_t agno,
>                             ino_tree_node_t *ino_rec);
>  
>  extern avltree_desc_t     **inode_tree_ptrs;
> +
> +static inline int
> +get_inode_offset(struct xfs_mount *mp, xfs_ino_t ino, ino_tree_node_t *irec)
> +{
> +     return XFS_INO_TO_AGINO(mp, ino) - irec->ino_startnum;
> +}
>  static inline ino_tree_node_t *
>  findfirst_inode_rec(xfs_agnumber_t agno)
>  {