Sure I can.
But I spent about 1h reviewing the patch and web, and I got to a point where I am a bit confused.
The old patch seems to create file with list of 4-character long hexa addresses, i.e.:
1234
BCDE
I have very little idea about what is the current version of unreal and it's specifications, but if I would guess by this project:
https://github.com/mkoloberdin/unrea...bgbpx.cpp#L679
the breakpoints file format is now more complicated, allowing for various conditional expressions, but just simple 4char hexa data will be not read at all.
Can you please confirm what kind of breakpoints file can be used with what emulator, and at which version? Should I add the original form from patch? Would it help to add also some new form compatible with that parser in source I link above?
I'm planning to add also ZEsarUX type of output, which will look like this (and can be used by mixing it into the command line while launching the emulator):
--enable-breakpoints --set-breakpoint 1 "PC=1234h" --set-breakpoint 2 "PC=0BCDEh"
Maybe I can even add support for any-expression (the ZEsarUX allows for something like "PC=1234 AND B=255 AND C=123", etc...), although I'm not sure how useful that will be, if I will add it, I will probably not verify the syntax too much, so it will require good knowledge of emulator on the side of the coder writing asm, to use it correctly.
Also with the machines having extra memory beyond 64kiB address range, if you can give me info how to add also memory paging info to the breakpoint, it will be even better (I'm not even sure how to do that in ZEsarUX, will take a look, maybe it does support something like "PC=... AND somethingLikeMemoryPage=...", or I can ask Cesar to add it maybe... (just talking loud while thinking).
...
So, please specify correct format with list of emulator+version which supports it, so I can add that to documentation, and add examples of different types of breakpoints output for different emulators. And some examples how the output file should look (please make it explicit if hexa values like "BCDE" need initial zero or not, and if they need suffix/prefix like 0xBCDE or 0BCDEh - the old patch seems to produce only "BCDE"? That's a bit weird.)
Thanks.