Some loadable modules not working in romtool about amitools

[cnvogelg]

blizkick modules need the resident pointer patched to work correctly. this special handling is currently not done. Adding this check shouldn't be that hard. I'll put this feature on my todo.

[paulroberthill]

Thanks! Makes sense. Source is available for all the blitkick modules so I will try correcting it.

[cnvogelg]

hang on a second... I already have the blizkick fixer written here. will commit it soon

[cnvogelg]

here we go: f01cf2a adds support for BlizKick modules... please test if it works for you

[paulroberthill]

Perfect! Many thanks Christian, that seems to work well.
I get a small size difference compared to Remus. Probably not a problem, but I'll add this anyway:

Remus: 21,496 bytes free
Romtool: padding 21,456 bytes with ff

Remus:
...
@00079b78 +0007a534 NT_LIBRARY +103 utility.library utility 40.1 (10.2.93)
@0007a536 +0007a5f8 NT_TASK -120 workbench.task wbtag 39.1 (20.4.92)
@0007a5fa +0007a694 NT_UNKNOWN +104 MoveVBR MoveVBR 1.2 (11.9.96)
@0007a694 +0007a77c NT_UNKNOWN +39 AddDefaultKeyMap AddDefaultKeyMap 1.0 (25/11/06)

RomTool:
...
@00079b78 +0007a534 NT_LIBRARY +103 utility.library utility 40.1 (10.2.93)
@0007a536 +0007a5f8 NT_TASK -120 workbench.task wbtag 39.1 (20.4.92)
@0007a5f8 +0007a692 NT_UNKNOWN +104 MoveVBR MoveVBR 1.2 (11.9.96)
@0007a6bc +0007a7a4 NT_UNKNOWN +39 AddDefaultKeyMap AddDefaultKeyMap 1.0 (25/11/06)

RomTool has a 'gap' between 0007a692 - 0007a6bc.

[cnvogelg]

indeed we can do better: 1c9be62 removes the data segment with version info from a blizkick module. now we include only the resident's size in the ROM

[paulroberthill]

That's great! 2 bytes smaller than Remus (padding?). Seems to work in WinUAE.

Romtool: padding 21498 bytes with ff

@00079b78 +0007a534 NT_LIBRARY +103 utility.library utility 40.1 (10.2.93)
@0007a536 +0007a5f8 NT_TASK -120 workbench.task wbtag 39.1 (20.4.92)
@0007a5f8 +0007a692 NT_UNKNOWN +104 MoveVBR MoveVBR 1.2 (11.9.96)
@0007a692 +0007a77a NT_UNKNOWN +39 AddDefaultKeyMap AddDefaultKeyMap 1.0 (25/11/06)

I think I can retire Remus. Many thanks again Christian! I'll use it to burn my next EPROMS.

[cnvogelg]

Just to be save I have added long word padding, too.
So the results should be similar to Remus' output.

[paulroberthill]

Nice! Slightly different but the total size is the same :-)

Remus:
...
@00079b7 +0007a534 NT_LIBRARY +103 utility.library utility 40.1 (10.2.93)
@0007a53 +0007a5f8 NT_TASK -120 workbench.task wbtag 39.1 (20.4.92)
@0007a5f +0007a694 NT_UNKNOWN +104 MoveVBR MoveVBR 1.2 (11.9.96)
@0007a69 +0007a77c NT_UNKNOWN +39 AddDefaultKeyMap AddDefaultKeyMap 1.0 (25/11/06)

RomTool:
...
@00079b78 +0007a534 NT_LIBRARY +103 utility.library utility 40.1 (10.2.93)
@0007a536 +0007a5f8 NT_TASK -120 workbench.task wbtag 39.1 (20.4.92)
@0007a5f8 +0007a692 NT_UNKNOWN +104 MoveVBR MoveVBR 1.2 (11.9.96)
@0007a694 +0007a77c NT_UNKNOWN +39 AddDefaultKeyMap AddDefaultKeyMap 1.0 (25/11/06)

Remus: 21,496 bytes free
RomTool: padding 21496 bytes with ff

I will burn an EPROM this weekend and let you know if I have any problems. Looks good! Many thanks again!

[cnvogelg]

Issue seems to be fixed