Bug at flight_control_sample.cpp

[chocksmith]

Hello,

This is the code that checks if the motor has started:

            while ((vehicle->broadcast->getStatus().flight
			!= DJI::OSDK::VehicleStatus::M100FlightStatus::TAKEOFF)
			&& motorsNotStarted < timeoutCycles) {
		motorsNotStarted++;
		usleep(100000);
	}

	if (motorsNotStarted == timeoutCycles) {
		std::cout << "Successful TakeOff!" << std::endl;
	}

The code above works, but it is not correct. The correct would be something like:

           while ((vehicle->broadcast->getStatus().flight
			< DJI::OSDK::VehicleStatus::M100FlightStatus::TAKEOFF)
			&& motorsNotStarted < timeoutCycles) {
		motorsNotStarted++;
		usleep(100000);
	}

	if (motorsNotStarted < timeoutCycles) {
		std::cout << "Successful TakeOff!" << std::endl;
	}

Same mistake happens on others subsequent checks.

[rohitsantdji]

Good catch! Can you submit a PR?

[chocksmith]

I will! Tks!

…

On 17 Oct 2017 18:28 -0200, Rohit Sant ***@***.***>, wrote: Good catch! Can you submit a PR? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.