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 json library (IDFGH-4532) #6356

Closed
joelguittet opened this issue Jan 4, 2021 · 2 comments
Closed

Upgrade json library (IDFGH-4532) #6356

joelguittet opened this issue Jan 4, 2021 · 2 comments
Labels
Type: Feature Request Feature request for IDF

Comments

@joelguittet
Copy link

Is your feature request related to a problem? Please describe.

I'm working on an application depending of cJSON.
The current ESP-IDF is fetching cJSON 1.7.12, but I need a later version because new API fonctions are available ("cJSON_GetNumberValue" in my example).

Describe the solution you'd like

Upgrade the library to the latest release which is v1.7.14 (instead of v1.7.12).

Describe alternatives you've considered

Forking ESP-IDF in my github and have the modification locally. I prefer this to be supported in the mainline (and it will benefit other people).

Additional context

Nothing more.

@joelguittet joelguittet added the Type: Feature Request Feature request for IDF label Jan 4, 2021
@github-actions github-actions bot changed the title Upgrade json library Upgrade json library (IDFGH-4532) Jan 4, 2021
@Alvin1Zhang
Copy link
Collaborator

Thanks for raising this feature request, thanks.

@AxelLin
Copy link
Contributor

AxelLin commented Feb 1, 2021

Hi,
There are a couple null pointer dereference fixes after v1.7.14.
I think it worth update for the cjson component, please check below in cJson git tree:
$ git shortlog --no-merges 8e84db4c4e631c5578a1d2365bbcc9d814ab8a51..HEAD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Request Feature request for IDF
Projects
None yet
Development

No branches or pull requests

3 participants