On 20 August 2014 19:49, Amit Kapila Wrote
> There are some comments I would like to share with you
>
>
>
> 1. Rebase the patch to current GIT head.
>Done.
>> + 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.
I have done the testing and behavior is as per expectation,
Do we need to do some document change? I mean is this limitation on windows is mentioned anywhere ?
If no change then i will move the patch to “Ready For Committer”.
Thanks & Regards,
Dilip
