[PATCH v2 10/10] xfs: isolate xfs_qm_quotacheck to its source file
Jeff Liu
jeff.liu at oracle.com
Fri Apr 25 03:05:36 CDT 2014
On 04/25 2014 14:55 PM, Christoph Hellwig wrote:
> On Fri, Apr 18, 2014 at 08:59:00AM +0800, Jeff Liu wrote:
>> From: Jie Liu <jeff.liu at oracle.com>
>>
>> Isolate xfs_qm_quotacheck() to xfs_qm.c as there is no other users of it.
>>
>> Signed-off-by: Jie Liu <jeff.liu at oracle.com>
>
> Looks good, but the description is a bit confusing, we'd normallay say
> something like:
>
> "xfs: mark xfs_qm_quotacheck as static
>
> xfs_qm_quotacheck is not used outside of xfs_qm.c. Mark it static and
> move it around in the file to avoid a forward declaration."
Nice suggestion, thanks for your review.
Thanks,
-Jeff
More information about the xfs
mailing list