Re: [FFmpeg-devel] [PATCH v4] avformat/http, icecast, rtsp: Add option to disable send-100-continue

2019-03-27 Thread Jun Li
On Mon, Mar 25, 2019 at 2:42 PM Jun Li wrote: > > > On Fri, Mar 22, 2019 at 4:12 PM Jun Li wrote: > >> The current setting for send-100-continue option is either >> enabled if applicable or forced enabled, no option to force >> disable the header. This change i

Re: [FFmpeg-devel] [PATCH v4] avformat/http, icecast, rtsp: Add option to disable send-100-continue

2019-03-29 Thread Jun Li
On Wed, Mar 27, 2019 at 11:27 PM Jun Li wrote: > > > On Mon, Mar 25, 2019 at 2:42 PM Jun Li wrote: > >> >> >> On Fri, Mar 22, 2019 at 4:12 PM Jun Li wrote: >> >>> The current setting for send-100-continue option is either >>> enabl

[FFmpeg-devel] [PATCH v5] avformat/doc, http, icecast, rtsp: Add option to disable send-expect-100

2019-03-30 Thread Jun Li
Fix ticket #7297 The current setting for send-expect-100 option is either enabled if applicable or forced enabled, no option to force disable the header. This change is to expand the option setting to provide more flexibility, which is useful for rstp case. --- doc/protocols.texi| 5 + li

Re: [FFmpeg-devel] [PATCH v4] avformat/http, icecast, rtsp: Add option to disable send-100-continue

2019-03-30 Thread Jun Li
Thanks Michael for review. I updated the iteration to address your feedback: https://patchwork.ffmpeg.org/patch/12540/ Best Regards, -Jun On Sat, Mar 30, 2019 at 6:14 PM Michael Niedermayer wrote: > On Fri, Mar 29, 2019 at 03:59:21PM -0700, Jun Li wrote: > > On Wed, Mar 27, 2019 at

[FFmpeg-devel] [PATCH v1] libavf/movenc: Fix indent

2019-04-01 Thread Jun Li
--- libavformat/movenc.c | 144 +-- 1 file changed, 72 insertions(+), 72 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 26cb2e6ea1..9f05d48733 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5544,87 +5544,87 @@

[FFmpeg-devel] [PATCH v1] lavf/rtsp.c: Fix stimeout option not applied on http tunnel

2019-04-05 Thread Jun Li
stimeout option is already used in tcp transport, since http is based on tcp, pass the option to http for tunneling case. --- libavformat/rtsp.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index 033095905d..2ca1aca94e 100644 -

[FFmpeg-devel] [PATCH v2] lavf/rtsp.c: Fix stimeout option not applied on http tunnel

2019-04-05 Thread Jun Li
stimeout option is already used in tcp transport, since http is based on tcp, pass the option to http for tunneling case. --- libavformat/rtsp.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index 033095905d..8349840c96 10064

Re: [FFmpeg-devel] [PATCH v2] lavf/rtsp.c: Fix stimeout option not applied on http tunnel

