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

Fixed SPI functions out of scope/Multiple definition errors #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

XsylenZ
Copy link

@XsylenZ XsylenZ commented Jun 18, 2019

We tried to use the BrickPi3 drivers for our C++ project. We had a problem creating a proper object from BrickPi3.cpp. After looking at the header file we found the spi variables and functions defined out of the class definition.

After replacing them everything worked just fine for us, it's running for 2 months without any problems now.

image

@CleoQc CleoQc requested review from CleoQc and mattallen37 and removed request for mattallen37 September 6, 2019 14:46
Copy link
Contributor

@mattallen37 mattallen37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For CleoQc to review.

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.

2 participants