summaryrefslogtreecommitdiffstats
path: root/libass
diff options
context:
space:
mode:
authorRodger Combs <rodger.combs@gmail.com>2019-10-14 20:33:31 -0500
committerOleg Oshmyan <chortos@inbox.lv>2019-11-07 19:43:14 +0200
commit2793e1d4a6ed4ff15f916a352aaa3be192b678a9 (patch)
tree720c997e2a8d450c427938f3fb8091f618e7e24e /libass
parentc659c897bcd697a95307406d50564f5eedbe788d (diff)
downloadlibass-2793e1d4a6ed4ff15f916a352aaa3be192b678a9.tar.bz2
libass-2793e1d4a6ed4ff15f916a352aaa3be192b678a9.tar.xz
coretext: move meta var into loop and zero-initialize
This makes it a bit clearer that the struct's contents won't be reused across multiple iterations
Diffstat (limited to 'libass')
-rw-r--r--libass/ass_coretext.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/libass/ass_coretext.c b/libass/ass_coretext.c
index 7371f7c..f733277 100644
--- a/libass/ass_coretext.c
+++ b/libass/ass_coretext.c
@@ -123,8 +123,6 @@ static char *get_name(CTFontDescriptorRef fontd, CFStringRef attr)
static void process_descriptors(ASS_Library *lib, ASS_FontProvider *provider,
CFArrayRef fontsd)
{
- ASS_FontProviderMetaData meta;
-
if (!fontsd)
return;
@@ -135,6 +133,7 @@ static void process_descriptors(ASS_Library *lib, ASS_FontProvider *provider,
}
for (int i = 0; i < CFArrayGetCount(fontsd); i++) {
+ ASS_FontProviderMetaData meta = {0};
CTFontDescriptorRef fontd = CFArrayGetValueAtIndex(fontsd, i);
int index = -1;
@@ -145,8 +144,6 @@ static void process_descriptors(ASS_Library *lib, ASS_FontProvider *provider,
continue;
}
- memset(&meta, 0, sizeof(meta));
-
char *ps_name = get_name(fontd, kCTFontNameAttribute);
if (ass_get_font_info(lib, ftlib, path, ps_name, -1, &meta)) {