Fixed in master. You would still
Get warnings if you have settings added twice or names only differ by special characters.
On Sep 27, 2013 1:14 PM, "Chris" <
address@hidden> wrote:
Hi.
I still get a lot of warnings using the latest master (as updated at 14:11 27 Sep 2013) like:
/home/hendrix/paparazzi/var/C182/generated/settings.h:33:26: warning: ISO C99 requires whitespace after the macro name [enabled by default]
/home/hendrix/paparazzi/var/C182/generated/settings.h:34:26: warning: ISO C99 requires whitespace after the macro name [enabled by default]
/home/hendrix/paparazzi/var/C182/generated/settings.h:34:0: warning: "SETTINGS_ap_state" redefined [enabled by default]
which are caused by those settings in the "settings.xml" file as it doesn like the "->" sign
<dl_settings NAME="trim">
<dl_setting MAX="960" MIN="-960" STEP="1" VAR="ap_state->command_roll_trim" shortname="roll_trim" module="inter_mcu" param="COMMAND_ROLL_TRIM"/>
<dl_setting MAX="960" MIN="-960" STEP="1" VAR="ap_state->command_pitch_trim" shortname="pitch_trim" param="COMMAND_PITCH_TRIM"/>
</dl_settings>
Also if i use the "versatile.xml" flight plan i get a lot of warnings because of the naming used for the block names like "%" character (this can be easily avoided by changing the block's name):
<block name="Circle 0% throttle">
<circle pitch="fp_pitch" radius="nav_radius" throttle="0.0" until="ground_alt+50 > GetPosAlt()" vmode="throttle" wp="1"/>
<deroute block="Standby"/>
</block>
Of course i am not in a hurry as i have the flight proven version as a backup :-)
but i thought that it might helpfull to report it.
Chris
_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel