search

mikaelpatel / Arduino-Shell

RPN Postscript/Forth Command Shell for Arduino
19 stars 5 forks source link

Bug: ShellDemo doesn't appear to work. #30

Closed dpharris closed 8 years ago

dpharris commented 8 years ago

The other demos appear to work fine.

mikaelpatel commented 8 years ago

Do you have any more info? Does not compile?

dpharris commented 8 years ago

Yes, compiles, prints welcome. Seems to fail on the iom.read() step. Never makes it to the (added) print(bp): void loop() { Serial.println("."); // works (added) if (shell.read(bp)) { Serial.print("==>"); Serial.println(bp); // never gets here (added) shell.execute(buf); bp = buf; } delay(500); // (added) }

mikaelpatel commented 8 years ago

What board are you using? I tested on Uno and Mega without any problem.

mikaelpatel commented 8 years ago

Should try Serial.println(buf) instead. bp points to the end of the input line (0).

mikaelpatel commented 8 years ago

Serial Monitor must be in [Newline] mode.

dpharris commented 8 years ago

Oh, I think I turned that off .....

On Sun, Mar 6, 2016 at 1:34 PM, Mikael Patel notifications@github.com wrote:

Serial Monitor must be in [Newline] mode.

— Reply to this email directly or view it on GitHub https://github.com/mikaelpatel/Arduino-Shell/issues/30#issuecomment-193001684 .

dpharris commented 8 years ago

Yes, that was the problem! Thanks ... will need to note that.
(( I had forgotten I had disabled it for another program I was working on)). David

mikaelpatel commented 8 years ago

Added some startup info about the [Newline] mode in the example sketches that use read().