Skip to content

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

Merged
baylesj merged 11 commits into
open-source-parsers:masterfrom
dota17:code_example
Sep 17, 2019
Merged

Create an example directory and add some code examples.#944
baylesj merged 11 commits into
open-source-parsers:masterfrom
dota17:code_example

Conversation

@dota17

@dota17 dota17 commented Jun 13, 2019

Copy link
Copy Markdown
Member

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.

Comment thread example/stringWrite/stringWrite.cpp Outdated
Comment thread example/stringWrite/stringWrite.cpp Outdated
Comment thread example/streamWrite/streamWrite.cpp Outdated
Comment thread example/streamWrite/streamWrite.cpp Outdated
Comment thread example/CMakeLists.txt Outdated
@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
#
Comment thread CMakeLists.txt Outdated
Comment thread example/readFromStream/readFromStream.cpp Outdated
Comment thread example/readFromStream/readFromStream.cpp Outdated
Comment thread example/readFromStream/readFromStream.cpp Outdated
Comment thread example/readFromStream/readFromStream.cpp Outdated
Comment thread example/readFromString/readFromString.cpp Outdated
Comment thread example/readFromString/readFromString.cpp Outdated
Comment thread example/readFromString/readFromString.cpp Outdated
Comment thread example/streamWrite/streamWrite.cpp Outdated
Comment thread example/streamWrite/streamWrite.cpp Outdated
@dota17

dota17 commented Jul 15, 2019

Copy link
Copy Markdown
Member Author

@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

dota17 commented Aug 27, 2019

Copy link
Copy Markdown
Member Author

@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.

Comment thread example/README.md Outdated
Comment thread example/README.md Outdated
Comment thread example/README.md Outdated
Comment thread example/README.md Outdated
Comment thread example/README.md Outdated
Comment thread example/stringWrite/stringWrite.cpp Outdated
Comment thread example/stringWrite/stringWrite.cpp Outdated
Comment thread example/stringWrite/stringWrite.cpp Outdated
Comment thread example/stringWrite/stringWrite.cpp Outdated
Comment thread src/lib_json/CMakeLists.txt Outdated
@baylesj

baylesj commented Sep 16, 2019

Copy link
Copy Markdown
Contributor

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