Kdopen rename library #493
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…te json_tokener_parse to use that instead of json_tokener_parse_ex. Fix a typo in the string represenations of the json_tokener_error_depth error (s/to deep/too deep/)
…on_type_null for NULL objects.
…t to. Change the resize check from "count > size" to "count >= size" to avoid a potential infinite loop with high load factors and a full hash table.
… and add several more cases to check various incremental parsing situations.
…parse multiple objects. Now, json_tokener_reset() does not need to be called after a valid object is parsed.
…nd append things like whitespace indentation.
…easily turned on in the Makefile.
…memset and an off-by-one error in printbuf_memappend.
Changing root name of library to json-c, and also the directory where header files are installed to .../jsdon-c/*. This avoids clashes with other implementations of JSON libraries.
This allows systems certain types of build setups to work. Specifically, this will help when building on Android and using json-c as a dependency for another package.
This is convention in most open source projects.
This dependson the Androgenizer project, which helps keep autofoo-based build systems in sync with the Android build.
(The current implementation will fail when adding index 65.)
…aker with #define HAVE_X where #define HAVE_X 1|0 is meant.
…d #define strcasecmp for Visual C++.
Add missing casts from void* and replace #if HAVE_X with #ifdef HAVE_X
array_list_expand_internal needs length, not index.
…t to fix compilation on Linux with GCC 4.7.1. The issue likely caused by my previous commits related to Visual C++ port of the code.
Added a bunch of missing HAVE_* defines
Android build system
First, we should be able to execute as "./autogen.sh". Second, add support for NOCONFIGURE=1. For more information, see http://people.gnome.org/~walters/docs/build-api.txt
…lready exists so keys remain valid. This is particularly useful when replacing values in a loop, since it allows the key used by json_object_object_foreach to continue to be used.
autogen.sh: Add /bin/sh interpreter, honor NOCONFIGURE=1 …
…ng valid numbers after parsing an out of range number works.
… options are specified.
… by an f, not a literal formfeed) and extend the test_parse test to check all valid escape sequences.
…o kdopen-rename_library Conflicts: .gitignore Makefile.am configure.in printbuf.c test1.c test1.expected
…oduced with the last merge)
…warn_references macro.
…json-c, but emits a warning encouraging the use of the new library.
…eation of the libjson.so library, and only include libjson-c.so
…-linked to use it.
Also rename the json-c-uninstalled.pc file.
…art of the build. I think this should work, but I can't test it.
|
The 0.10 branch is old and unused, and even if it wasn't it doesn't seem to make sense to merge this whole pile of commits into it. Closing w/o merge. |
|
Also, the kdopen-rename_library branch doesn't appear to actually exist with any real changes, so I'm deleting the branch. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#214