[PATCH 04/14] repair: split up scanfunc_ino

Christoph Hellwig hch at infradead.org
Tue Oct 13 17:19:46 CDT 2009


On Mon, Oct 12, 2009 at 03:06:40PM -0500, Eric Sandeen wrote:
>> +					agno, agbno,
>> +				mp->m_sb.sb_inopblock);
>
> pretty weird indentation here can't you just merge w/ previous line?

XFS (especially userspace code) uses this in lots of places.  Gives
more space to messages but keeps the normal arguments normally aligned.

> Also is the change from bno to agbno intentional in the message?
> I guess it's fine.

That version is more correct.  Don't remember how it got in, though.




More information about the xfs mailing list