qgroundcontrol icon indicating copy to clipboard operation
qgroundcontrol copied to clipboard

Published 20 hours ago • verified publisher icon mavlink

fix: comm link bugs

Open dakejahl opened this issue 2 years ago • 2 comments

  • Fix UI update when a link is disconnected
  • Clean up properly if a link fails to connect

We should probably get rid of all of the std::shared_ptr and use QSharedPointer instead. ~A PR for another day.~ I pushed a commit that replace them, let me know if I should revert it. Seems like using Qt smart pointers is probably better for a Qt project. I can go through and change it everywhere in the code base if we want to do that.

dakejahl avatar Sep 14 '22 07:09 dakejahl

Does this relate to https://github.com/mavlink/qgroundcontrol/issues/9411 ?

booo avatar Sep 15 '22 17:09 booo

Closes #9411

dakejahl avatar Sep 15 '22 17:09 dakejahl

reverted the last commit

dakejahl avatar Oct 11 '22 03:10 dakejahl