Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compile issue - remove duplicate copy of arg_processing.c.src #44

Closed
wants to merge 34 commits into from

Conversation

robogeek42
Copy link
Contributor

@robogeek42 robogeek42 commented Aug 7, 2024

Fix compile issue - remove duplicate copy of arg_processing.c.src

Now also includes fixes to the vdp_adv_*multiple commands. They need a terminating -1 after the list of bufferIDs.

robogeek42 and others added 30 commits March 19, 2024 12:59
Add core function:
- vdp_adv_add_sprite_bitmap (VDU 23,27,0x26)
and helper funtions:
- vdp_adv_load_sprite_bitmaps
- vdp_adv_create_sprite
mirroring the non "adv" versions of each
 : uint8_t  mos_port_read(uint8_t port);
 : uint8_t  mos_port_write(uint8_t port, uint8_t data);
 : uint8_t  mos_port_read(uint8_t port);
 : uint8_t  mos_port_write(uint8_t port, uint8_t data);
Make additional changes for vpd_pflags->vdp_pflags
Add back in vdp_get_scr_dims for backward compatibility
Commands 23,0,0xA0  13,16,19,21,25 and 26
@robogeek42
Copy link
Contributor Author

Tied to create another Pull Request for fixes to the vdp_adv*multiple commands, but gitgub appears to have merged with my earlier pull request. ?

This now includes :

Author: robogeek <assif42@gmail.com>
Date:   Sat Aug 10 18:43:17 2024 +0100

    Buffer command helpers. Fix: terminate the buffer list.
    Commands 23,0,0xA0  13,16,19,21,25 and 26

changes to src/agon/vdp_vdu.c from my fork,

@astralaster
Copy link
Collaborator

I'm not sure why, but I cannot edit this branch to fix it. I did clone it into a new PR to merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants