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

Replace C-like arrays with std::array #468

Open
SSoelvsten opened this issue Mar 15, 2023 · 0 comments
Open

Replace C-like arrays with std::array #468

SSoelvsten opened this issue Mar 15, 2023 · 0 comments
Labels
✨ code quality Uncle Bob would be proud 📁 internal This is where the ✨magic✨happens 🎓 student programmer Work, work...

Comments

@SSoelvsten
Copy link
Owner

SSoelvsten commented Mar 15, 2023

As part of #467 , it turns out that the MSVC compiler does not at all like references to C-like arrays. Furthermore, one could argue it is prettier to stick to the C++ way rather than doing half-C some of the times. Hence, we should replace all of these with std::array instead.

@SSoelvsten SSoelvsten added ✨ code quality Uncle Bob would be proud 📁 internal This is where the ✨magic✨happens 🎓 student programmer Work, work... labels Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ code quality Uncle Bob would be proud 📁 internal This is where the ✨magic✨happens 🎓 student programmer Work, work...
Projects
None yet
Development

No branches or pull requests

1 participant