On Wed, Aug 20, 2014 at 12:12 PM, Dilip kumar <
dilip.kumar@huawei.com> wrote:
>
> I have reviewed the patch and did not find any major comments.
Thanks for the review.
> There are some comments I would like to share with you
>
>
>
> 1. Rebase the patch to current GIT head.
Done.
>
> 2. + * Construct symlink file
>
> + */
>
> + initStringInfo(&symlinkfbuf);
>
> I think declaration and initialization of symlinkfbuf string can be moved under #ifdef WIN32 compile time macro,
>
> for other platform it’s simply allocated and freed which can be avoided.
Agreed, I have changed the patch as per your suggestion.
>
> 3. + /*
>
> + * native windows utilites are not able create symlinks while
>
> + * extracting files from tar.
>
> + */
>
>
>
> Rephrase the above sentence and fix spelling mistake (utilities are not able to create)
Done.