APM: fixed bug in do_jump() navigation

this avoids a problem where the jump can cause the next command to be
reset to 0

ArduPlane/commands_logic.pde

@@ -11,7 +11,7 @@ handle_process_nav_cmd()
     land_complete = false;
     reset_I();
 
-    gcs_send_text_fmt(PSTR("Executing command ID #%i"),next_nav_command.id);
+    gcs_send_text_fmt(PSTR("Executing nav command ID #%i"),next_nav_command.id);
     switch(next_nav_command.id) {
 
     case MAV_CMD_NAV_TAKEOFF:
@@ -491,33 +491,48 @@ static void do_loiter_at_location()
 
 static void do_jump()
 {
+    if (next_nonnav_command.lat == 0) {
+        // the jump counter has reached zero - ignore
+        gcs_send_text_fmt(PSTR("Jumps left: 0 - skipping"));
+        return;
+    }
+    if (next_nonnav_command.p1 >= g.command_total) {
+        gcs_send_text_fmt(PSTR("Skipping invalid jump to %i"), next_nonnav_command.p1);
+        return;        
+    }
+
     struct Location temp;
+    temp = get_cmd_with_index(g.command_index);
+
     gcs_send_text_fmt(PSTR("Jump to WP %u. Jumps left: %d"),
                       (unsigned)next_nonnav_command.p1,
                       (int)next_nonnav_command.lat);
     if (next_nonnav_command.lat > 0) {
+        // Decrement repeat counter
+        temp.lat                        = next_nonnav_command.lat - 1;                                          
+        set_cmd_with_index(temp, g.command_index);
+    }
+
     nav_command_ID          = NO_COMMAND;
     next_nav_command.id     = NO_COMMAND;
     non_nav_command_ID      = NO_COMMAND;
 
-        temp                            = get_cmd_with_index(g.command_index);
-        temp.lat                        = next_nonnav_command.lat - 1;                                          // Decrement repeat counter
-
-        set_cmd_with_index(temp, g.command_index);
     gcs_send_text_fmt(PSTR("setting command index: %i"), next_nonnav_command.p1 - 1);
     g.command_index.set_and_save(next_nonnav_command.p1 - 1);
     nav_command_index       = next_nonnav_command.p1 - 1;
     next_WP = prev_WP;                      // Need to back "next_WP" up as it was set to the next waypoint following the jump
-        process_next_command();
+
-    } else if (next_nonnav_command.lat == -1) {                                                                 // A repeat count of -1 = repeat forever
+    temp = get_cmd_with_index(g.command_index);
-        nav_command_ID  = NO_COMMAND;
+
-        non_nav_command_ID      = NO_COMMAND;
+    next_nav_command = temp;
-        gcs_send_text_fmt(PSTR("setting command index: %i"),next_nonnav_command.p1 - 1);
+    nav_command_ID = next_nav_command.id;
-        g.command_index.set_and_save(next_nonnav_command.p1 - 1);
+    non_nav_command_index = g.command_index;
-        nav_command_index       = next_nonnav_command.p1 - 1;
+    non_nav_command_ID = WAIT_COMMAND;
-        next_WP = prev_WP;                      // Need to back "next_WP" up as it was set to the next waypoint following the jump
+
-        process_next_command();
+    if (g.log_bitmask & MASK_LOG_CMD) {
+        Log_Write_Cmd(g.command_index, &next_nav_command);
     }
+    process_nav_cmd();
 }
 
 static void do_change_speed()

ArduPlane/commands_process.pde

@@ -50,12 +50,11 @@ static void process_next_command()
     // and loads conditional or immediate commands if applicable
 
     struct Location temp;
-    byte old_index = 0;
+    byte old_index = nav_command_index;
 
     // these are Navigation/Must commands
     // ---------------------------------
     if (nav_command_ID == NO_COMMAND) {    // no current navigation command loaded
-        old_index = nav_command_index;
         temp.id = MAV_CMD_NAV_LAST;
         while(temp.id >= MAV_CMD_NAV_LAST && nav_command_index <= g.command_total) {
             nav_command_index++;