Update tiny-json.c #15

Open

smdjeff wants to merge 3 commits from smdjeff/master into master

smdjeff commented

2022-07-07 21:46:25 -05:00

(Migrated from github.com)

fix for json_getProperty not finding the object if it's the first element.

Alcantor commented

2023-02-24 06:15:17 -06:00

(Migrated from github.com)

What about: { "propA": { "propA": "testValue"}}

If I want to get the property "propA" of "propA", do I get "testValue" after this patch. I do not have tested it, but I'am a little bit concerned.

By reading the comments, I do understand that the json_getPropertyValue() function search a property into an object and returning the object itself sould not be possible, otherwise we no more searching into it. What about: { "propA": { "propA": "testValue"}} If I want to get the property "propA" of "propA", do I get "testValue" after this patch. I do not have tested it, but I'am a little bit concerned.

This pull request is broken due to missing fork information.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin 'smdjeff/master:smdjeff/master'

git switch 'smdjeff/master'

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch 'master'

git merge --no-ff 'smdjeff/master'

git switch 'smdjeff/master'

git rebase 'master'

git switch 'master'

git merge --ff-only 'smdjeff/master'

git switch 'smdjeff/master'

git rebase 'master'

git switch 'master'

git merge --no-ff 'smdjeff/master'

git switch 'master'

git merge --squash 'smdjeff/master'