[MessageBroker] Fixed C++ standard issue by removing comment on no discard (supported only in c++20)

[Code] Fixed compilation/debugging with visual studio code and partial fix for compilation on windows

- Remaining issue: Could not start application without 2008 C++ runtime dll (that are deprecated)

Code compiles but does not link with make (qmake + make)

I had issues compiling with CMake due to configuration issues with qt libraries and this needs further investigation on my side (Windows 11 with qt's mingw).

Subscribe returns a unique_ptr of subscription that will unsubscribe
automatically when deleted

- Fixed error in Module.cpp: std::shared_ptr<Module>(this) is a mistake
  that can lead to crashes, plus replaceMe accepts raw pointers.
-  customContextMenuActions is now a list of unique pointers since the
   custom actions are stored within the class and used by QT directly.

- rateLabel and logCheckBox are now unique pointers since they are used
  only within the class

- Now windows will maintain a shared_ptr of the contained module
- replace child accepts shared_ptr of new modules
- error displayer is a shared pointer
 as it is used out of the window class
- Added window deleter to update the message broker by
   unsubscribing from all filters
   since the module is being deleted

- I cannot use a shared_ptr for this call since I cannot use a shared_ptr
instance of "this" pointer

- Using smart pointers to store modules (using weak ptr since this must
      not control the life of a module)
- Added functionality to unsubscribe from all filters
- callbacks are now stored in a unique_ptr to prevent memory leaks when
    the pair is removed from the MultiMap

- Changed replaceMe function to use normal pointers as oldModule.
    I could not use shared_ptr because I cannot give a shared_ptr of a
    module whithin the module.
- Used new SkywardHubCore method to add menu actions

- Using smart pointers to contain windows and actions
- Added method to add actions to QMenus directly from the Core
- rework of addWidowFromXML to work with smart pointers

Now every window will maintain a shared_ptr of a module in order to
prevent it from being deleted
and automatically delete it when the window is destroyed

Modules are now created as shared pointers to avoid deletition
while still in use by other components