Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix Pebble compilation (#49) #57

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

Merged
merged 1 commit into from
Aug 16, 2021
Merged

Fix Pebble compilation (#49) #57

merged 1 commit into from
Aug 16, 2021

Conversation

zackp30
Copy link
Contributor

@zackp30 zackp30 commented May 4, 2016

No description provided.

@LB--
Copy link
Member

LB-- commented May 4, 2016

It looks like your editor changed the indentation levels.

@zackp30
Copy link
Contributor Author

zackp30 commented May 5, 2016

Whoops, thank you! Fixing now.

@zackp30
Copy link
Contributor Author

zackp30 commented May 5, 2016

Fixed!

@LB--
Copy link
Member

LB-- commented May 5, 2016

@zackp30 there's still a couple spaces missing from line 50 ;)

@zackp30
Copy link
Contributor Author

zackp30 commented May 5, 2016

Haha, thanks! Fixed. :)

@LB-- LB-- added this to the v1.1.1 milestone Aug 14, 2021
@LB-- LB-- linked an issue Aug 16, 2021 that may be closed by this pull request
@LB-- LB-- merged commit b695e3d into json-parser:master Aug 16, 2021
DimitriPapadopoulos added a commit to DimitriPapadopoulos/json-parser that referenced this pull request Aug 18, 2021
@DimitriPapadopoulos
Copy link
Contributor

isdigit expects int, not unsigned char. See #120 for a variant.

DimitriPapadopoulos added a commit to DimitriPapadopoulos/json-parser that referenced this pull request Aug 19, 2021
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.

Portability issue on pebble
3 participants