Opened 4 years ago

Closed 4 years ago

#4595 closed enhancement (fixed)

time specification improvements

Reported by: gajjanag 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 Changed 4 years ago by cehoyos

  • Component changed from undetermined to documentation
  • Keywords time removed
  • Priority changed from normal to wish

comment:2 Changed 4 years ago by gajjanag

  • Cc gajjanagadde@gmail.com added

comment:3 Changed 4 years ago by saste

  • Analyzed by developer set
  • Priority changed from wish to normal
  • Reproduced by developer set
  • Resolution set to fixed
  • Status changed from new to 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.