search

DccPlusPlus / BaseStation

DCC++ Base Station for Arduino Uno and Mega
207 stars 150 forks source link

Errors verifying/compiling #42

Open DDanhauer opened 5 years ago

DDanhauer commented 5 years ago

I have attempted multiple times to load this on my Uno and Mega. Both show issues with compiling. I am able to load it to either and test it successfully with commands <1> and with lights reacting correctly on the motor shield. I am unable to get JMRI to send commands to the programming or main tracks except to turn on or off power. Here are the compiling errors.

C:\Users\David Danhauer\Documents\Train\BaseStation-master\BaseStation-master\DCCpp_Uno\DCCpp_Uno.ino:427:152: warning: backslash and newline separated by space

 R.currentBit=0;                                       /*   reset current bit pointer and determine which Register and Packet to process next--- */ \   

                                                                                                                                                    ^

C:\Users\David Danhauer\Documents\Train\BaseStation-master\BaseStation-master\DCCpp_Uno\DCCpp_Uno.ino:433:100: warning: backslash and newline separated by space

   R.tempPacket=R.currentReg->activePacket;            /*     flip active and update Packets */ \        

                                                                                                ^

C:\Users\David Danhauer\Documents\Train\BaseStation-master\BaseStation-master\DCCpp_Uno\DCCpp_Uno.ino:449:103: warning: backslash and newline separated by space

} / END-ELSE / \ 

                                                                                                   ^

C:\Users\David Danhauer\Documents\Train\BaseStation-master\BaseStation-master\DCCpp_Uno\DCCpp_Uno.ino:450:88: warning: backslash and newline separated by space

                                                                                    \ 

                                                                                    ^

C:\Users\David Danhauer\Documents\Train\BaseStation-master\BaseStation-master\DCCpp_Uno\DCCpp_Uno.ino:196:59: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]

CurrentMonitor mainMonitor(CURRENT_MONITOR_PIN_MAIN,""); // create monitor for current on Main Track

                                                       ^

C:\Users\David Danhauer\Documents\Train\BaseStation-master\BaseStation-master\DCCpp_Uno\DCCpp_Uno.ino:197:59: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]

CurrentMonitor progMonitor(CURRENT_MONITOR_PIN_PROG,""); // create monitor for current on Program Track

                                                       ^

Sketch uses 18506 bytes (57%) of program storage space. Maximum is 32256 bytes. Global variables use 888 bytes (43%) of dynamic memory, leaving 1160 bytes for local variables. Maximum is 2048 bytes.

positron96 commented 5 years ago

The warnings are not errors. As you see, the firmware fully works. The code should, however, get fixed by someone (but the project seems to be abandoned by author)