This happens for me on OSX 10.15.7 with ruxnasm built from the source code.

I can see the errors and there is no rom produced but the return code is 0. This is an issue when trying to use ruxnasm from within other processes.

Expected behaviour: 0 should only be returned if the assemble was successful, greater than 0 should be returned otherwise.

~/Documents/ruxnasm/target/release(main) » ./ruxnasm ~/life.tal /tmp/yolo.rom                                                                                                                                                                                                                                xyz@abc
error: macro `JSR345#%$#%@##%@$#%$#%#$%4` is not defined
   ┌─ /Users/xyz/life.tal:43:25
   │
43 │ ,fsdfdsaf ;fsdfdasfdsf  JSR345#%$#%@##%@$#%$#%#$%4
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^

error: macro `435` is not defined
   ┌─ /Users/xyz/life.tal:46:1
   │
46 │ 435
   │ ^^^

error: macro `345` is not defined
   ┌─ /Users/xyz/life.tal:47:1
   │
47 │ 345
   │ ^^^

error: macro `243543erf` is not defined
   ┌─ /Users/xyz/life.tal:49:1
   │
49 │ 243543erf DEO
   │ ^^^^^^^^^

error: found bytes on the zeroth page
   ┌─ /Users/xyz/life.tal:43:1
   │
43 │ ╭ ,fsdfdsaf ;fsdfdasfdsf  JSR345#%$#%@##%@$#%$#%#$%4
44 │ │ %#$543
45 │ │ 5345
46 │ │ 435
47 │ │ 345
48 │ │ 3245
49 │ │ 243543erf DEO
   │ ╰─────────────^

warning: macro `//` is never used
  ┌─ /Users/xyz/life.tal:9:40
  │
9 │ %++ { ADD2 } %-- { SUB2 } %** { MUL2 } %// { DIV2 }
  │                                        ^^^

warning: macro `!!` is never used
   ┌─ /Users/xyz/life.tal:10:40
   │
10 │ %<< { LTH2 } %>> { GTH2 } %== { EQU2 } %!! { NEQ2 }
   │                                        ^^^

warning: macro `#$543` is never used
   ┌─ /Users/xyz/life.tal:44:1
   │
44 │ %#$543
   │ ^^^^^^

-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
~/Documents/ruxnasm/target/release(main) » echo $?                                                                                                                                                                                                                                                           xyz@abc
0

This change removes is_hex_digit and substitutes it with pattern matching, eliminating the duplicate expressions. This also removes the panic branch in the binary caused by unwrapping, a potential marginal performance benefit.

Main changes:

• Rename NOP to NIP, as per uxn commit be360ffc1e5c1a41f2daae1ae04673079bf4175b
• Merge LIT and BRK into opcode zero, add INC opcode as per commit 5b4ec0be6b7ca9740fa72d99cfa9d9392dd5d14d
• Rearrange opcodes as per commit 107a59affac6e2e08c406ec096b6f3866fab547a

Other changes:

• Changed the InstructionKind enum to use associated values for more readability. A nice side effect is that the huge match in src/emitter.rs can be replaced with a simple instruction_kind as u8.

I was a bit surprised that the hello world example from https://wiki.xxiivv.com/site/uxntal.html didn't assemble with ruxnasm, now it does!

This pull request includes the uxntal-test-suite repository as a Git submodule in the tests folder. The test cases are ran by the default Rust test runner and are generated by a procedural macro from located in tests/generator.

This changes the opcode order to agree with https://wiki.xxiivv.com/site/uxntal_reference.html.

I don't know if your big rewrite is still in progress, but this made ruxnasm basically usable for me 🙌