Removed completely unused variables. Ifdef’ed out variable which is
referenced from commented out code.

Fixes Issue #542

This patch fixes a bug that display 0 for distances to waypoint more
than 10km.

Besides that, it also homogenize the naming of the label to
"distToWaypoint" instead of "distToWP". Old configuration files (in
home directory), should be hacked or removed in order to see this
change.

Signed-off-by: Paul Chavent <paul.chavent@onera.fr>

MainWindow::loadDataView has been commented out for 9 months now.
Removed commented out code instead of leaving cruft in.

showLabel was connected to signal but not set as a slot

There is no such things as a clicked signal on QWidget. You activate
new UAS from right-click menu.

ui->waitingLabel is deleted and set to NULL when mode is PX4. Must
check for NULL before using it.

The connect call was generating debug output to identify that it was
failing. After looking at the code realized that this connect is not
needed. The group box is removed when UASListWidget::removeUAS method
is called.

Second change in the file fixes a crashing bug when the last UAS is
removed. Need to exit the iterator after we modify it.

Right-click menu works cleaner.
Coloring of output text is correct.
Output text now handles all severities of messages.

Use a pragma around our use of that header to silence the warning. OSG
3.01 has these warnings, 3.2 has fixed them, but 3.2 doesn’t isn’t the
widely used release yet. So at some point when we upgrade to 3.2 this
will no longer be needed.

Disable the active WP setting in the edit list, as its super error prone - only allow it in the view list and switch between views based on activity on the system / UI

Use a pragma around our use of that header to silence the warning. OSG
3.01 has these warnings, 3.2 has fixed them, but 3.2 doesn’t isn’t the
widely used release yet. So at some point when we upgrade to 3.2 this
will no longer be needed.