Rooba
/
ReC98
mirror of https://github.com/nmlgc/ReC98.git
1
1
Fork 0
Code Issues Packages Projects Releases Wiki Activity
ReC98/th05/formats/pi.h

14 lines
370 B
C
Raw Normal View History

[Maintenance] Clean up PI function declarations and comments
2019-12-15 19:17:00 +00:00
#include "th03/formats/pi.h"
[Maintenance] Only define pi_load_put_8_free() a single time The change of pi_free() from a macro to a function in TH05 doesn't require a complete redefinition. Part of P0124, funded by [Anonymous] and Blue Bolt.
2020-10-25 13:02:12 +00:00
#undef pi_free
[Separate translation units] [th05] pi_free() (undecompilable) Well, it *would* have been decompilable, but that ridiculous placement of the nullptr assignment would have forced the entire function call to be spelled out in inline ASM, verbatim. No amount of comma operator trickery would have generated the same instructions either. And for a function this small and obvious in what its decompilation *should* be, it really defeated the purpose of adding a separate translation unit… Part of P0134, funded by [Anonymous].
2021-02-12 16:39:51 +00:00
void pascal pi_free(int slot);
[Reverse-engineering] [th05] Masked PI display As used for the title screen fade-in effect. Another function that apparently was deliberately written to run not that fast, by blitting each row individually to the 400th VRAM row just so that it can then turn on the EGC, perform the *actual* masked blit to the VRAM destination, and then turn the EGC off before moving to the next now. The same effect could have entirely been accomplished by copying graph_pack_put_8() and applying the mask there; it's not like ZUN didn't know how to modify master.lib… (See also 44ad3eb4) --Nmlgc
2019-12-10 07:52:24 +00:00
[Maintenance] Indicate byte alignment for all .PI blitting functions Part of P0124, funded by [Anonymous] and Blue Bolt.
2020-10-23 17:27:10 +00:00
// Like pi_put_8() and pi_put_quarter_8(), but applying the mask with the given
// ID while blitting.
[Maintenance] Decide on `_masked` instead of `_mask` for blitting functions Because there are only three functions that didn't already follow this naming scheme. (Maintenance mode commit)
2020-12-01 20:58:19 +00:00
int pascal pi_put_masked_8(
screen_x_t left, vram_y_t top, int slot, int mask_id
);
int pascal pi_put_quarter_masked_8(
[Maintenance] Declare distinct types for screen, VRAM, and TRAM coordinates Whew, time to look at every `int` variable we ever declared! The best moment to do this would have been a year ago, but well, better late than never. No need to communicate that in comments anymore. These shouldn't be used for widths, heights, or sprite-space coordinates. Maybe we'll cover that another time, this commit is already large enough. Part of P0111, funded by [Anonymous] and Blue Bolt.
2020-08-20 19:59:45 +00:00
screen_x_t left, vram_y_t top, int slot, int quarter, int mask_id
[Reverse-engineering] [th05] Masked PI display As used for the title screen fade-in effect. Another function that apparently was deliberately written to run not that fast, by blitting each row individually to the 400th VRAM row just so that it can then turn on the EGC, perform the *actual* masked blit to the VRAM destination, and then turn the EGC off before moving to the next now. The same effect could have entirely been accomplished by copying graph_pack_put_8() and applying the mask there; it's not like ZUN didn't know how to modify master.lib… (See also 44ad3eb4) --Nmlgc
2019-12-10 07:52:24 +00:00
);