[x265] [PATCH] cli: move raw bitstream output to separate file

Xinyue Lu maillist at 7086.in
Wed Apr 1 00:44:34 CEST 2015



2015/3/31 8:02, Steve Borho:
> ok
>
> watch the space between if and ()
>
> For new files, you can run them through uncrustify using the config file
> in our doc/ folder. This will enforce the coding style mechanically.
>
> here too
>
> nit remove extra space
>
> watch the brace style, open brace on new-line
>

Oops

>
> does pts queueing need to be done in the CLI, or can it be done in the
> output file class (if it needs it)?

AFAIK Matroska & L-SMASH Muxer both require the last PTS delta to compute the actual duration of the video.

I'm afraid that all container muxers require this, except the YUV/Y4M which basically are not container formats.

However I only planned to port these 2 muxers so, I'll follow your decision.


More information about the x265-devel mailing list