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.

Sign up for GitHub

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

Jump to bottom

Fixes to MatrixEsp32Hub75Refresh*Impl.h required for C++20 compilation. #178

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

Fixes to MatrixEsp32Hub75Refresh*Impl.h required for C++20 compilation. #178

wants to merge 1 commit into from

Commits on Nov 29, 2023

  1. Fixes to MatrixEsp32Hub75Refresh*Impl.h required for C++20 compilation. 

    i2s_parallel_config_t is initialized with the C99/GNU designated member
initializers and old-style positional initializations. As of C++20, mixing
like this is no longer allowed.

The error is pretty self-explanatory:
src/MatrixEsp32Hub75Refresh_NT_Impl.h:449:9: error: either all initializer clauses should be designated or none of them should be

Tested working withe ESP32 HUB75 Matrix in NightDriverLED/Mesmerizer.
    @robertlipe
    robertlipe committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    7120359 View commit details
    Browse the repository at this point in the history