Skip to content

Create an example directory and add some code examples. #944

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 11 commits into from
Sep 17, 2019

Conversation

dota17
Copy link
Member

@dota17 dota17 commented Jun 13, 2019

Personally speaking, I think the original code examples for this project are not very easy to find, and the readability is not very high,so I create a example directory and add some code examples here.

@dota17 dota17 force-pushed the code_example branch 2 times, most recently from 75313b8 to 9de4234 Compare June 26, 2019 11:13
…for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
#
# Date: Wed Jul 10 21:26:16 2019 +0800
#
# On branch code_example
# Your branch is up-to-date with 'origin/code_example'.
#
# Changes to be committed:
#	modified: example/CMakeLists.txt
#
@dota17
Copy link
Member Author

dota17 commented Jul 15, 2019

@baylesj @BillyDonahue Thank you all for all your precious suggests, and now the problem is that the build failed for undefined reference to XXX, seems that it can't find the header file, I guess it blames to CMakeLists.txt file, I am not very proficient in its usage. can you give me some suggests on example/CMakeLists.txt file?

@dota17
Copy link
Member Author

dota17 commented Aug 27, 2019

@baylesj @BillyDonahue I have modified all change requests, and if there is any thing else to be done, please point it out.
If this PR is not so necessary, then I can close it because it has been submitted for a long time.

@baylesj
Copy link
Contributor

baylesj commented Sep 16, 2019

I am glad you did this patch, I definitely want example code for this project. If you can fix up the errors I'm happy to merge it in.

@dota17 dota17 requested a review from baylesj September 17, 2019 12:13
@baylesj baylesj merged commit e9ccbe0 into open-source-parsers:master Sep 17, 2019
dawesc pushed a commit to EFTlab/jsoncpp that referenced this pull request Sep 25, 2019
…parsers#944)

* update example directory

* modify some compile error.

* update with clang-format

* update

* update

* add_definitions("../include/json")

# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
#
# Date:      Wed Jul 10 21:26:16 2019 +0800
#
# On branch code_example
# Your branch is up-to-date with 'origin/code_example'.
#
# Changes to be committed:
#	modified:   example/CMakeLists.txt
#

* change CMakeLists.txt

* update streamWrite.cpp

* update

* Update readFromStream.cpp

* fix typo
@dota17 dota17 deleted the code_example branch October 8, 2019 13:10
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.

4 participants