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 Yuxin Wang)

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 Yuxin Wang, 18 months ago

Description: modified (diff)
Resolution: fixed
Status: newclosed

comment:2 by Yuxin Wang, 18 months ago

Description: modified (diff)
Resolution: fixed
Status: closedreopened
Note: See TracTickets for help on using tickets.