search

gazebosim / gz-gui

Builds on top of Qt to provide widgets which are useful when developing robotics applications, such as a 3D view, plots, dashboard, etc, and can be used together in a convenient unified interface.
https://gazebosim.org
Apache License 2.0
66 stars 39 forks source link

Tidy namespaces #582

Closed mjcarroll closed 8 months ago

mjcarroll commented 9 months ago

Housekeeping to clean up namespaces.

Remove using namespace gz and use nested namespace declarations where possible.

codecov[bot] commented 9 months ago

Codecov Report

Merging #582 (02260dd) into gz-gui8 (c01d327) will decrease coverage by 0.01%. Report is 8 commits behind head on gz-gui8. The diff coverage is 89.88%.

:exclamation: Current head 02260dd differs from pull request most recent head 3290544. Consider uploading reports for the commit 3290544 to get more accurate results

@@             Coverage Diff             @@
##           gz-gui8     #582      +/-   ##
===========================================
- Coverage    68.10%   68.09%   -0.01%     
===========================================
  Files           38       39       +1     
  Lines         5367     5372       +5     
===========================================
+ Hits          3655     3658       +3     
- Misses        1712     1714       +2
Files Coverage Δ
include/gz/gui/GuiEvents.hh 100.00% <100.00%> (ø)
include/gz/gui/MainWindow.hh 100.00% <100.00%> (ø)
include/gz/gui/Plugin.hh 100.00% <100.00%> (ø)
src/Application.cc 84.13% <100.00%> (+0.03%) :arrow_up:
src/Conversions.cc 95.18% <100.00%> (ø)
src/Dialog.cc 100.00% <ø> (ø)
src/DragDropModel.cc 100.00% <ø> (ø)
src/GuiEvents.cc 100.00% <ø> (ø)
src/InstallationDirectories.cc 100.00% <100.00%> (ø)
src/MainWindow.cc 95.68% <ø> (ø)
... and 23 more
mjcarroll commented 9 months ago

@osrf-jenkins retest this please