On Wed, Jan 31, 2018 at 12:23 PM, zhangyi (F) <yi.zhang@xxxxxxxxxx> wrote: > Hi All, > > The master branch on my overlayfs-progs [1] has been updated. > > No feature add, just handle comments form Amir in last iterations and > fix some bugs: > - Improve header comments for many functions, these becomes more readable. Looking very good! FYI the "header comment" which I requested, are the comment on the top of the file. But hey I'm not complaining ;-) > - Fix some path manipulation bugs in path.c. Sigh. Too bad you couldn't find a path manipulation library for joinpath() and basename2(). Things like that should really never be re-implemented. If you really have to implement them or really want to, then you should write simple unit tests that run on 'make test' to verify that your helpers do what you claim they do for all the listed inputs, because right now, I doubt that you have good test coverage for complex path composing just from running fsck.overlay itself. > - Improve impurities check. > - Split out functions and constants from kernel into overlayfs.c/.h. > - Update README, split out and detail TODO list. > Looks good! Amir. -- To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
