search

branlwyd / bdcpu16

DCPU-16 simulator based on version 1.7 of the DCPU-16 specification. See http://dcpu.com/.
0 stars 0 forks source link

not all monitor tests pass #14

Closed branlwyd closed 11 years ago

branlwyd commented 11 years ago

The monitor testing suite from https://github.com/0x10c-crap/emulator-test/ has some failures. Specifically the "Border" and "Default Font" tests fail.

The Default Font test may be ignorable (if it's due to a difference in default font)--but should check that the format of our font dump matches what is expected, since we're using the default passed down by Notch.

The Border test is worrying--test result is only based on whether we push 'Y' or 'N'. That we get it wrong indicates a possible problem with instruction compilation.

branlwyd commented 11 years ago

The Border test mysteriously started passing when I switched to per-character images (not committed because it turned out coloring these images was slower than just recreating them each time!) Not sure what to make of that.

branlwyd commented 11 years ago

The two tests that were actually failing were the default font & default palette test (which are listed as LEM Border and LEM Default Font, respectively--yes, the default palette test is listed as LEM Default Font).

e01e247a963cd0b68142fbd3c7633742ed41df09 fixes an issue with palette dumping and now the default palette test passes. The default font test investigation is ongoing...

branlwyd commented 11 years ago

...and 0721fea82e1fd8ac526e44f319f0738c4c1b5661 fixes an issue with font dumping, and the default font test now passes. Closing.