search

BleuLlama / TinyBasicPlus

A C implementation of Tiny Basic, with a focus on support for Arduino
401 stars 117 forks source link

Fix for newer arduino version #14

Closed imchristina closed 7 years ago

bgolab commented 8 years ago

I tried arduino 1.6.5: Arduino: 1.6.5 (Windows 7), Board: "Arduino Uno"

Using library EEPROM in folder: C:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\libraries\EEPROM

Using library SD in folder: C:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\libraries\SD

Using library SPI in folder: C:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\libraries\SPI

C:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\tools\avr/bin/avr-g++ -c -g -Os -w -fno-exceptions -ffunction-sections -fdata-sections -fno-threadsafe-statics -MMD -mmcu=atmega328p -DF_CPU=16000000L -DARDUINO=10605 -DARDUINO_AVR_UNO -DARDUINO_ARCH_AVR -IC:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\cores\arduino -IC:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\variants\standard -IC:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\libraries\EEPROM -IC:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\libraries\SD\src -IC:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\libraries\SPI C:\Users\abg015\AppData\Local\Temp\build8011603945389636597.tmp\TinyBasicPlus.cpp -o C:\Users\abg015\AppData\Local\Temp\build8011603945389636597.tmp\TinyBasicPlus.cpp.o

In file included from C:\Users\abg015\Desktop\news\tiny2\bin\arduino-1.6.5\hardware\arduino\avr\cores\arduino/Arduino.h:28:0, from TinyBasicPlus.ino:81: TinyBasicPlus.ino:278:33: error: variable 'keywords' must be const in order to be put into read-only section by means of 'attribute((progmem))' TinyBasicPlus.ino:370:33: error: variable 'func_tab' must be const in order to be put into read-only section by means of 'attribute((progmem))' TinyBasicPlus.ino:385:31: error: variable 'to_tab' must be const in order to be put into read-only section by means of 'attribute((progmem))' TinyBasicPlus.ino:390:33: error: variable 'step_tab' must be const in order to be put into read-only section by means of 'attribute((progmem))' TinyBasicPlus.ino:395:34: error: variable 'relop_tab' must be const in order to be put into read-only section by means of 'attribute((progmem))' TinyBasicPlus.ino:415:36: error: variable 'highlow_tab' must be const in order to be put into read-only section by means of 'attribute((progmem))' variable 'keywords' must be const in order to be put into read-only section by means of 'attribute((progmem))'

imchristina commented 8 years ago

It compiles & runs fine on both my Due and my Uno. I'm on Linux running 1.6.5.

bgolab commented 8 years ago

I tried arduino 1.6.5 for Windows. Uno board was selected.

Thanks,Bogdan

  From: nutcase84 <notifications@github.com>

To: BleuLlama/TinyBasicPlus TinyBasicPlus@noreply.github.com Cc: bgolab bgolab@yahoo.com Sent: Sunday, September 20, 2015 7:54 PM Subject: Re: [TinyBasicPlus] Fix for newer arduino version (#14)

It compiles & runs fine on both my Due and my Uno. I'm on Linux running 1.6.5.— Reply to this email directly or view it on GitHub.

imchristina commented 8 years ago

The errors that you are receiving are exactly the same errors as the unmodified version without my commit. Are you sure you are using my commit?

bgolab commented 8 years ago

Yes, you are right. I took master copy instead of patched file. It compiles without errors!

I am sorry for not testing the right build.

Unfortunately there is very little memory available for variables: "Global variables use 2,003 bytes (97%) of dynamic memory, leaving 45 bytes for local variables. Maximum is 2,048 bytes. Low memory available, stability problems may occur."

imchristina commented 8 years ago

Low memory doesn't seem to be a problem in my testing. I think it reserves variable space, as it says 1024 bytes free on my Uno on startup.