yc/add matrix demo - Adding a server example for MATRIX Creator/Voice boards#69
Open
yoelrc88 wants to merge 3 commits intointrolab:masterfrom
Open
yc/add matrix demo - Adding a server example for MATRIX Creator/Voice boards#69yoelrc88 wants to merge 3 commits intointrolab:masterfrom
yoelrc88 wants to merge 3 commits intointrolab:masterfrom
Conversation
Contributor
yoelrc88
commented
Jun 5, 2018
- added configuration files ready for this example
- added small guide on how to install/run the demo
- added a gif as a reference of what the demo does
Member
|
Thank you for this code! I see you are using json-c to parse from JSON. I try to avoid using external libraries as much as possible and so yesterday I just wrote a json parser to integrate directly in odas. As soon as I have push this in a dev branch, I'll look how I can merge your code to odas. |
Contributor
Author
|
Oh great! As soon as you push that json code I will update this example. |
Member
|
Here is the code for the homemade JSON parser. I'm still working on the server side implementation... I'll merge to master when I'm done and I clean a bit, but you can already have an idea of the way the json parser works here: https://github.com/introlab/odas/blob/dev/src/odas/utils/json.c |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.