Skudak/stm32-openinverter-teslacharger
2
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix: drop extern "C" on main to clear -Wpedantic warning (-S4) #6

Merged
bastian merged 1 commits from fix/main-extern-c-pedantic-warning into main 2026-05-12 15:41:44 -04:00
Conversation 0 Commits 1 Files Changed 3
+8 -3
3 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.gitignore
+1
View File
@@ -4,3 +4,4 @@ stm32_charger*
linker.map linker.map
*.layout *.layout
*.out *.out
.DS_Store
include/param_prj.h
+2 -2
View File
@@ -126,9 +126,9 @@
#define CAT_COMM "Communication" #define CAT_COMM "Communication"
// SKUDAK customization suffix: bump on every Skudak-side change so the OpenInverter // SKUDAK customization suffix: bump on every Skudak-side change so the OpenInverter
// web UI shows a distinct version (e.g. "4=1.20.R-S3") and we can visually confirm // web UI shows a distinct version (e.g. "4=1.20.R-S4") and we can visually confirm
// the right firmware is flashed. Match the stm32-sine -S<N> convention. // the right firmware is flashed. Match the stm32-sine -S<N> convention.
#define VERSTR STRINGIFY(4=VER-S3) #define VERSTR STRINGIFY(4=VER-S4)
/***** enums ******/ /***** enums ******/
src/main.cpp
+5 -1
View File
@@ -158,7 +158,11 @@ extern "C" void tim2_isr(void)
scheduler->Run(); scheduler->Run();
} }
extern "C" int main(void) // C++ forbids a linkage specification on `main` (-Wpedantic). The reset_handler
// in libopencm3's vector.c forward-declares `int main(void)` with default
// linkage and resolves it at link time by symbol name, so dropping `extern "C"`
// here doesn't affect how main is invoked at boot.
int main(void)
{ {
extern const TERM_CMD termCmds[]; extern const TERM_CMD termCmds[];