Re: [PATCH 0/2] common: make common/rc easier to manage

[Eryu Guan <eguan@redhat.com>]

On Wed, Nov 30, 2016 at 08:32:31AM +1100, Dave Chinner wrote:
> Hi folks,
> 
> common/rc has grown huge with lots of library functions, and it's
> becoming hard to manage. The following two patches split the XFS and
> btrfs specific functionality in common/rc into separate files which
> are sourced directly from common/rc based on $FSTYP.

Thanks, Dave!

> 
> This moves a large chunk of code spread throughout common/rc into
> smaller, more contained files where it is easier to see how the
> filesystem specific pieces are put together. I'd like to see this
> happen for other filesytems, and quite possibly other groups of
> functionality that make sense to manage separately.
> 
> Thoughts and comments welcome!

They look good to me, and I'm testing them now. Two things I noticed:
- Can we take this opportunity to fix some code style issues, along with
  the movement? e.g. tab indention, if-then-else-fi format and while-do
  format and whitespace issues.
- Should _require_scratch_richacl_xfs() be moved to common/xfs too?

Thanks,
Eryu