momo0853 / mp4v2

mp4v2 for android
Other
19 stars 20 forks source link

"In atom meta unexpected child atom free" warnings #95

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Use practically any mp4v2 tool (e.g., mp4info) on recent, apparently 
correctly playing audio or video .mp4s

What is the expected output? What do you see instead?
mp4info output example:
mp4v2::impl::MP4Atom::ReadChildAtoms: "Y:\Movies\CGI\Shrek (2001).mp4": In atom 
meta unexpected child atom free
Track   Type    Info
1       video   H264 High@3, 5399.720 secs, 949 kbps, 720x480 @ 23.995318 fps
2       audio   MPEG-4 AAC LC, 5401.130 secs, 124 kbps, 48000 Hz
3       audio   MPEG-4 AAC LC, 5401.173 secs, 74 kbps, 24000 Hz
4       subp
5       subp
6       subp
7       subp
8       subp
9       subp
10      text
mp4v2::impl::MP4Atom::ReadChildAtoms: "Y:\Movies\CGI\Shrek (2001).mp4": In atom 
meta unexpected child atom free
 Name: Shrek
 Sort Name: Shrek 1
 Release Date: 2001
 Genre: CGI
 Cover Art pieces: 1
 Media Type: Movie
 TV Show: Shrek
 Sort TV Show: Shrek 1
 Short Description: It ain't easy bein' green--especially if you're a likable (albeit smelly) ogre named Shrek (Mike Myers). On a mi
ssion to retrieve a gorgeous princess (Cameron Diaz) from the clutches of a 
fire-breathing dragon, Shrek teams up with an unlikely c
ompatriot--
 Long Description: It ain't easy bein' green--especially if you're a likable (albeit smelly) ogre named Shrek (Mike Myers). On a mis
sion to retrieve a gorgeous princess (Cameron Diaz) from the clutches of a 
fire-breathing dragon, Shrek teams up with an unlikely co
mpatriot--a wisecracking donkey (Eddie Murphy). This is the widescreen version 
of the film and includes a commentary, technical goof
s, a storyboard pitch of deleted scenes and more.

Note the "mp4v2::impl::MP4Atom::ReadChildAtoms: "Y:\Movies\CGI\Shrek 
(2001).mp4": In atom meta unexpected child atom free" warnings.

What version of the product are you using? On what operating system?
Versions checked out via SVN within the last two weeks (on Windows 7, x64)

Please provide any additional information below.
Again, these warnings appear to be harmless but I've not seen them until recent 
versions of mp4v2 tools.  They make me worry a little that my build process may 
have something that needs fixing.  Please let me know if these are symptoms of 
a more serious problem.

PS: Sorry to hammer the site with issue reports over the last week, most of 
which are either trivial or probably harmless, but I imagine you'd want to 
know.  If they are a bother, please let me know and I'll confine myself to 
serious problems.

Original issue reported on code.google.com by CarlEd...@gmail.com on 16 Apr 2011 at 3:58

GoogleCodeExporter commented 8 years ago
the logging in mp4v2 recently received an update, so it's probably not your 
build process or anything that's changed for you.  dbyron will hopefully have a 
bit of time to look into this one for you.  Thanks for reporting it.

Original comment by kid...@gmail.com on 17 Apr 2011 at 6:37

GoogleCodeExporter commented 8 years ago
This is another case where the text of a message made me think of a warning, 
but maybe it's not as severe as that.  Before the logging changes it was:

            VERBOSE_READ(GetVerbosity(),
                         printf("Warning: In atom %s unexpected child atom %s\n",
                                GetType(), pChildAtom->GetType()));

and now it's
            log.warningf("%s: \"%s\": In atom %s unexpected child atom %s", __FUNCTION__,
                         m_File.GetFilename().c_str(), GetType(), pChildAtom->GetType());

(Fingers crossed that the line wrapping comes out right).  Anyway, I'll change 
to verbose1f and leave out the word warning as I did for issue #91.

Original comment by dbyr...@gmail.com on 19 Apr 2011 at 4:20

GoogleCodeExporter commented 8 years ago
fixed in svn r460

Original comment by dbyr...@gmail.com on 19 Apr 2011 at 4:23