Opened 9 years ago

Closed 9 years ago

#4595 closed enhancement (fixed)

time specification improvements

Reported by: Ganesh Ajjanagadde Owned by:
Priority: normal Component: documentation
Version: git-master Keywords:
Cc: gajjanagadde@gmail.com Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: yes

Description

According to the docs,

-ss pos

Seek to a given position in seconds.

-t duration

play <duration> seconds of audio/video

Nevertheless, examining the source code reveals that this description is incomplete.
For instance, ffmpeg does accept the common HH:MM:SS format specifier for times.
The doc above is misleading as it leads one to conclude that the user must convert
their time string to a raw number of seconds before passing it in.

Also, the exact format accepted is not documented in the source code.
I believe bringing it in line with ISO 8601
(​https://en.wikipedia.org/wiki/ISO_8601)
could be a usability improvement.

Change History (3)

comment:1 by Carl Eugen Hoyos, 9 years ago

Component: undetermined → documentation
Keywords: time removed
Priority: normal → wish

comment:2 by Ganesh Ajjanagadde, 9 years ago

Cc: gajjanagadde@gmail.com added

comment:3 by Stefano Sabatini, 9 years ago

Analyzed by developer: set
Priority: wish → normal
Reproduced by developer: set
Resolution: → fixed
Status: new → closed

Fixed by you with commit:

commit 8b63a8849426e89a35fbc87f9d0c4c8240e3798a
Author: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
Date:   Tue Aug 11 13:00:46 2015 -0400

    doc/ffmpeg,doc/ffplay: reference timespec definitions
    
    Reference time definition syntax in option descriptions when appropriate.
    
    Fixes trac ticket #4595.
    
    Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
    Signed-off-by: Stefano Sabatini <stefasab@gmail.com>

Thanks.

Note: See TracTickets for help on using tickets.