3 votes

Java code to delete all empty folders within a directory

You're using an actual logger - that's good. However, for the errors that appear in cleanupDirectory, it's probably a better idea to throw an exception. The caller ...
Reinderien's user avatar
  • 60.7k
3 votes

Java code to delete all empty folders within a directory

You're not taking into account loops created by symbolic links. This may not be a problem to your use case but in a generic case it will lead to a stack overflow error. A quick but a bit dirty fix ...
TorbenPutkonen's user avatar
2 votes
Accepted

recursive_invocable and recursive_project_invocable Concept Implementation in C++

Your code is correct, however it can be simplified in a number of ways: Unnecessary if-statements Whenever you see ...
G. Sliepen's user avatar
  • 54.1k
1 vote
Accepted

recursive_find and recursive_find_if_not Template Functions Implementation in C++

Looks good, except for one issue that I see I also missed in my previous reviews of your recursive find algorithms: The projection is applied incorrectly The projection should always be applied to the ...
G. Sliepen's user avatar
  • 54.1k
1 vote
Accepted

A recursive_find_if Template Function with Unwrap Level Implementation in C++

This looks fine, except that you indeed did not constrain Proj and UnaryPredicate, which will result in hard to read error ...
G. Sliepen's user avatar
  • 54.1k

Only top scored, non community-wiki answers of a minimum length are eligible