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

new C api functions ModPlug_GetVersion and ModPlug_Tell #47

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sezero
Copy link
Contributor

@sezero sezero commented Nov 26, 2019

  1. Add new C api function ModPlug_GetVersion, add version macros to modplug.h
  2. Add new C api function ModPlug_Tell to get the playing position in msecs
    The function was originally authored and is provided by Vitaly Novichkov.
    Increase the version number to 0.8.10 so the new functions can be checked
    easily at compile time.

@sezero
Copy link
Contributor Author

sezero commented Dec 24, 2019

We would use this functionality in SDL_mixer. Any plans for merging?

@sezero
Copy link
Contributor Author

sezero commented Jan 13, 2022

Can we get this in please so that SDL_mixer and others can use it?
Do you need any changes to it?

@sezero sezero force-pushed the newapis branch 3 times, most recently from 0434fde to 740266a Compare January 28, 2022 08:28
@sezero
Copy link
Contributor Author

sezero commented Jan 28, 2022

Rebased to current master.

the function was originally authored and is provided by Vitaly Novichkov.

increase the version number to 0.8.10 so the new functions can be checked
easily at compile time.
@sezero
Copy link
Contributor Author

sezero commented Jan 29, 2022

Rebased (again.)

@sezero
Copy link
Contributor Author

sezero commented Jan 30, 2022

Is there any chance that this gets in ?

@sezero
Copy link
Contributor Author

sezero commented Feb 25, 2022

PING?

@Wohlstand
Copy link

I'll add my ping too.

@sezero
Copy link
Contributor Author

sezero commented Jun 24, 2022

PING?

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