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

Support 64bit integer if the compiler support #566

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

xiaoxiang781216
Copy link

@xiaoxiang781216 xiaoxiang781216 commented Mar 30, 2021

64bit support is the hotest topic(#557 #421 #348 #162 #151 #87 #14), this PR try to resolve it in a compatible way:

  1. Auto turn off 64bit if the compiler can't support it
  2. No special function/field is added to support it

@DaveGamble and @FSMaxB do you think this patchset is general enough to merge?

…plication

Change-Id: I99c0aeb6665c73b50147757104ad487404380ea7
Signed-off-by: Xiang Xiao <[email protected]>
Change-Id: I96d3466928361f963600970dd4f5175d5c304f13
Signed-off-by: Xiang Xiao <[email protected]>
@xiaoxiang781216 xiaoxiang781216 changed the title Add cJSON_CreateInt function paired with cJSON_SetIntValue Support 64bit integer if the compiler support Mar 30, 2021
@xiaoxiang781216 xiaoxiang781216 force-pushed the int branch 4 times, most recently from cd5db45 to 3bcdd0d Compare March 30, 2021 19:22
Change-Id: I872888b97dafa72da9d18d65fa59fcbc280ea04c
Signed-off-by: Xiang Xiao <[email protected]>
@xiaoxiang781216 xiaoxiang781216 force-pushed the int branch 3 times, most recently from 2199013 to f3e178f Compare April 8, 2021 06:10
Change-Id: I95458471970a78448f484c8de02ac79632dd229f
Signed-off-by: Xiang Xiao <[email protected]>
@xmm1989218
Copy link

how about this patch? will it merge into master?

@xiaoxiang781216
Copy link
Author

It look like this project stop maintain.:(

@xmm1989218
Copy link

It look like this project stop maintain.:(

did you use this patch at production env?

@xiaoxiang781216
Copy link
Author

Yes.

@dongly
Copy link

dongly commented Aug 21, 2022

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants