Opened 18 months ago
Last modified 18 months ago
#11516 reopened enhancement
avformat/mov: Parse 'name' atom for title metadata
| Reported by: | Yuxin Wang | Owned by: | |
|---|---|---|---|
| Priority: | minor | Component: | avformat |
| Version: | git-master | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Reproduced by developer: | no | |
| Analyzed by developer: | no |
Description (last modified by )
mov_write_track_udta_tag in libavformat/movenc.c uses the 'name' atom for recording track titles, but the 'name' atom is not recognized in libavformat/mov.c. So I submitted a patch.
---
libavformat/mov.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavformat/mov.c b/libavformat/mov.c
index 57d9364..6e8c40c 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -392,6 +392,7 @@ static int mov_read_udta_string(MOVContext *c, AVIOContext *pb, MOVAtom atom)
return mov_metadata_loci(c, pb, atom.size);
case MKTAG( 'm','a','n','u'): key = "make"; break;
case MKTAG( 'm','o','d','l'): key = "model"; break;
+ case MKTAG( 'n','a','m','e'): key = "title"; break;
case MKTAG( 'p','c','s','t'): key = "podcast";
parse = mov_metadata_int8_no_padding; break;
case MKTAG( 'p','g','a','p'): key = "gapless_playback";
--
2.39.5 (Apple Git-154)
I submitted the patch several days ago, but it has received little attention. In addition, I found a similar patch from 2018 that was also not merged.
Could someone please review the patch?
Change History (2)
comment:1 by , 18 months ago
| Description: | modified (diff) |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:2 by , 18 months ago
| Description: | modified (diff) |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
Note:
See TracTickets
for help on using tickets.


