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

pkg: add jsmn minimal JSON parser #5202

Merged
merged 2 commits into from
Mar 30, 2016
Merged

Conversation

kaspar030
Copy link
Contributor

From https://github.com/zserge/jsmn:

"Jsmn is a world fastest JSON parser/tokenizer."

edit

The supplied test application is a basically unmodified copy of jsmn's examples/simple.c. I've just added the license, fixed main's prototype, lowered the token number to adapt to RIOT's little stack and run uncrustify over it.

@kaspar030 kaspar030 added the Area: pkg Area: External package ports label Mar 30, 2016
@kaspar030 kaspar030 added this to the Release 2016.04 milestone Mar 30, 2016
@OlegHahm
Copy link
Member

"a world fastest"? lol

@cgundogan
Copy link
Member

nice! works perfectly on master, however I don't get output when trying on an samr21-xpro

@Kijewski
Copy link
Contributor

"a world fastest"

I saw a project a few days ago that claimed to be the fastest JSON tokenizer by eating anything that roughly resembles JSON. :) E.g.

{
    "answer": 42,
    "key"   true_but_actually_only_the_leading_t_is_checked;
    "number" -111.hm.not.really
    fALSE "actually, why should keys be strings?"
]

@kaspar030
Copy link
Contributor Author

however I don't get output when trying on an samr21-xpro

Huh, just tried, works like a charm here. Need to press the reboot button after "make flash term" as the output comes very fast.

@cgundogan
Copy link
Member

Need to press the reboot button after "make flash term" as the output comes very fast.

I forget this every time..

@kaspar030 kaspar030 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 30, 2016
@kaspar030
Copy link
Contributor Author

@cgundogan was that an ACK? :)

@cgundogan
Copy link
Member

nope, will test again now.

@cgundogan
Copy link
Member

Works like charm! ACK

@kaspar030
Copy link
Contributor Author

Thanks for reviewing! &go.

@kaspar030 kaspar030 merged commit dc1616b into RIOT-OS:master Mar 30, 2016
@kaspar030 kaspar030 deleted the pkg_add_jsmn branch March 30, 2016 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants