* Use correct of version pthread_setname_np depending on platform.

On Linux, this function takes two parameters. On most other platforms
(including all BSD-based platforms) it only takes a single parameter.

* Omit -Wl,--export-dynamic flag on macOS.

This flag is not supported on Clang / LLVM and on macOS it is not needed
anway because exporting all symbols is the default behavior.

When using Clang on other platforms (e.g. Linux) --export_dynamic or
-rdynamic might have to be specified instead.

* clang compiler compatibility

Co-authored-by: Sebastian Marsching <sebastian.marsching-git-2016@aquenos.com>

They were previously hidden by the "BUG found" message in
ApplicationBase due to the missing call to shutdown(). This is now being
called automatically in the Application destructor if the application
was not yet started fully, in which case this is ok to do there
(calling shutdown before destroying the Application data members is
important only if threads have been started already).

- ControlSystemModule
- VirtualModule
- findTag
- excludeTag

and some related functions.

Jens Georg authored 2 years ago and Martin Christoph Hierholzer committed 2 years ago

 - ApplicationModule can only be owned by ModuleGroup

Jens Georg authored 2 years ago and Martin Christoph Hierholzer committed 2 years ago

Move everything inside TestableMode and add decorator helper functions
to TestableMode

Jens Georg authored 2 years ago and Martin Christoph Hierholzer committed 2 years ago

While at it, fix linter issues in ModuleGroup as well

Modernise interface of status monitors and add tests of initial value propagation and data validity propagation (#8279). Improve code formating/readability.

* fix code formatting

* enable coding style test

it was anyway dysfunctional and mainly slightly reducing performance.