-
Notifications
You must be signed in to change notification settings - Fork 81
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 any use of std::vector #52
Comments
It's possible to change std::vector to std::array in the project ? The array template is easier to implement than the vector in embedded systems. |
@patrickelectric It is? |
@patrickelectric Ideally we'd move to c arrays so that Fido could immediately work on AVR |
@truell20 and @joshuagruenstein, take a look here. |
@patrickelectric Thank you for the link! |
@patrickelectric: I still think we should switch to c arrays for a number of reasons.
Basically, it seem to me that using a library like that would be a bit of a "hack." I think switching to C arrays, although a larger process right now, will be beneficial for the future of the Fido Library. Of course, that means we need to get around to actually making the switch. @truell20 and I are both somewhat busy this summer, so we'd appreciate help from contributors such as @FlyingGraysons, @Sydriax, and of course you @patrickelectric. |
Is the goal of this issue still to switch to C arrays? |
@joshuagruenstein, Do you still think about moving to C arrays ? |
Switch to C arrays for speed and availability on microcontrollers
The text was updated successfully, but these errors were encountered: