Hi folks at Makeblock,
Hi @peterlee and @Pioneer
Great to see the new makeblock library V3.0 for Arduino IDE appears on GitHub. I have installed the new library (version as of 05 Nov 2015) with the latest version of Arduino IDE (1.6.6) and it works fine in general. Please see below some issues I found.
-
Uploading Sketches
to Orion board is impossible when Bluetooth Module is connected to the Orion board. This has been described in a separate thread. Please fix this, as it is most annoying and creates endless confusion! -
Wrong include directive to “arduino.h” in all header files
All header files in libraries/makeblock/src/ contain an error, as they attempt to include the library “arduino.h”, which does not exist. The correct include statement would be #include “Arduino.h”. This causes compilation errors with all examples and all sketches written by your users.
LINUX is case-sensitive, thus the capital “A” matters. This should be no issue on MS-Windows when changed, as Windumb is not case-sensitive. (For those who have experienced the problem: It can temporarily be fixed by creating a copy of arduino-1.6.6/hardware/arduino/avr/cores/arduino/Arduino.h and renaming it arduino.h.) -
Warning when compiling MeOrion.h
The following warning is given when compiling a sketch which includes MeOrion.h:
arduino-1.6.6/libraries/makeblock/src/MeOrion.h:77:1: warning: narrowing conversion of ‘-1’ from ‘int’ to ‘uint8_t {aka unsigned char}’ inside { } [-Wnarrowing]
};
^
I hope this input helps you to improve the library so that it also works with LINUX installations of the Arduino IDE.
Cheers, Stefanstrong text