Time appropriate greetings,
Thanks for the patch.
On Tue, Jan 12, 2021 at 09:58:54PM +0100, Robin Opletal wrote:
> + // Used to handle "/", "." and ".." in filenames correctly
When I said add a comment explaining why I meant something a bit more descriptive ;)
Anyhow, fixed it up locally and merged it.
I also took the liberty to change the if chain to a switch statement, which
hopefully helps the reader a bit.
Cheers,
Reto
On Thu Jan 14, 2021 at 7:19 AM CET, Reto wrote:
> Time appropriate greetings,> Thanks for the patch.>> On Tue, Jan 12, 2021 at 09:58:54PM +0100, Robin Opletal wrote:> > + // Used to handle "/", "." and ".." in filenames correctly>> When I said add a comment explaining why I meant something a bit more> descriptive ;)> Anyhow, fixed it up locally and merged it.> I also took the liberty to change the if chain to a switch statement,> which> hopefully helps the reader a bit.>> Cheers,> Reto
Hi Reto,
Nice - I have tried to keep it simple with the comment, perhaps I have,
but too simple :)
Best,
Robin