diff options
author | Rodger Combs <rodger.combs@gmail.com> | 2019-10-14 20:33:31 -0500 |
---|---|---|
committer | Oleg Oshmyan <chortos@inbox.lv> | 2019-11-07 19:43:14 +0200 |
commit | 2793e1d4a6ed4ff15f916a352aaa3be192b678a9 (patch) | |
tree | 720c997e2a8d450c427938f3fb8091f618e7e24e /libass | |
parent | c659c897bcd697a95307406d50564f5eedbe788d (diff) | |
download | libass-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.c | 5 |
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)) { |