#503 Remove unnecessary tests for preprocessor directives.

[hiker]

A small performance optimisation for parsing preprocessor directives. Originally, utils.c would test each cpp directive individually. With this small change, it will first see if the line actually starts with "#" before triggering these tests, which can greatly reduce the number of unnecessary tests.

I also moved matching of preprocessor lines after the test for endcls (since it should be more likely for any code to match the expected end directive than a preprocessor directives, which are all in all rather rare), which shaves of another 500 or so tests for one lfric example.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.21%. Comparing base (a197a85) to head (bd50fc6).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #504      +/-   ##
==========================================
- Coverage   92.21%   92.21%   -0.01%     
==========================================
  Files          87       87              
  Lines       13832    13831       -1     
==========================================
- Hits        12755    12754       -1     
  Misses       1077     1077              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[hiker]

Ready for review.

[sergisiso]

@hiker I agree with this change, the code looks clearer and it does the right thing with the codes I tested it with.