juliancheal / LPD8806-Firmata

Library for using LPD8806 Digital RGB LED with Firmata
MIT License
2 stars 0 forks source link

ino file problems #1

Open basvaneijk opened 10 years ago

basvaneijk commented 10 years ago

Hi,

I would love to see this firmata patch to work together with openframeworks. However it isn't yet. Btw I'm on the dev branch (since I thought it was the latest that had updates), using a mac 10.9.3, arduino 1.5 and a Arduino Diecimila.

When I upload the files to my library and open up the firmata patch, my console tells me it can't find the h files:

In file included from ldp8806_firmata.ino:3: /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:19:21: error: > LPD8806.h: No such file or directory

In file included from ldp8806_firmata.ino:3: /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:27: error: expected > constructor, destructor, or type conversion before ';' token /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:28: error: expected > constructor, destructor, or type conversion before ';' token /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:29: error: expected > constructor, destructor, or type conversion before ';' token /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:30: error: expected > constructor, destructor, or type conversion before ';' token ldp8806_firmata.ino: In function 'void firmataStringCallback(char*)': ldp8806_firmata:39: error: 'parse_message' was not declared in this scope

So I changed the wrapper file line 19 into

include

After that my console tells me:

In file included from ldp8806_firmata.ino:3: /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:27: error: expected > constructor, destructor, or type conversion before ';' token /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:28: error: expected > constructor, destructor, or type conversion before ';' token /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:29: error: expected > constructor, destructor, or type conversion before ';' token /Volumes/Macintosh HD/Users/SJ/Documents/Arduino/libraries/ldp8806_wrapper/LPD8806_Wrapper.h:30: error: expected > constructor, destructor, or type conversion before ';' token

ldp8806_firmata.ino: In function 'void firmataStringCallback(char*)': ldp8806_firmata:39: error: 'parse_message' was not declared in this scope

basvaneijk commented 10 years ago

Not sure if this is the correct way, but changing this at line 27, 28, 29, 30 in LPD8806_Wrapper.h is helping reduce the errors in the console

void lpd8806_initialise(); void parse_message(String& message, int message_start); void setPixelColor(uint16_t n, uint8_t r, uint8_t g, uint8_t b); void setPixelColor(uint16_t n, uint32_t c); void show(void);

So now the error console gives:

ldp8806_firmata.ino: In function 'void firmataInitialize()': ldp8806_firmata:28: error: 'firmataSysexCallback' was not declared in this scope

So I see the firmataSysexCallback() is missing in the dev branch, so I copy pasted it from the master and all errors are gone now.