Compiler warning

[stumunro74]

Describe the bug
Compiler warning when enabled.

C:\Users\Stuart\Documents\Arduino\libraries\ELMDuino\src\ELMduino.cpp: In member function 'bool ELM327::initializeELM(char)':
C:\Users\Stuart\Documents\Arduino\libraries\ELMDuino\src\ELMduino.cpp:116:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
     if (sendCommand(command) == ELM_SUCCESS)
     ^~
C:\Users\Stuart\Documents\Arduino\libraries\ELMDuino\src\ELMduino.cpp:120:2: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
  return connected;
  ^~~~~~

To Reproduce
Enable compiler warnings.

Expected behavior
No compiler warnings.

Code

    if (sendCommand(command) == ELM_SUCCESS)
        if (strstr(payload, "OK") != NULL)
            connected = true;

	return connected;

Troubleshooting Connection Issues

• Try 38400 baud instead of 115200
• If using an HC-05 for communication, ensure it's configured properly
• Ensure the car is on and running during troubleshooting

[PowerBroker2]

If you change the offending lines to the following, do you still get the error?

    if (sendCommand(command) == ELM_SUCCESS)
        if (strstr(payload, "OK") != NULL)
            connected = true;

    return connected;

[stumunro74]

The issue was with the following line.

return connected;

Changing the tab to spaces removes the warning.

return connected;