Open-Agriculture / AgIsoStack-plus-plus

AgIsoStack++ is the completely free open-source C++ ISOBUS library for everyone
https://agisostack.com/
MIT License
167 stars 40 forks source link

[TC]: Allow disabling the data dictionary to save on binary/RAM size #460

Closed ad3154 closed 2 months ago

ad3154 commented 3 months ago

Describe your changes

This allows you to disable the ISOBUS data dictionary if you want to save a bunch of ROM/RAM space in your binary. This was very helpful on the STM32 platform I'm testing on.

How has this been tested?

Unit tests which use the data dictionary interface still compile with the database disabled.

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

ad3154 commented 2 months ago

Yes I agree, we should explore centralizing our config in a way that's more cross-platform friendly.

My initial thought is to explore one main config file with #defines that CMake edits on CMake builds, or can be manually edited otherwise, but needs some thought. I will create an issue for it.