Remove functionality from File.h to fix undefined behaviour

[RobinTF]

The function ad_utiliyt::File::readFile was was subject to undefined behaviour if the buffer passed to the function does not contain a null terminator. This function is now completely removed and its only usage is replaced by std::ifstream.
Also some parts of the class ad_utility::File are cleaned up by removing unused functionality and unused includes.

[codecov]

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (34af3a7) 84.30% compared to head (6e7c6b8) 84.31%.

Files	Patch %	Lines
src/util/File.h	87.09%	3 Missing and 1 partial ⚠️
src/engine/Operation.cpp	40.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1225      +/-   ##
==========================================
+ Coverage   84.30%   84.31%   +0.01%     
==========================================
  Files         304      304              
  Lines       29091    29057      -34     
  Branches     3440     3437       -3     
==========================================
- Hits        24524    24500      -24     
+ Misses       3163     3157       -6     
+ Partials     1404     1400       -4     

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

[joka921]

I have two very minor suggestions.

[sonarcloud]

Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

7 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud