mathertel / DmxSerial2

An Arduino library for sending and receiving DMX RDM packets.
BSD 3-Clause "New" or "Revised" License
99 stars 30 forks source link

Verify error #15

Open ghost opened 6 years ago

ghost commented 6 years ago

Verifying in Arduino 1.8.5 returns;

RDMSerialRecv:67: error: too many initializers for 'RDMINIT' };

peternewman commented 6 years ago

Was this on the code downloaded from GitHub @claysam75 ? The number of arguments seems right to me by observation, but I've not tried a verify, so there may be something we're doing that's upsetting it.

peternewman commented 6 years ago

I get a different error on 1.8.2:

/tmp/arduino_modified_sketch_43130/RDMSerialRecv.ino:67:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 };
 ^
/tmp/arduino_modified_sketch_43130/RDMSerialRecv.ino:67:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]

This PR fixes that (I don't know if it fixes your issue too?): https://github.com/mathertel/DmxSerial2/pull/16

ghost commented 6 years ago

Hi, I applied your fix but the issue persists. Now running 1.8.7

peternewman commented 6 years ago

Hmm, is there anything else logged, or just that one line?

ghost commented 6 years ago

Screenshot of error output is attached (with just my google drive email covered)

screen shot 2018-09-18 at 21 14 07
peternewman commented 6 years ago

@mathertel this needs re-opening as #15 doesn't actually fix it unfortunately.

ghost commented 6 years ago

@mathertel Any update on this error?

peternewman commented 6 years ago

Bump @mathertel , can you reopen this please?