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

Preliminary work ahead of threading support #753

Merged
merged 5 commits into from
Apr 25, 2024

Conversation

rcombs
Copy link
Member

@rcombs rcombs commented Apr 23, 2024

Extracted from #636. These are infrastructural changes that don't directly involve multithreading, but either were necessary for its implementation or came about during the work. Best reviewed commit-by-commit.

Copy link
Member

@MrSmile MrSmile left a comment

Choose a reason for hiding this comment

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

Overall LGTM.

@@ -76,6 +76,29 @@ static void text_info_done(TextInfo* text_info)
free(text_info->combined_bitmaps);
}

static bool setup_render_context(RenderContext *state, ASS_Renderer *priv)
Copy link
Member

Choose a reason for hiding this comment

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

Maybe it's better to rename into render_context_init for uniformity?

@@ -43,4 +43,15 @@ char *ass_strndup_fallback(const char *s, size_t n); // definition in ass_utils.
#define strndup ass_strndup_fallback
#endif

#if defined(_WIN32)
Copy link
Member

Choose a reason for hiding this comment

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

This could be an #ifdef.

@rcombs rcombs merged commit a2c8801 into libass:master Apr 25, 2024
1 check passed
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