2019-04-05 Thread Jun Li
On Fri, Apr 5, 2019 at 11:50 AM Jun Li wrote: > stimeout option is already used in tcp transport, since > http is based on tcp, pass the option to http for tunneling > case. > --- > libavformat/rtsp.c | 10 -- > 1 file changed, 8 insertions(+), 2 deletions(-

Re: [FFmpeg-devel] [PATCH v2] lavf/rtsp.c: Fix stimeout option not applied on http tunnel

2019-04-08 Thread Jun Li
Ping. On Fri, Apr 5, 2019 at 12:00 PM Jun Li wrote: > > > On Fri, Apr 5, 2019 at 11:50 AM Jun Li wrote: > >> stimeout option is already used in tcp transport, since >> http is based on tcp, pass the option to http for tunneling >> case. >> --- >> lib

Re: [FFmpeg-devel] [PATCH v2] lavf/rtsp.c: Fix stimeout option not applied on http tunnel

2019-04-08 Thread Jun Li
On Mon, Apr 8, 2019 at 6:23 PM Liu Steven wrote: > > > > 在 2019年4月9日,上午8:54,Jun Li 写道: > > > > Ping. > > > > On Fri, Apr 5, 2019 at 12:00 PM Jun Li wrote: > > > >> > >> > >> On Fri, Apr 5, 2019 at 11:50 AM Jun Li wrot

Re: [FFmpeg-devel] [PATCH v2] lavf/rtsp.c: Fix stimeout option not applied on http tunnel

2019-04-10 Thread Jun Li
On Mon, Apr 8, 2019 at 8:19 PM Jun Li wrote: > On Mon, Apr 8, 2019 at 6:23 PM Liu Steven wrote: > >> >> >> > 在 2019年4月9日,上午8:54,Jun Li 写道: >> > >> > Ping. >> > >> > On Fri, Apr 5, 2019 at 12:00 PM Jun Li wrote: >&

[FFmpeg-devel] [PATCH v1] lavf/img2enc: add support for option strftime_source

2019-04-14 Thread Jun Li
Currently the strftime option generate timestamp based on generation time. The new option would calcualte timestamp from source's start_realtime and pkt->pts, try to generate a timestamp matches the source starting time. --- doc/muxers.texi | 4 libavformat/img2enc.c | 39 +

Re: [FFmpeg-devel] [PATCH v1] lavf/img2enc: add support for option strftime_source

2019-04-17 Thread Jun Li
On Sun, Apr 14, 2019 at 11:43 PM Jun Li wrote: > Currently the strftime option generate timestamp based on generation > time. The new option would calcualte timestamp from source's > start_realtime and pkt->pts, try to generate a timestamp matches the > source startin

Re: [FFmpeg-devel] [PATCH v1] lavf/img2enc: add support for option strftime_source

2019-04-17 Thread Jun Li
On Wed, Apr 17, 2019 at 3:11 PM Carl Eugen Hoyos wrote: > 2019-04-15 8:43 GMT+02:00, Jun Li : > > Currently the strftime option generate timestamp based on generation > > time. The new option would calcualte timestamp from source's > > start_realtime and pkt->pts

[FFmpeg-devel] [PATCH v2] lavf/img2enc: add support for option strftime_start_realtime

2019-04-18 Thread Jun Li
Currently the strftime option generate timestamp based on generation time. The new option would calcualte timestamp from start_time_realtime and pkt->pts, based on output's timescale. --- doc/muxers.texi | 5 + libavformat/img2enc.c | 34 ++ 2 files chang

Re: [FFmpeg-devel] [PATCH v1] lavf/img2enc: add support for option strftime_source

2019-04-18 Thread Jun Li
On Wed, Apr 17, 2019 at 6:23 PM Carl Eugen Hoyos wrote: > 2019-04-18 2:02 GMT+02:00, Jun Li : > > On Wed, Apr 17, 2019 at 3:11 PM Carl Eugen Hoyos > wrote: > > > >> 2019-04-15 8:43 GMT+02:00, Jun Li : > >> > Currently the strftime option generate timesta

Re: [FFmpeg-devel] [PATCH v2] lavf/img2enc: add support for option strftime_start_realtime

2019-04-21 Thread Jun Li
On Thu, Apr 18, 2019 at 5:59 PM Jun Li wrote: > Currently the strftime option generate timestamp based on generation > time. The new option would calcualte timestamp from start_time_realtime > and pkt->pts, based on output's timescale. > --- > doc/muxers.texi |

[FFmpeg-devel] [PATCH v1] lavf/dashenc: Add option for calculting pkt duration

2019-04-21 Thread Jun Li
Fix #7144. The current packet duration calculation is heuristic, which uses the historical durtion as current duration. This commit adds the option to buffer one packet and calcuate its duration when next packet arrives. Obviously it adds one frame latency, which might be significant for VFR live c

Re: [FFmpeg-devel] [PATCH v1] lavf/dashenc: Add option for calculting pkt duration

2019-04-24 Thread Jun Li
On Sun, Apr 21, 2019 at 5:51 PM Jun Li wrote: > Fix #7144. > The current packet duration calculation is heuristic, which uses the > historical durtion as current duration. This commit adds the option > to buffer one packet and calcuate its duration when next packet arrives. > Ob

Re: [FFmpeg-devel] [PATCH v1] lavf/dashenc: Add option for calculting pkt duration

2019-04-25 Thread Jun Li
On Wed, Apr 24, 2019 at 11:07 PM Jeyapal, Karthick wrote: > > On 4/24/19 11:30 PM, Jun Li wrote: > > On Sun, Apr 21, 2019 at 5:51 PM Jun Li wrote: > > > >> Fix #7144. > >> The current packet duration calculation is heuristic, which uses the > >>

[FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg Add stream metadata from first frame's metadata

2019-05-04 Thread Jun Li
Fix #6945 Exif extension has 'Orientaion' field for image flip and rotation. This change is to add the first frame's exif into stream so that autorotation would use the info to adjust the frames. --- fftools/ffmpeg.c | 45 + 1 file changed, 45 insertions

[FFmpeg-devel] [PATCH v1 1/2] fftools/ffmpeg_filter, ffplay Add flip support for rotation

2019-05-04 Thread Jun Li
Current implemantion for autoratation does not support flip. That is, if the matrix contains flip info, the API get_rotation only reflects partial information. This change is for adding support for hflip (vflip can be achieved by rotation+hflip). --- fftools/cmdutils.c| 4 ++-- fftools/cm

[FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-05 Thread Jun Li
Fix #6945 Exif extension has 'Orientaion' field for image flip and rotation. This change is to add the first frame's exif into stream so that autorotation would use the info to adjust the frames. --- fftools/ffmpeg.c | 52 1 file changed, 52 inserti

[FFmpeg-devel] [PATCH v2 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-05 Thread Jun Li
Current implemantion for autoratation does not support flip. That is, if the matrix contains flip info, the API get_rotation only reflects partial information. This change is for adding support for hflip (vflip can be achieved by rotation+hflip). --- fftools/cmdutils.c| 4 ++-- fftools/cm

Re: [FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg Add stream metadata from first frame's metadata

2019-05-05 Thread Jun Li
On Sun, May 5, 2019 at 6:06 PM Michael Niedermayer wrote: > On Sat, May 04, 2019 at 07:13:34PM -0700, Jun Li wrote: > > Fix #6945 > > Exif extension has 'Orientaion' field for image flip and rotation. > > This change is to add the first frame's exif into strea

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-06 Thread Jun Li
On Mon, May 6, 2019 at 1:52 AM Nicolas George wrote: > Jun Li (12019-05-05): > > Fix #6945 > > Exif extension has 'Orientaion' field for image flip and rotation. > > This change is to add the first frame's exif into stream so that > > autorotati

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-06 Thread Jun Li
On Mon, May 6, 2019 at 3:04 PM Carl Eugen Hoyos wrote: > Am Mo., 6. Mai 2019 um 05:41 Uhr schrieb Jun Li : > > +uint8_t* sd = NULL; > > Wouldn't it simplify the patch if this were a pointer to int32_t? > Or is there a aliasing violation that can't be avoided? >

[FFmpeg-devel] [PATCH v3 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-06 Thread Jun Li
Current implemantion for autoratation does not support flip. That is, if the matrix contains flip info, the API get_rotation only reflects partial information. This change is for adding support for hflip (vflip can be achieved by rotation+hflip). --- fftools/cmdutils.c| 4 ++-- fftools/cm

[FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-06 Thread Jun Li
Fix #6945 Exif extension has 'Orientaion' field for image flip and rotation. This change is to add the first frame's exif into stream so that autorotation would use the info to adjust the frames. --- fftools/ffmpeg.c | 57 +++- 1 file changed, 56 inserti

Re: [FFmpeg-devel] [PATCH v2 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-07 Thread Jun Li
On Tue, May 7, 2019 at 3:10 PM Michael Niedermayer wrote: > On Sun, May 05, 2019 at 08:41:27PM -0700, Jun Li wrote: > > Current implemantion for autoratation does not support flip. > > That is, if the matrix contains flip info, the API get_rotation > > only reflects part

[FFmpeg-devel] [PATCH v4 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-07 Thread Jun Li
Current implemantion for autoratation does not support flip. That is, if the matrix contains flip info, the API get_rotation only reflects partial information. This change is for adding support for hflip (vflip can be achieved by rotation+hflip). --- fftools/cmdutils.c| 4 ++-- fftools/cm

[FFmpeg-devel] [PATCH v4 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-07 Thread Jun Li
Fix #6945 Exif extension has 'Orientaion' field for image flip and rotation. This change is to add the first frame's exif into stream so that autorotation would use the info to adjust the frames. --- fftools/ffmpeg.c | 57 +++- 1 file changed, 56 inserti

Re: [FFmpeg-devel] [PATCH v3 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-07 Thread Jun Li
On Tue, May 7, 2019 at 2:04 AM Moritz Barsnick wrote: > On Mon, May 06, 2019 at 22:36:41 -0700, Jun Li wrote: > > +double av_display_rotation_hflip_get(const int32_t matrix[9], int > *hflip) > > +{ > > +int32_t m[9]; > > +*hflip = 0; > > +m

Re: [FFmpeg-devel] [PATCH v4 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-07 Thread Jun Li
On Tue, May 7, 2019 at 11:24 PM Jun Li wrote: > Current implemantion for autoratation does not support flip. > That is, if the matrix contains flip info, the API get_rotation > only reflects partial information. This change is for adding > support for hflip (vflip can be achieved

Re: [FFmpeg-devel] [PATCH v4 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-07 Thread Jun Li
On Tue, May 7, 2019 at 11:40 PM Gyan wrote: > > > On 08-05-2019 11:54 AM, Jun Li wrote: > > Fix #6945 > > Exif extension has 'Orientaion' field for image flip and rotation. > > This change is to add the first frame's exif into stream so that > &g

Re: [FFmpeg-devel] [PATCH v4 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-08 Thread Jun Li
On Wed, May 8, 2019 at 12:09 AM Gyan wrote: > > > On 08-05-2019 12:25 PM, Jun Li wrote: > > On Tue, May 7, 2019 at 11:40 PM Gyan wrote: > > > > > >> Also, is there a chance that there may be multiple sources for > >> orientation data ava

[FFmpeg-devel] [PATCH v5 1/2] fftools/ffmpeg_filter, ffplay: Add flip support to rotation

2019-05-08 Thread Jun Li
Current implemantion for autoratation does not support flip. That is, if the matrix contains flip info, the API get_rotation only reflects partial information. This change is for adding support for hflip (vflip can be achieved by rotation+hflip). --- fftools/cmdutils.c| 4 ++-- fftools/cm

[FFmpeg-devel] [PATCH v5 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-08 Thread Jun Li
Fix #6945 'Orientation' field from EXIF tags in first decoded frame is extracted as stream side data so that ffmpeg can apply auto-rotation. --- fftools/ffmpeg.c | 63 +++- 1 file changed, 62 insertions(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/

Re: [FFmpeg-devel] [PATCH v4 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-08 Thread Jun Li
On Wed, May 8, 2019 at 12:32 AM Jun Li wrote: > > > On Wed, May 8, 2019 at 12:09 AM Gyan wrote: > >> >> >> On 08-05-2019 12:25 PM, Jun Li wrote: >> > On Tue, May 7, 2019 at 11:40 PM Gyan wrote: >> > >> > >> >> Also, is th

[FFmpeg-devel] [PATCH v1] lavf/mov: Fix timestamp rescale on sidx atom

2019-05-09 Thread Jun Li
Fix #5090 Fix the timestamp rescale issue, from sidx timebase to stream's timebase. --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 78f692872b..d058855e6c 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-09 Thread Jun Li
On Thu, May 9, 2019 at 1:52 AM Nicolas George wrote: > Jun Li (12019-05-06): > > I agree with it that the patch can't solve the case that "not all frames > > have the same orientation". > > The only way I can think of (correct me if I am wrong) is a filter t

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-09 Thread Jun Li
On Thu, May 9, 2019 at 2:29 AM Nicolas George wrote: > Jun Li (12019-05-09): > > The image's exif ("orientation") info is kept inside of frame metadata, > > maybe the filter can directly read from frame, instead of re-config > > the stream level metad

Re: [FFmpeg-devel] [PATCH v1] lavf/mov: Fix timestamp rescale on sidx atom

2019-05-10 Thread Jun Li
On Thu, May 9, 2019 at 2:08 AM Jun Li wrote: > Fix #5090 > Fix the timestamp rescale issue, from sidx timebase to > stream's timebase. > --- > libavformat/mov.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mov.c b/libavfo

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

2019-05-10 Thread Jun Li
On Thu, May 9, 2019 at 1:52 AM Nicolas George wrote: > Jun Li (12019-05-06): > > I agree with it that the patch can't solve the case that "not all frames > > have the same orientation". > > The only way I can think of (correct me if I am wrong) is a filter t

[FFmpeg-devel] [PATCH v1] fftools/ffmpeg: Add support for per frame rotation and flip

2019-05-11 Thread Jun Li
Fix #6945 Current implementaion for autorotate works fine for stream level rotataion but no support for frame level operation and frame flip. This patch is for adding flip support and per frame operations. --- fftools/cmdutils.c| 9 ++--- fftools/cmdutils.h| 2 +- fftools/ffmpeg.

Re: [FFmpeg-devel] [PATCH v1] fftools/ffmpeg: Add support for per frame rotation and flip

2019-05-11 Thread Jun Li
On Sat, May 11, 2019 at 10:47 PM Jun Li wrote: > Fix #6945 > Current implementaion for autorotate works fine for stream > level rotataion but no support for frame level operation > and frame flip. This patch is for adding flip support and > per frame operations. > --- >

Re: [FFmpeg-devel] [PATCH v1] lavf/mov: Fix timestamp rescale on sidx atom

2019-05-12 Thread Jun Li
On Fri, May 10, 2019 at 7:25 PM Jun Li wrote: > > On Thu, May 9, 2019 at 2:08 AM Jun Li wrote: > >> Fix #5090 >> Fix the timestamp rescale issue, from sidx timebase to >> stream's timebase. >> --- >> libavformat/mov.c | 2 +- >> 1 file changed

[FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-14 Thread Jun Li
Fix #6945 Current implementaion for autorotate works fine for stream level rotataion but no support for frame level operation and frame flip. This patch is for adding flip support and per frame operations. --- fftools/cmdutils.c | 9 ++--- fftools/cmdutils.h | 2 +- fftools/ffmpeg.c

[FFmpeg-devel] [PATCH v2 1/2] lavu/display: add av_display_rotation_hflip_get and bump version

2019-05-14 Thread Jun Li
Add av_display_rotation_hflip_get to get information whether the matrix indicates horizontal flip. --- doc/APIchanges| 3 +++ libavutil/display.c | 14 ++ libavutil/display.h | 14 ++ libavutil/tests/display.c | 8 libavutil/version.h

Re: [FFmpeg-devel] [PATCH v1] fftools/ffmpeg: Add support for per frame rotation and flip

2019-05-14 Thread Jun Li
On Mon, May 13, 2019 at 2:58 AM Michael Niedermayer wrote: > On Sat, May 11, 2019 at 10:57:01PM -0700, Jun Li wrote: > > On Sat, May 11, 2019 at 10:47 PM Jun Li wrote: > > > > > Fix #6945 > > > Current implementaion for autorotate works fine for stream > &g

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-15 Thread Jun Li
On Wed, May 15, 2019 at 5:18 AM Andriy Gelman wrote: > On Tue, 14. May 22:36, Jun Li wrote: > > Fix #6945 > > Current implementaion for autorotate works fine for stream > > level rotataion but no support for frame level operation > > and frame flip. This patch is

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-15 Thread Jun Li
On Wed, May 15, 2019 at 11:45 AM Andriy Gelman wrote: > On Tue, 14. May 22:36, Jun Li wrote: > > Fix #6945 > > Current implementaion for autorotate works fine for stream > > level rotataion but no support for frame level operation > > and frame flip. This patch is

[FFmpeg-devel] [PATCH v3 1/2] lavu/display: add av_display_rotation_hflip_get and bump version

2019-05-16 Thread Jun Li
Add av_display_rotation_hflip_get to get information whether the matrix indicates horizontal flip. --- doc/APIchanges| 3 +++ libavutil/display.c | 13 + libavutil/display.h | 14 ++ libavutil/tests/display.c | 26 ++ libavut

[FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Jun Li
Fix #6945 Current implementaion for autorotate works fine for stream level rotataion but no support for frame level operation and frame flip. This patch is for adding flip support and per frame operations. --- fftools/cmdutils.c | 9 ++--- fftools/cmdutils.h | 2 +- fftools/ffmpeg.c

Re: [FFmpeg-devel] [PATCH v2 1/2] lavu/display: add av_display_rotation_hflip_get and bump version

2019-05-16 Thread Jun Li
On Wed, May 15, 2019 at 3:22 PM Michael Niedermayer wrote: > On Tue, May 14, 2019 at 10:36:44PM -0700, Jun Li wrote: > > Add av_display_rotation_hflip_get to get information whether the > > matrix indicates horizontal flip. > > --- > > doc/APIchanges| 3 +

Re: [FFmpeg-devel] [PATCH v1] lavf/mov: Fix timestamp rescale on sidx atom

2019-05-16 Thread Jun Li
On Sun, May 12, 2019 at 7:44 PM Jun Li wrote: > > > On Fri, May 10, 2019 at 7:25 PM Jun Li wrote: > >> >> On Thu, May 9, 2019 at 2:08 AM Jun Li wrote: >> >>> Fix #5090 >>> Fix the timestamp rescale issue, from sidx timebase to >>> stream

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Jun Li
On Thu, May 16, 2019 at 1:21 AM Nicolas George wrote: > Jun Li (12019-05-16): > > Fix #6945 > > Current implementaion for autorotate works fine for stream > > level rotataion but no support for frame level operation > > and frame flip. This patch is for adding fli

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Jun Li
On Thu, May 16, 2019 at 2:06 AM Jun Li wrote: > > > On Thu, May 16, 2019 at 1:21 AM Nicolas George wrote: > >> Jun Li (12019-05-16): >> > Fix #6945 >> > Current implementaion for autorotate works fine for stream >> > level rotataion but no support

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Jun Li
On Thu, May 16, 2019 at 12:04 PM Paul B Mahol wrote: > On 5/16/19, Nicolas George wrote: > > Jun Li (12019-05-16): > >> Sure. > >> This patch is checking the frame's orientation status, and apply input > >> filter if necessary. > >>f

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Jun Li
On Thu, May 16, 2019 at 12:54 PM Andriy Gelman wrote: > Hi Jun, > > On Thu, 16. May 00:12, Jun Li wrote: > > Fix #6945 > > Current implementaion for autorotate works fine for stream > > level rotataion but no support for frame level operation > > and frame fl

Re: [FFmpeg-devel] Export display matrix mirroring info as part of the rotate API

2019-05-16 Thread Jun Li
On Thu, May 16, 2019 at 4:34 PM Ted Meyer < tmathmeyer-at-google@ffmpeg.org> wrote: > Right now ffmpeg doesn't export a mirroring status when checking the > display matrix for rotation. > Here is an example video: https://files.tedm.io/flip.mp4 > -Ted >

[FFmpeg-devel] [PATCH v4] lavu/display: add av_display_rotation_get2 and bump version

2019-05-16 Thread Jun Li
Add av_display_rotation_get2 to get information whether the matrix indicates horizontal flip. --- doc/APIchanges| 3 +++ libavutil/display.c | 13 + libavutil/display.h | 14 ++ libavutil/tests/display.c | 26 ++ libavutil/ve

Re: [FFmpeg-devel] Export display matrix mirroring info as part of the rotate API

2019-05-17 Thread Jun Li
On Thu, May 16, 2019 at 7:38 PM Vittorio Giovara wrote: > On Thu, May 16, 2019 at 9:32 PM Jun Li wrote: > > > On Thu, May 16, 2019 at 4:34 PM Ted Meyer < > > tmathmeyer-at-google@ffmpeg.org> wrote: > > > > > Right now ffmpeg doesn't export a mirr

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-17 Thread Jun Li
On Thu, May 16, 2019 at 9:21 PM Andriy Gelman wrote: > On Thu, 16. May 18:28, Jun Li wrote: > > On Thu, May 16, 2019 at 12:54 PM Andriy Gelman > > wrote: > > > > > Hi Jun, > > > > > > On Thu, 16. May 00:12, Jun Li wrote: > > > > Fix #

Re: [FFmpeg-devel] [PATCH v1] lavf/mov: Fix timestamp rescale on sidx atom

2019-05-21 Thread Jun Li
On Thu, May 16, 2019 at 1:00 AM Jun Li wrote: > > > On Sun, May 12, 2019 at 7:44 PM Jun Li wrote: > >> >> >> On Fri, May 10, 2019 at 7:25 PM Jun Li wrote: >> >>> >>> On Thu, May 9, 2019 at 2:08 AM Jun Li wrote: >>> >>>

[FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-24 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/vf_transpose.c | 258 + 1 file changed, 207 insertions(+), 51 deletions(-) diff --git a/libavfilter/vf_transpose.c b/libavfilter/vf_transpose.c index dd54947bd9..4aebfb9ee4 100644 --- a/libavfil

[FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-24 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 3 ++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 19 ++- 3 files changed, 22 insertions(+), 3 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c

[FFmpeg-devel] [PATCH v2 1/2] lavf/vf_transpose: add exif orientation support

2019-05-25 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/vf_transpose.c | 258 + 1 file changed, 207 insertions(+), 51 deletions(-) diff --git a/libavfilter/vf_transpose.c b/libavfilter/vf_transpose.c index dd54947bd9..4aebfb9ee4 100644 --- a/libavfil

[FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-25 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 3 ++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 35 ++- 3 files changed, 30 insertions(+), 11 deletions(-) diff --git a/fftools/ffmpeg.c b

Re: [FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-25 Thread Jun Li
On Sat, May 25, 2019 at 6:16 AM Michael Niedermayer wrote: > On Fri, May 24, 2019 at 05:04:27PM -0700, Jun Li wrote: > > Fix #6945 > > Rotate or/and flip frame according to frame's metadata orientation > > --- > > fftools/ffmpeg.c| 3 ++- &

Re: [FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-25 Thread Jun Li
On Sat, May 25, 2019 at 2:53 AM Nicolas George wrote: > Jun Li (12019-05-24): > > Fix #6945 > > Rotate or/and flip frame according to frame's metadata orientation > > Since ffmpeg does not handle resolution changes very well, do we want > this enabled by default?

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-25 Thread Jun Li
On Sat, May 25, 2019 at 2:46 AM Nicolas George wrote: > Jun Li (12019-05-24): > > Add exif orientation support and expose an option. > > --- > > libavfilter/vf_transpose.c | 258 + > > 1 file changed, 207 insertions(+), 51 deletion

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-26 Thread Jun Li
On Sun, May 26, 2019 at 1:16 AM Paul B Mahol wrote: > On 5/25/19, Jun Li wrote: > > Add exif orientation support and expose an option. > > --- > > libavfilter/vf_transpose.c | 258 + > > 1 file changed, 207 insertions(+), 51 deleti

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-26 Thread Jun Li
On Sun, May 26, 2019 at 2:09 AM Paul B Mahol wrote: > On 5/26/19, Jun Li wrote: > > On Sun, May 26, 2019 at 1:16 AM Paul B Mahol wrote: > > > >> On 5/25/19, Jun Li wrote: > >> > Add exif orientation support and expose an option. > >> &

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Jun Li
On Mon, May 27, 2019 at 12:32 AM Paul B Mahol wrote: > On 5/27/19, Jun Li wrote: > > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol wrote: > > > >> On 5/26/19, Jun Li wrote: > >> > On Sun, May 26, 2019 at 1:16 AM Paul B Mahol > wrote: > >> &g

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Jun Li
On Mon, May 27, 2019 at 1:39 AM Paul B Mahol wrote: > On 5/27/19, Jun Li wrote: > > On Mon, May 27, 2019 at 12:32 AM Paul B Mahol wrote: > > > >> On 5/27/19, Jun Li wrote: > >> > On Sun, May 26, 2019 at 2:09 AM Paul B Mahol > wrote: > >> >

[FFmpeg-devel] [PATCH v3 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 3 + libavfilter/vf_hflip.c | 43 ++--- libavfilter/vf_transpose.c | 173 + 3 files changed, 171 insertions(+), 48 deletions(-) diff --git a/libavfilter/hflip.h b/lib

[FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-27 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 16 +++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 28 +++- 3 files changed, 40 insertions(+), 7 deletions(-) diff --git a/fftools/ffmpe

Re: [FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-27 Thread Jun Li
On Mon, May 27, 2019 at 7:56 AM Nicolas George wrote: > Jun Li (12019-05-25): > > I believe it has been enabled by default, the 'autorotate' value is true > by > > default. > > Yes, but it is currently enabled for constant rotation. Variable > rotation is mor

Re: [FFmpeg-devel] [PATCH v1 1/2] lavf/vf_transpose: add exif orientation support

2019-05-27 Thread Jun Li
On Mon, May 27, 2019 at 7:52 AM Nicolas George wrote: > Jun Li (12019-05-27): > > I compared the perf between vflip and this patch before, they take almost > > the same time, around 9ms for a frame. > > Hflip is a little special, but surely I can update the patch to avoid

[FFmpeg-devel] [PATCH v4 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-28 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 16 +++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 28 +++- 3 files changed, 40 insertions(+), 7 deletions(-) diff --git a/fftools/ffmpe

[FFmpeg-devel] [PATCH v4 1/2] lavf/vf_transpose: add exif orientation support

2019-05-28 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 2 + libavfilter/transpose.h| 14 libavfilter/vf_hflip.c | 41 ++--- libavfilter/vf_transpose.c | 166 - 4 files changed, 189 insertions(+), 34 deletions(-)

[FFmpeg-devel] [PATCH v5 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-28 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 16 +++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 28 +++- 3 files changed, 40 insertions(+), 7 deletions(-) diff --git a/fftools/ffmpe

[FFmpeg-devel] [PATCH v5 1/2] lavf/vf_transpose: add exif orientation support

2019-05-28 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 2 + libavfilter/transpose.h| 14 libavfilter/vf_hflip.c | 41 ++--- libavfilter/vf_transpose.c | 166 - 4 files changed, 189 insertions(+), 34 deletions(-)

Re: [FFmpeg-devel] [PATCH v3 1/2] lavf/vf_transpose: add exif orientation support

2019-05-28 Thread Jun Li
On Tue, May 28, 2019 at 7:21 AM Paul B Mahol wrote: > On 5/28/19, Jun Li wrote: > > Add exif orientation support and expose an option. > > --- > > libavfilter/hflip.h| 3 + > > libavfilter/vf_hflip.c | 43 ++--- > >

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-28 Thread Jun Li
On Tue, May 28, 2019 at 1:50 PM Michael Niedermayer wrote: > On Mon, May 27, 2019 at 11:18:26PM -0700, Jun Li wrote: > > Fix #6945 > > Rotate or/and flip frame according to frame's metadata orientation > > --- > > fftools/ffmpeg.c| 16 +++- &

[FFmpeg-devel] [PATCH v6 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-05-30 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 16 +++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 28 +++- 3 files changed, 40 insertions(+), 7 deletions(-) diff --git a/fftools/ffmpe

[FFmpeg-devel] [PATCH v6 1/2] lavf/vf_transpose: add exif orientation support

2019-05-30 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 2 + libavfilter/transpose.h| 14 libavfilter/vf_hflip.c | 40 ++--- libavfilter/vf_transpose.c | 161 - 4 files changed, 185 insertions(+), 32 deletions(-)

Re: [FFmpeg-devel] [PATCH v6 1/2] lavf/vf_transpose: add exif orientation support

2019-05-31 Thread Jun Li
On Thu, May 30, 2019 at 5:29 PM Jun Li wrote: > Add exif orientation support and expose an option. > --- > libavfilter/hflip.h| 2 + > libavfilter/transpose.h| 14 > libavfilter/vf_hflip.c | 40 ++--- > libavfilter/vf

Re: [FFmpeg-devel] [PATCH v6 1/2] lavf/vf_transpose: add exif orientation support

2019-06-03 Thread Jun Li
On Fri, May 31, 2019 at 6:15 PM Jun Li wrote: > > On Thu, May 30, 2019 at 5:29 PM Jun Li wrote: > >> Add exif orientation support and expose an option. >> --- >> libavfilter/hflip.h| 2 + >> libavfilter/transpose.h| 14 >

[FFmpeg-devel] [PATCH v7 1/2] lavf/vf_transpose: add exif orientation support

2019-06-04 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 2 + libavfilter/transpose.h| 14 libavfilter/vf_hflip.c | 40 ++--- libavfilter/vf_transpose.c | 163 - 4 files changed, 187 insertions(+), 32 deletions(-)

Re: [FFmpeg-devel] [PATCH v6 1/2] lavf/vf_transpose: add exif orientation support

2019-06-04 Thread Jun Li
On Tue, Jun 4, 2019 at 12:50 AM Paul B Mahol wrote: > On 5/31/19, Jun Li wrote: > > Add exif orientation support and expose an option. > > --- > > libavfilter/hflip.h| 2 + > > libavfilter/transpose.h| 14 > > libavfilter/vf_hflip.c

[FFmpeg-devel] [PATCH v7 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-06-04 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 16 +++- fftools/ffmpeg.h| 3 ++- fftools/ffmpeg_filter.c | 28 +++- 3 files changed, 40 insertions(+), 7 deletions(-) diff --git a/fftools/ffmpe

[FFmpeg-devel] [PATCH v8 1/2] lavf/vf_transpose: add exif orientation support

2019-06-06 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 2 + libavfilter/transpose.h| 14 libavfilter/vf_hflip.c | 40 ++--- libavfilter/vf_transpose.c | 163 - 4 files changed, 187 insertions(+), 32 deletions(-)

[FFmpeg-devel] [PATCH v8 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-06-06 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 5 +++-- fftools/ffmpeg.h| 8 fftools/ffmpeg_filter.c | 36 +++- 3 files changed, 42 insertions(+), 7 deletions(-) diff --git a/fftools/ffm

Re: [FFmpeg-devel] [PATCH v7 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-06-06 Thread Jun Li
On Thu, Jun 6, 2019 at 4:02 AM Michael Niedermayer wrote: > On Tue, Jun 04, 2019 at 06:12:28PM -0700, Jun Li wrote: > > Fix #6945 > > Rotate or/and flip frame according to frame's metadata orientation > [..] > > > + > > +return filterst !=

Re: [FFmpeg-devel] [PATCH v8 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-06-07 Thread Jun Li
On Fri, Jun 7, 2019 at 11:54 AM Michael Niedermayer wrote: > On Thu, Jun 06, 2019 at 08:28:15PM -0700, Jun Li wrote: > > Fix #6945 > > Rotate or/and flip frame according to frame's metadata orientation > > --- > > fftools/ffmpeg.c| 5 +++-- &

[FFmpeg-devel] [PATCH v9 1/2] lavf/vf_transpose: add exif orientation support

2019-06-08 Thread Jun Li
Add exif orientation support and expose an option. --- libavfilter/hflip.h| 2 + libavfilter/transpose.h| 14 libavfilter/vf_hflip.c | 40 ++--- libavfilter/vf_transpose.c | 168 - 4 files changed, 192 insertions(+), 32 deletions(-)

[FFmpeg-devel] [PATCH v9 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-06-08 Thread Jun Li
Fix #6945 Rotate or/and flip frame according to frame's metadata orientation --- fftools/ffmpeg.c| 5 +++-- fftools/ffmpeg.h| 8 fftools/ffmpeg_filter.c | 40 +++- 3 files changed, 46 insertions(+), 7 deletions(-) diff --git a/fftools

Re: [FFmpeg-devel] [PATCH v8 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

2019-06-08 Thread Jun Li
On Sat, Jun 8, 2019 at 2:25 AM Nicolas George wrote: > Jun Li (12019-06-07): > > I think a complete validation should be creating a function like > "atodigit" > > , string to digit, instead of using atoi, > > The problem of validation is a common one, and a

  1   2   3   >