Keep consistent the declaration of the execute functions in all child classes and change the scope of the execute function in the MemoryManagementService

Changed comments

Fixed names in ErrorHandler and enumeration of errors and removed explicit declaration of true or false in a CHECK

- changed variable names
- added breaks
- corrected a comment
- deleted explicit declaration of true/false in ifs
- fixed a bug in tests caused by passing the message by reference

- Changed disableEventActionDefinition
according to the needs of multimap
- Added errors in ErrorHandler

- Changed deleteEventActionDefinition
according to the multimap needs

- Added a new variable eventActionDefinitionID
- Added the size of the multimap as #define in ECSS_Definitions.hpp

Changed the way add action-definition function works after rechecking the standard and updated the corresponding tests as well as added documentation in those tests

# Conflicts:
#	CMakeLists.txt
#	src/Services/EventReportService.cpp

Fix a missing semicolon

Attempt #1 at fixing Vera

Attempt #2 at fixing vera

Fix Vera

Replace the unacceptable disgrace called parsing based on spaces with new, readString() based method. Also write some tests for insertion and calling, and general code cleanup and reformatting.

Be quiet Vera!

I said be quiet Vera! And you, clang-tidy, do not even think about raising your voice!

Got an idea to pacify clang-tidy without commenting anything out, let's see if it's going to work

Revert "Got an idea to pacify clang-tidy without commenting anything out, let's see if it's going to work"

This reverts commit db9e73ad

Revert "Got an idea to pacify clang-tidy without commenting anything out, let's see if it's going to work"

This reverts commit db9e73ad9d8508e59764d7359408aac962a20afe

Fix a blunder with the test version of call()

Attempt to calm down clang-tidy and fix some stupid warnings in my CRC code (nothing much, just turned the counter int in a for-loop into a uint32_t to shut gcc up)

Comment out the empty constructor to mute clang-tidy's warning about trivial constructors

Complete basic functionality (i.e. retrieval of function pointer and execution with arguments). Needs more testing and safeguarding against corrupted messages

Solve the pointer storage problem to the map, storing them as void*. Retrieval tested with explicit casts, now looking for a way to store type information of functions in a retrievable way

Make clang-tidy happy again, attempt #1

Attempt #2 at calming down clang-tidy