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 API versions and improve playlist API docs #857

Conversation

kblaschke
Copy link
Member

@kblaschke kblaschke commented Dec 9, 2024

Added @since tags for all API files, functions and types stating since which projectM release each one is available. This will make it easier for developers to decide on which function they want to use to support older versions, or resolve functions available in newer versions at runtime to make use of a wider range or projectM shared library versions and features based on availability.

Also added clarifications in the API docs for the issue discussed in #854 regarding how callbacks are registered and reset by the playlist library.

Note this one includes the changes already PR'd in #856 - ignore those, I'll lazily merge this one after the copyright date changeset to save me a rebase.

@kblaschke kblaschke added this to the 4.2 milestone Dec 9, 2024
@kblaschke kblaschke self-assigned this Dec 9, 2024
Doxygen documentation now mentions that the playlist library will set the preset switch callbacks in projectM and needs them to function properly. Added hints on how to restore functionality if needed.
@kblaschke kblaschke force-pushed the add-api-versions-and-improve-playlist-api-docs branch from 9d6aa98 to e90dc28 Compare December 9, 2024 13:29
Copy link
Collaborator

@revmischa revmischa left a comment

Choose a reason for hiding this comment

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

nice!

@kblaschke kblaschke merged commit e90dc28 into projectM-visualizer:master Dec 16, 2024
9 checks passed
@kblaschke kblaschke deleted the add-api-versions-and-improve-playlist-api-docs branch December 16, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants