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

Add Reality Adlib Tracker (RAD) format support #159

Merged
merged 16 commits into from Feb 23, 2019
Merged

Conversation

AliceLR
Copy link
Owner

@AliceLR AliceLR commented Feb 15, 2019

TODO:

  • Fix warnings
  • Add RAD 1.x support.
  • set_order
  • get_order
  • set_position
  • get_position
  • get_length
  • Add test

@AliceLR AliceLR added this to the 2.92 milestone Feb 15, 2019
@AliceLR AliceLR changed the title Add initial Reality Adlib Tracker (RAD) format support Add Reality Adlib Tracker (RAD) format support Feb 15, 2019
@asiekierka
Copy link
Contributor

XMP allegedly supports .RAD files, which would be good for ensuring we can handle for the oddball platforms without a proper C++ compiler.

@AliceLR
Copy link
Owner Author

AliceLR commented Feb 15, 2019

XMP actually does not support RAD files. All support for RAD/OPL S3Ms/other Adlib formats was removed in 4.4.0 because it was apparently really bad. Using the official RAD player also ensures we have RAD 2.x support (which other players like AdPlug currently don't). I think the concern about C++ being required is valid though (dunno which platforms that would be, but).

@AliceLR AliceLR changed the base branch from master to master-2.92 February 22, 2019 00:54
@asiekierka
Copy link
Contributor

I think the concern about C++ being required is valid though (dunno which platforms that would be, but).

None of the ones we handle now, but as we continue looking into the absurd...

@AliceLR AliceLR merged commit 1d7076b into master-2.92 Feb 23, 2019
@AliceLR AliceLR deleted the add-reality branch February 23, 2019 09:20
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

2 participants