Skip to content

ffft Integration#1022

Draft
TheRealQuantam wants to merge 2 commits intoFiendsOfTheElements:devfrom
TheRealQuantam:ffftintegration
Draft

ffft Integration#1022
TheRealQuantam wants to merge 2 commits intoFiendsOfTheElements:devfrom
TheRealQuantam:ffftintegration

Conversation

@TheRealQuantam
Copy link

Creating a draft pull request for code review purposes.

/// <summary>
/// Buffer for binary data that can read and write individual and series of 8-bit and 16-bit values with automatic endian conversion.
/// </summary>
public class BinaryBuffer
Copy link
Author

Choose a reason for hiding this comment

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

My BinaryBuffer class from the MM2 Randomizer

/// <summary>
/// FamiTracker module binary class for parsing and modifying FTMs. Mainly used for rebasing and swapping square channels.
/// </summary>
public class FtmBinary
Copy link
Author

Choose a reason for hiding this comment

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

My FtmBinary class from the MM2 Randomizer

Copy link
Author

Choose a reason for hiding this comment

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

Due to the number of changes made, by far the easiest way to deploy the ffft changes is via a patch file.

Copy link
Author

@TheRealQuantam TheRealQuantam Jul 21, 2023

Choose a reason for hiding this comment

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

XML format is not really something the FF1 randomizer uses, but I was hoping to keep it that way so that entries can be trivially copied between the MM2 and FF1 randomizers.

Currently contains the Dragon Warrior and FF2/3 soundtracks, but more can be easily added.

Copy link
Author

Choose a reason for hiding this comment

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

Move music stuff to its own file for tidiness.

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.

1 participant