misleading / wrong comment in cpp file [imported]

[cmaglie]

This is Issue 786 moved from a Google Code project.
Added by 2012-01-11T08:22:22.000Z by seb.ha...@gmail.com.
Please review that bug for more context and additional comments, but update this bug.

Original labels: Type-Defect, Priority-Medium

Original description

in function
void FirmataClass::processInput(void)
there is this code:
case SET_PIN_MODE:
waitForData = 2; // two data bytes needed
executeMultiByteCommand = command;
break;
case REPORT_ANALOG:
case REPORT_DIGITAL:
waitForData = 1; // two data bytes needed
executeMultiByteCommand = command;
break;

I suppose the second comment should read:
// one data byte needed

right ?

[ffissore]

fixed in next 1.5.x release

[soundanalogous]

Changes should not be made to Firmata in this repo. The Firmata repo is hosted separately: https://github.com/firmata/arduino. Maybe Arduino should include Firmata as a submodule or otherwise indicate that the source is hosted separately. I'll reflect these changes in the Firmata repo.