search

EdouardBERGE / rasm

RASM powerful Z80 assembler
132 stars 16 forks source link

Respect the -sb and -ss arguments. #33

Closed rgiot closed 10 months ago

rgiot commented 10 months ago

Current version of rasm does not respect the-sb and -ss arguments. This patch aims at respecting them.

Current modification consists at removing the code that forces remu. A probable better way, would be to force remu if and only if -sb or -ss are not provided. Such behavior would be better than the current one (always forced) or the one I suggest in this patch (never forced).

Or an alternative is to fully remove these arguments, but I think it would be a bad idea because these chunks are documented and will probably not evolve over time.

Note that I have not checked if there are side effects with this modification

rgiot commented 10 months ago

I wrongly opened the source in utf8 instead of windows encoding. The number of edit is minimal and concerns only lines with remu=

EdouardBERGE commented 10 months ago

this is working fine SYMB or BRKC are not exported without their respective options (and are exported when -ss or -sb are enabled) like documented in the help

rgiot commented 10 months ago

Sorry, my mistake. You're right, the chunks are present. I have probably not seen them because of the visual space taken by REMU when I hexdumped my test snapshot. However, it remains weird to have this remu chunk present when not requested