Replace libdparse with DMD in AlwaysCurlyCheck

[Vladiwostok]

This check ensures some statements (if, for, foreach, while, do, try-catch) are always written with braces. My solution is to visit Compound statements and mark their parent as having braces, since compound statements in DMD are statements having more than 1 child statement (forcing a brace block). AST node changes include:

• libdparse BlockStatement => DMD CompoundStatement;
• libdparse ForeachStatement => DMD ForeachStatement and ForeachRangeStatement;
• libdparse TryStatement => DMD TryCatchStatement (try statement with catch blocks) and TryFinallyStatement (try statement with finally block).

For visiting the common statements I have created a template. For TryCatchStatement and TryFinallyStatement I have created separated visit functions because:

• a try with only catch blocks and no finally block is a TryCatchStatement ;
• a try with a finally block and no catches is a TryFinallyStatement ;
• a try with catch blocks and finally statement is a TryFinallyStatement having a TryCatchStatement as a child. So, for a TryCatchStatement => check the try block and each catch block. For a TryFinallyStatement, if the tryBody is a TryCatchStatement => just visit the block, if not also check it; check the finally block.

I've also extended the unit tests by adding test cases for passing scenarios and a test case for a try-catch-finally statement.

[RazvanN7]

I have a hard time understanding the logic here. Could you maybe explain a bit how you implemented this given that dmd does not store whether braces have been used or not?

[RazvanN7]

Please, when you do PR try to explain a bit what the check does and how you implemented it. This makes it easier for me to review it and merge it.

[RazvanN7]

mind the conflicts