quietvoid / dovi_tool

dovi_tool is a CLI tool combining multiple utilities for working with Dolby Vision.
MIT License
582 stars 57 forks source link

Implement that the 'info' command shows the total number of frames in case of 'out of range' error #88

Closed manuelrn closed 2 years ago

manuelrn commented 2 years ago

Hi!

When with the info command you try to see the information of a frame greater than the number of frames that the RPU contains, this error occurs:

Parsing RPU file...
Error: info: invalid frame number (out of range)


Perhaps it would be useful for the user that, in addition to notifying this error, the number of available frames is also notified. For example, something like this:

Parsing RPU file...
Error: info: invalid frame number (out of range)
Metadata length: 123456


Thanks and regards!

quietvoid commented 2 years ago

Now prints something like this:

Parsing RPU file...
Error: info: invalid frame number (out of range).
Number of valid RPUs parsed: 185341
manuelrn commented 2 years ago

Now prints something like this:

Parsing RPU file...
Error: info: invalid frame number (out of range).
Number of valid RPUs parsed: 185341

Thank you very much for your quick reply.