Skip to content

Initialize newResponse manually#288

Merged
PowerBroker2 merged 1 commit intoPowerBroker2:masterfrom
SemvdH:master
Apr 2, 2025
Merged

Initialize newResponse manually#288
PowerBroker2 merged 1 commit intoPowerBroker2:masterfrom
SemvdH:master

Conversation

@SemvdH
Copy link
Contributor

@SemvdH SemvdH commented Apr 2, 2025

I was getting an error when compiling version 3.4.0:

ELMDuino\src\ELMduino.cpp:2613:39: error: variable-sized object 'newResponse' may not be initialized
    char newResponse[PAYLOAD_LEN] = {0};

This is caused by PAYLOAD_LEN not being a compile time constant, so I fixed it by manually initializing the array.

@PowerBroker2 PowerBroker2 merged commit 0af6777 into PowerBroker2:master Apr 2, 2025
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