Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chapters with 0 duration repeat the chapter title #49

Closed
GoogleCodeExporter opened this issue Aug 19, 2015 · 4 comments
Closed

Chapters with 0 duration repeat the chapter title #49

GoogleCodeExporter opened this issue Aug 19, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

If you call MP4SetChapters with an array of chapters with different chapter
titles, but 0 durations, the first chapter title gets written to very
chapter.  I've attached a test program that creates this.

Dan

Original issue reported on code.google.com by danahins...@gmail.com on 19 Dec 2009 at 11:09

Attachments:

@GoogleCodeExporter
Copy link
Author

I don't doubt this is reproducible, but why would you want to create chapters 
with zero duration?

Original comment by kid...@gmail.com on 27 Jul 2010 at 6:36

@GoogleCodeExporter
Copy link
Author

I have a movie tagging program, MetaX, and the user supplies the chapter names 
and durations.  Sometimes they have the chapter titles, but not the durations 
(due to the way they rip the DVD).  But I agree this isn't a big issue.

Original comment by danahins...@gmail.com on 27 Jul 2010 at 11:16

@GoogleCodeExporter
Copy link
Author

I looked into this more closely, and I think your best bet is to enforce a 
non-zero duration (i.e. if users enter zero, bump it up to something non-zero). 
 Basically, mp4v2 doesn't have a means of returning samples at the same time 
(so far as I can see) on the same track; it'll always return the first sample.  
It's up to libraries calling the API to correctly set track duration.

Let me know if this is an issue and I'll open the issue again and take another 
look.

Original comment by kid...@gmail.com on 26 Jun 2011 at 6:22

  • Changed state: WontFix

@GoogleCodeExporter
Copy link
Author

OK, I'll just make the change in my code.

Original comment by danahins...@gmail.com on 11 Jul 2011 at 5:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant