One of my big pet-peeves is people who write:
return true [or even worse] boolean-exp
Because, obviously, it should simply be:
But what about (assuming 2 unrelated boolean expressions boolean-exp-1 & boolean-exp-2):
Now, by boolean logic, you could combine these into:
return (boolean-exp-1) && (boolean-exp-2);
which is equivalent.
But should you? I can't decide.
On the one hand, the latter is more concise. But I feel like it looses some of the meaning, because it merges the two boolean conditions into a single condition which might be confusing to later maintainers of the code who don't have the original insight.
By the way, I know that there is another option in many languages:
But that's just magic concisi-fying syntactic sugar applied to the earlier code.