Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade ArduinoJson 6 to 7 #1538

Closed
proddy opened this issue Jan 4, 2024 · 2 comments
Closed

Upgrade ArduinoJson 6 to 7 #1538

proddy opened this issue Jan 4, 2024 · 2 comments
Labels
technical Technical enhancement, or tech-debt issue
Milestone

Comments

@proddy
Copy link
Contributor

proddy commented Jan 4, 2024

ArduinoJson 7 is out. Would be interesting to test both overall performance and memory usage. Now that it uses dynamic memory allocation I'm curious what happens with mem fragmentation (Max Alloc).

https://arduinojson.org/v7/how-to/upgrade-from-v6/

@proddy proddy added the technical Technical enhancement, or tech-debt issue label Jan 4, 2024
@proddy proddy added this to the v3.7.0 milestone Jan 4, 2024
@proddy
Copy link
Contributor Author

proddy commented Jan 5, 2024

Added to 3.6.5-dev.8

@proddy proddy modified the milestones: v3.7.0, v3.6.5 Jan 5, 2024
@proddy proddy closed this as completed Jan 5, 2024
proddy added a commit that referenced this issue Jan 5, 2024
ArduinoJson version 7.0.0 - #1538
@proddy
Copy link
Contributor Author

proddy commented Jan 8, 2024

I also removed all the & references from JsonObject and JsonArray as there are already semantically references to JsonDocument. And added JsonObjectConst where necessary. It doesn't affect the code size or performance, perhaps prevent dangling references if they ever happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical Technical enhancement, or tech-debt issue
Projects
None yet
Development

No branches or pull requests

1 participant