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

Skip to content

autotools: allow cross-compilation by honoring --host if given #34

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

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

Conversation

wkz
Copy link
Contributor

@wkz wkz commented Sep 9, 2013

No description provided.

wkz added 2 commits September 9, 2013 14:38
json_object_get: get a value from a key
json_array_for_each: iterate over all values in an array
json_string_get: get the underlying string from a string value
json_object_get_string: get a string from an object key
@jamesamcl
Copy link
Collaborator

I'm happy to accept the autotools changes, but I'm not including high level accessors at this time (the parser is supposed to be an absolute bare minimum API).

I'm considering adding an optional utility header with some of this stuff, so I'll have to get back to you on that...

@wkz
Copy link
Contributor Author

wkz commented Sep 10, 2013

Understandable, how about moving these to a separate c-file/header, and conditionally compiling/installing these based on an --enable-accessors argument given to configure? Would that be a way forward?

@jamesamcl
Copy link
Collaborator

I'm not so worried about conditional compilation - the autotools libjsonparser can include everything. I just like to keep json.c and json.h as minimal as possible.

I think moving this stuff to a json_util.c and json_util.h would be a good way to proceed - that would also give us a place to add some printing and serialization functions later.

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.

2 participants