Minor stylistic fixes in vectornav and nps (#2321)

conf/modules/ins_vectornav.xml

@@ -51,10 +51,4 @@
       endif
     </raw>
   </makefile>
-
-  <!-- TODO: make this dynamically loadable - will hopefully work better
-  once nps and HITL are separate targets -->
-  <!--makefile target="hitl">
-    <file name="nps_ins_vectornav.c" dir="nps"/>
-  </makefile-->
 </module>

sw/airborne/subsystems/ins/ins_vectornav.c

@@ -335,7 +335,6 @@ void ins_vectornav_propagate()
     gps.last_3dfix_ticks = sys_time.nb_sec_rem;
   }
 
-
   // update internal states for telemetry purposes
   // TODO: directly convert vectornav output instead of using state interface
   // to support multiple INS running at the same time

sw/simulator/nps/nps_autopilot_fixedwing.c

@@ -157,8 +157,8 @@ void nps_autopilot_run_step(double time)
     float dist = (float) sensors.sonar.value;
     AbiSendMsgAGL(AGL_SONAR_NPS_ID, dist);
 
-    uint16_t foo = 0;
 #ifdef SENSOR_SYNC_SEND_SONAR
+    uint16_t foo = 0;
     DOWNLINK_SEND_SONAR(DefaultChannel, DefaultDevice, &foo, &dist);
 #endif