Skip to content
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

Remove boost. Use header only libraries for logging, testing #380 #382

Merged
5 commits merged into from Feb 16, 2017
Merged

Remove boost. Use header only libraries for logging, testing #380 #382

5 commits merged into from Feb 16, 2017

Conversation

ghost
Copy link

@ghost ghost commented Feb 16, 2017

  • Use spdlog library (no install, header only) for logging
  • Introduced YLOG_TRACE, YLOG_DEBUG, YLOG_ERROR etc macros to simplify ydk logging
  • Use catch library (no install, header only) for testing
  • Remove top entity lookup from codec provider
  • Revamp entity lookup for codec decode
  • Update generated model tests

Also, includes Xiaoqin's changes to ydk-cpp for using smart pointers instead of raw pointers

psykokwak4 and others added 5 commits February 13, 2017 15:16
Use smart pointer

Update Rpc/RpcImpl

Use smart pointer

Update DataNode/Rpc

In source files for service/provider/path api and tests.

Update DataNode/Rpc

In source files for service/provider/path api and tests.
Update DataNode/Rpc pointer on restconf and test cases.
Replace return type for create_root_schema as unique_ptr
 * Use spdlog library for logging
 * Introduced YLOG_TRACE, YLOG_DEBUG, YLOG_ERROR etc macros to simplify ydk logging
 * Use catch library for testing
 * Remove top entity lookup from codec provider
@ghost ghost added the in progress label Feb 16, 2017
@ghost ghost merged commit 4a08c59 into CiscoDevNet:master Feb 16, 2017
@ghost ghost removed the in progress label Feb 16, 2017
This pull request was closed.
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.

1 participant