Open
Conversation
Owner
|
I have no idea how I am just now seeing this, but this is awesome! (disclaimer: C is not my strong point, I get by, and am learning!) I just pulled these additions and received an error on the index() function. I tried to track that down and so far what I have found is it is in a "strings.h" but not "string.h" as included. I looked for any reference to that function in arduino and so far coming up empty. Do you have any reference for that in arduino code? I can keep looking, but wanted to check with you first. Thanks, |
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
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.
Hi Kaleb,
I've implemented a few calculators in my day, so I thought I'd help a Maker out.
There's also a test, that should run in a Linux or mac command line by cd'ing to the test directory, then make and ./testCalculator. Since I don't have your Arduino setup I couldn't test it in situ, but it should work with minimal modification.