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.

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

Verify error #15

Open
ghost opened this issue Sep 15, 2018 · 8 comments · Fixed by #16
Open

Verify error #15

ghost opened this issue Sep 15, 2018 · 8 comments · Fixed by #16

Comments

@ghost
Copy link

ghost commented Sep 15, 2018

Verifying in Arduino 1.8.5 returns;

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

@peternewman
Copy link
Contributor

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 added a commit to peternewman/DmxSerial2 that referenced this issue Sep 16, 2018
Hopefully closes mathertel#15 

Untested on hardware
@peternewman
Copy link
Contributor

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?):
#16

@ghost
Copy link
Author

ghost commented Sep 17, 2018

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

@peternewman
Copy link
Contributor

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

@ghost
Copy link
Author

ghost commented Sep 18, 2018

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

screen shot 2018-09-18 at 21 14 07

@peternewman
Copy link
Contributor

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

@ghost
Copy link
Author

ghost commented Oct 4, 2018

@mathertel Any update on this error?

@peternewman
Copy link
Contributor

Bump @mathertel , can you reopen this please?

@mathertel mathertel reopened this Oct 27, 2018
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 a pull request may close this issue.

2 participants