Re: [PATCH 3/5] xfs: remove confusing ail cursor wrapper

Alex Elder
2011-07-07T20:34:01Z

On Mon, 2011-07-04 at 15:27 +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> xfs_trans_ail_cursor_set() doesn't set the cursor to the current log
> item, it sets it to the next item. There is already a function for
> doing this - xfs_trans_ail_cursor_next() - and the _set function is
> simply a two line wrapper. Remove it and open code the setting of
> the cursor in the two locations that call it to remove the
> confusion.
>
> Signed-off-by: Dave Chinner <dchinner@redhat.com>

Good idea. I also appreciate the comment improvement;
the "returns NULL" is something I wanted to suggest
while looking at the previous patch, but that section
of the file wasn't in that patch.

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