Mirror
/
ardupilot
mirror of https://github.com/ArduPilot/ardupilot
5
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

check sizes in VARTest 

ensures the AP types are plain old data
This commit is contained in:
Andrew Tridgell 2012-02-13 10:26:21 +11:00
parent d55a0c3a71
commit ed2271ceea
1 changed files with 33 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
Tools/VARTest/VARTest.pde
View File

@ -135,6 +135,9 @@ void test_variable(AP_Param *ap, enum ap_var_type type)
switch (type) { switch (type) {
case AP_PARAM_INT8: { case AP_PARAM_INT8: {
AP_Int8 *v = (AP_Int8 *)ap; AP_Int8 *v = (AP_Int8 *)ap;
if (sizeof(*v) != 1) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
v->set(value); v->set(value);
if (!v->save()) { if (!v->save()) {
Debug("failed set_and_save"); Debug("failed set_and_save");
@ -168,6 +171,9 @@ void test_variable(AP_Param *ap, enum ap_var_type type)
} }
case AP_PARAM_INT16: { case AP_PARAM_INT16: {
AP_Int16 *v = (AP_Int16 *)ap; AP_Int16 *v = (AP_Int16 *)ap;
if (sizeof(*v) != 2) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
v->set(value); v->set(value);
if (!v->save()) { if (!v->save()) {
Debug("failed set_and_save"); Debug("failed set_and_save");
@ -201,6 +207,9 @@ void test_variable(AP_Param *ap, enum ap_var_type type)
} }
case AP_PARAM_INT32: { case AP_PARAM_INT32: {
AP_Int32 *v = (AP_Int32 *)ap; AP_Int32 *v = (AP_Int32 *)ap;
if (sizeof(*v) != 4) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
v->set(value); v->set(value);
if (!v->save()) { if (!v->save()) {
Debug("failed set_and_save"); Debug("failed set_and_save");
@ -234,6 +243,9 @@ void test_variable(AP_Param *ap, enum ap_var_type type)
} }
case AP_PARAM_FLOAT: { case AP_PARAM_FLOAT: {
AP_Float *v = (AP_Float *)ap; AP_Float *v = (AP_Float *)ap;
if (sizeof(*v) != 4) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
v->set(value); v->set(value);
if (!v->save()) { if (!v->save()) {
Debug("failed set_and_save"); Debug("failed set_and_save");
@ -265,6 +277,27 @@ void test_variable(AP_Param *ap, enum ap_var_type type)
} }
break; break;
} }
case AP_PARAM_VECTOR3F: {
AP_Vector3f *v = (AP_Vector3f *)ap;
if (sizeof(*v) != 12) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
break;
}
case AP_PARAM_VECTOR6F: {
AP_Vector6f *v = (AP_Vector6f *)ap;
if (sizeof(*v) != 24) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
break;
}
case AP_PARAM_MATRIX3F: {
AP_Matrix3f *v = (AP_Matrix3f *)ap;
if (sizeof(*v) != 36) {
Debug("incorrect size %u", (unsigned)sizeof(*v));
}
break;
}
default: default:
break; break;
} }