felixge / node-ar-drone

A node.js client for controlling Parrot AR Drone 2.0 quad-copters.
http://nodecopter.com/
MIT License
1.76k stars 428 forks source link

Calling a direction command with no number arg causes drone crash. #62

Closed sblom closed 11 years ago

sblom commented 11 years ago

If you call, for example, back() (note: no number param), the drone shuts off all of its motors and falls from the sky.

felixge commented 11 years ago

Huh, that's unexpected. Patch would awesome

yocontra commented 11 years ago

Just going to add a quick check for that argument...

yocontra commented 11 years ago

Should probably be expanded to check for NaN after parseFloat is called to make sure we aren't crashing people's drones if they have a typo