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

Allow definition of WRAM as a contiguous segment from C000 - DFFF (i.e. DMG, SGB) #122

Merged
merged 1 commit into from
Mar 15, 2017

Conversation

PikalaxALT
Copy link
Contributor

Pass the -w flag to make this happen

@PikalaxALT
Copy link
Contributor Author

#60 takes a different approach to the same problem.

@BenHetherington
Copy link
Contributor

BenHetherington commented Feb 7, 2017

This approach looks much better than #60's. It'd be great if you could add the -w flag to the man documentation (rgblink.1) as well! Fixes #52.

Copy link
Member

@AntonioND AntonioND left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea, just fix this and it should be good to go.

@@ -10,6 +10,7 @@
extern SLONG options;
#define OPT_SMALL 0x01
#define OPT_SMART_C_LINK 0x02
#define OPT_CONTWRAM 0x04
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, change the space here by a tab, and align the 3 elements.

@@ -131,7 +131,7 @@ area_AllocAbsAnyBank(SLONG org, SLONG size, enum eSectionType type)

SLONG startBank = SECT_ATTRIBUTES[type].bank;
SLONG bankCount = SECT_ATTRIBUTES[type].bankCount;

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't fix the whitespace, or do it in another commit.

@AntonioND
Copy link
Member

In the end I've fixed the conflicts myself.

@AntonioND AntonioND mentioned this pull request Mar 15, 2017
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.

None yet

4 participants