Signed-off-by: Roman <bapstroman@gmail.com>

excess

This define results in super unreadable code.

With the new position control architercture manual mode and
potition mode run through the same logic and as a result
it had a throttle thershold for smooth takeoff also in
manual mode. This is fixed by by ignoring the threshold for
any mode which doesn't control the climb rate.

The same paramater was used twice, once for scaling the yaw stick
and once for limitng just after. This was useless.

The FlightTaskManual... tasks already had their internal
setpoint member variables. I switched them to use the
architecture with setpoint member variables as it was
implemented the commit before. They simplify a lot.

I realized that instead of using the setpoint setters inline
in real world tasks everyone started to have its own setpoint
member variable and only call the setter in the end for all the
privatly generate setpoints. This makes the setter useless and
therefore I switched to member setpoints in the architecture.
They bring more felxibility which is obviously needed but also
less structure which is the price to pay.

This commit is only done to enable incremental testing of Flighttask.

instantiate flight tasks and the refactored position controller class
only use the new functionality if a temporary parameter is set for testing

individual commit for the indentation because otherwise
the diff gets unreadable. this indentation is made because
afterwards the entire legacy position control functionality
is in an else case.

and make it use matrix library for flight task compatibility

# Conflicts:
#	src/modules/mc_pos_control/mc_pos_control_main.cpp

# Conflicts:
#	src/modules/mc_pos_control/mc_pos_control_main.cpp

all the ones with mc_pos_control need the library, we should use cmake dependencies!

because upstream there needs to be a
common mavlink command definition first
and then it can be easily reenabled again

This reverts commit cf2d794d.

on vehicle rotation in yaw