~rickcarlino

#92 Deduplication of d:source data 4 months ago

Comment by ~rickcarlino on ~crc_/retroforth

Any idea on how you would do this? I've thought about how this might work also. Maybe a dictionary entry that contains the string and then use a content hash in the header that points to a string lookup table? I don't have enough knowledge of Retro internals to say for sure but am curious.

#87 Edge case with new Unu warning 4 months ago

Ticket created by ~rickcarlino on ~crc_/retroforth

#Description of Problem

An error message was recently added to assist new users:

warning: no code blocks found!
         filename: foo.retro
         see http://unu.retroforth.org for a brief summary of
         the unu code format used by retro

If the user creates a file that is 100% unit tests, they will erroneously be presented the warning above. This does not stop the program from working, but the behavior might be confusing.

#Steps to Reproduce

  1. Create a file, foo.retro, which only contains triple backtick blocks.
  2. Run the file via retro -t foo.md
  3. Observe the error described above.

#72 `make` fails for nga-c-native-x86 1 year, 6 months ago

Ticket created by ~rickcarlino on ~crc_/retroforth

After running make, the following error was observed:

cp ../../ngaImage nativeImage
../../bin/retro-extend nativeImage x86/common.retro x86/cmos-rtc.retro x86/serial.retro x86/display.retro x86/ata.retro x86/listener.retro
Initial Image Size: 9914
s:index-of ?
   + 220 tokens from x86/common.retro
   + 59 tokens from x86/cmos-rtc.retro
   + 186 tokens from x86/serial.retro
0xB8000 ?
   + 517 tokens from x86/display.retro
   + 323 tokens from x86/ata.retro
   + 252 tokens from x86/listener.retro
New Image Size: 10814
MAX SP: 16, RP: 55
Stack not empty!

Stack: 
../../bin/retro-embedimage nativeImage >image.c
cc -fno-pie -Wall -m32 -DTARGET_X86              -c retro.c -o retro_qwerty.o
retro.c:7:10: fatal error: sys/limits.h: No such file or directory
    7 | #include <sys/limits.h>
      |          ^~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:7: native386] Error 1

#69 `-i -t` allows invalid filepaths 1 year, 7 months ago

Ticket created by ~rickcarlino on ~crc_/retroforth

#Steps to Reproduce

  1. Run retro -i -t foo.retro (assuming this file does not exist)
  2. Observe that retro goes into the REPL, but does not crash or emit warnings about the fact that the file is missing.

#Expected Behavior

Since most CLI tools will exit with code 1 and emit an error message (or emit a warning if they do not crash on bad input), I found the current behavior surprising.

#Version Info

RETRO 12 (2021.7)

#68 Behavior of REPL During Stack Underflow 1 year, 7 months ago

Ticket created by ~rickcarlino on ~crc_/retroforth

EDIT: A previous version of this issue incorrectly stated the problem. The problem does not relate to word not found errors as previously stated.

The current RetroForth REPL behavior differs slightly from what I expected from a traditional Forth.

In REPL mode, a stack underflow error will exit the process:

$ retro
RETRO 12 (2021.7)
524288 Max, 15004 Used, 509284 Free
s:put

ERROR (nga/execute): Stack Limits Exceeded!
At 9320, opcode 29. sp = -1

$

This behavior is unlike most desktop Forth systems (GForth, PForth, etc..), which recover from such an error.

#67 nga-c: reduce global state 1 year, 7 months ago

Comment by ~rickcarlino on ~crc_/retroforth

As multitasking becomes more integrated, there may be a need for "task local" variables. This is something I've been thinking a lot about lately, especially with regard to how hooks would work in a heavily multitasked VM.

#51 RFD: FFI Interface(s) 1 year, 7 months ago

Comment by ~rickcarlino on ~crc_/retroforth

I agree that a callback system would be useful, especially for eventually porting the VM to architectures that use interrupts.

#64 `binary` crashes under certain usage. 1 year, 7 months ago

Ticket created by ~rickcarlino on ~crc_/retroforth

#OK

RETRO 12 (2021.7)
524288 Max, 14948 Used, 509340 Free
'hmmm s:hash n:put
2090333460
bye

#Fails

RETRO 12 (2021.7)
524288 Max, 14948 Used, 509340 Free

binary 'hmmm s:hash n:put

ERROR (nga/execute): Invalid instruction!
At 7008, opcode 49
Instructions: 49d 0d 0d 0d

#63 Inconsistent CLI behavior with `-t` flag. 1 year, 7 months ago

Ticket created by ~rickcarlino on ~crc_/retroforth

You can run an application in non-test mode via retro app.retro. You can also accomplish this via retro -f app.retro.

It is possible to run tests via retro -t -f app.retro. It is not possible to run tests with retro -t app.retro (notice the missing -f)

Current behavior appears to be a noop. This behavior is inconsistent with the non-test equivalent.

#55 Regession on examples page of RetroForth.org 1 year, 9 months ago

Comment by ~rickcarlino on ~crc_/retroforth

Appears to be online again. Closing, thanks!

REPORTED RESOLVED FIXED