Mirror/ardupilot
5
0
Fork 0
mirror of https://github.com/ArduPilot/ardupilot synced 2025-02-21 23:33:57 -04:00
Code Issues Packages Projects Releases Wiki Activity

AP_Scripting: slightly improve codegen

Browse Source 
Get singleton pointer closer to where it is used.

Saves ~170B.
This commit is contained in:
Thomas Watson 2024-06-30 15:27:12 -05:00 committed by Andrew Tridgell
parent 60a9f1722c
commit 7212e35fb0
1 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
libraries/AP_Scripting/generator/src/main.c
View File

@ -1902,11 +1902,6 @@ void emit_userdata_method(const struct userdata *data, const struct method *meth
// emit comments on expected arg/type
struct argument *arg = method->arguments;
if ((data->ud_type == UD_SINGLETON) && !(data->flags & UD_FLAG_LITERAL)) {
// fetch and check the singleton pointer
fprintf(source, " %s * ud = check_%s(L);\n", data->name, data->sanatized_name);
}
// emit warning if configured
if (method->deprecate != NULL) {
fprintf(source, " static bool warned = false;\n");
@ -1916,7 +1911,6 @@ void emit_userdata_method(const struct userdata *data, const struct method *meth
fprintf(source, " }\n\n");
}
// sanity check number of args called with
arg_count = 1;
while (arg != NULL) {
@ -1933,8 +1927,12 @@ void emit_userdata_method(const struct userdata *data, const struct method *meth
fprintf(source, " %s * ud = check_%s(L, 1);\n", data->name, data->sanatized_name);
break;
case UD_SINGLETON:
if (!(data->flags & UD_FLAG_LITERAL)) {
// fetch and check the singleton pointer
fprintf(source, " %s * ud = check_%s(L);\n", data->name, data->sanatized_name);
}
break;
case UD_GLOBAL:
// this was bound early
break;
case UD_AP_OBJECT:
// extract the userdata, it was a pointer, so we need to grab it