* Fix more racey instances of `if (!weakPtr.expired()){...weakPtr.lock()...}`
* Fix `_multiLinkSingleVehicleTest` thinking that link 1 will always be the primary link
* Remove the `LinkInterface::Channel` object
* Remove the reserved channel 0 and make channel 0 reservable
* Ensure that any user of the `reserveMavlinkChannel()` function will get a compiler error to note that a return of 0 no longer means error
* Make the `APMFirmwarePlugin` use a static channel member mutexed also by a static member
* `_handleIncomingMavlinkBytes` needs to use a different channel because it processes messages parallel to `MAVLinkProtocol::receiveBytes`
* `_handleIncomingMavlinkBytes`'s auxiliary channel needs its own mutex guard
* causes all calls out to mavlink library from `MockLink` to use channel 0,
which causes problems with at least `VehicleLinkManagerTest::_multiLinkSingleVehicleTest()`
and possibly other tests
The main problem with the previous implementation is that the QGCCacheWorker
does not have an event loop (not calling exec).
Since network function that are not async (such as waitConnection) needs an eventloop,
the Fuctor will run in another thread.
To fix this behaviour, the implementation was moved from to an async approach with signals.
Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>
For increased type safety, some enums have been changed to
scoped enums in Qt 6.2, see
https://codereview.qt-project.org/c/qt/qtconnectivity/+/336678
This patch adapts qgroundcontrol to this change.
Since C++11, enums inject their symbols in both their own
and their parent namespace, so this patch can be merged right
now.
Signed-off-by: Andreas Buhr <andreas.buhr@qt.io>
* Fix racey access of `QGCCacheWorker::_taskQueue`
* `QGCCacheWorker::_valid` needs to be atomic
* Use `QScopedPointer<QSqlDatabase>` for safety
* Some refactoring for readability & debugging
* Name and layout change for UDP/TCP settings
- To be more coherent with terminology used by MAVLink proxy/router softwares, the connection fields' labels was changed.
- A slight redesign of UDP server/host connection settings to allow for explaining notes and easier overview.
* Inherit widths from parent for less static layout
* Removed non-functional and unused reference
Got a hint from the flatpak people on IRC and connected the license IDs with and "AND". Works perfectly fine with Gnome's software center.
Signed-off-by: Thomas Karl Pietrowski <thopiekar@gmail.com>
It provides metadata for app stores like KDE's Discover or the Gnome Store.
The info inside this file can be improved. It has been quickly generated with a generator made by the Freedesktop people.
Signed-off-by: Thomas Karl Pietrowski <thopiekar@gmail.com>
According to Freedesktop spec the icon should be placed in the fallback theme and use the id of the application.
Signed-off-by: Thomas Karl Pietrowski <thopiekar@gmail.com>
While working on the flatpaks, I got complaints from flatpak-builder that org.mavlink.qgroundcontrol.desktop was not found.
Since flatpak is made by the Freedesktop people, I strongly believe that this is actually an convention, which they want to enforce with their flatpak ecosystem.
Nevertheless, just doing so, so the builder stops complaining at least.
Signed-off-by: Thomas Karl Pietrowski <thopiekar@gmail.com>
Hobbs meter is implemented differently for PX4 and
Ardupilot. This commit adds support for Ardupilot
and moves the method to firmware plugin instead
of being hardcoded in vehicle
Basically unextracting the deb-file with ar and tar as dpkg would do, too.
This is more handy if you are in a non-Debian environment or very minimal one.
Signed-off-by: Thomas Karl Pietrowski <thopiekar@gmail.com>
Thies Lennart Alff
Keith Bennett
Patrick José Pereira
PX4BuildBot
DonLakeFlyer
Andreas Buhr
Christoffer Björnram
Remek Zajac
Tobin
Thomas Karl Pietrowski
david sastre
Hunter N. Morgan
Beat Küng