On 8/8/13 6:26 PM, Eric Sandeen wrote: > On 8/8/13 5:28 PM, Ben Myers wrote: >> * remove unused uuid unparse in xfs_agf_verify >> * fix an unnecessary loop in clear_dinode_core > > These should be 2 commits (they do 2 different things), > with properly descriptive summaries & changelogs. > > For the 2nd, it's not an unnecessary loop, it's a memory > corruptor; that should be noted in the changelog. > > TBH I've only reviewed the latter, I need to look at > the first. Yup the first is fine too, but should be a separate commit. Thanks, -Eric > -Eric > > _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
