On Thu, Apr 10, 2014 at 01:03:51AM +0800, Jun Li wrote: > Thanks Eric's analysis and review firstly. As not so clear to the application > context, so the first patch can not cover symlink scenarios. > In this patch, will check the backing_filename is a symlink or not firstly, > then return the full(absolute) path via realpath. > If this patch has something not coverd, please give me more suggestions. > Thx.
I don't know what series this patch is part of. I'm ignoring it, please resend a proper series with revision numbers. Also, the commit description should only contain information that will become part of the git commit history. Comments about code review or asking for suggestions are not appropriate for the commit description since they are irrelevant once the patch has been merged. Please make sure to put this stuff after the '---' line so git-am(1) will strip it from the commit description. Thanks, Stefan