search

r1cc4rdo / daily_coding_problem

Solutions to https://dailycodingproblem.com
The Unlicense
160 stars 39 forks source link

Find an edge case. And this should be legal. #4

Closed Jedshady closed 6 years ago

Jedshady commented 6 years ago

Hi, Just wanna say your solution is awesome. But with an edge case "file1.ext" just file name without a directory, here is a easy fix for it.

And after sleeping, I find another bug which happens when "tabs < len(dirs)" and what replaced dirs[-1] is a file after ascending. If so, we also need to update max_len in case file name is very long.

Thank you for your time. Cheers

r1cc4rdo commented 6 years ago

Thanks buddy. Merging your changes in.