[Bf-blender-cvs] [4fd73e74655] master: Remove Video Proxy Resolution limitation

Eitan noreply at git.blender.org
Mon Aug 10 08:06:13 CEST 2020


Commit: 4fd73e74655321c600f3bce1a9eed24f7a87ce1a
Author: Eitan
Date:   Mon Aug 10 07:54:51 2020 +0200
Branches: master
https://developer.blender.org/rB4fd73e74655321c600f3bce1a9eed24f7a87ce1a

Remove Video Proxy Resolution limitation

Height and width of video proxies was limited to multiples of 8 with
comment "JPEG requires this".

I could not find any resource that would confirm this to be the case
with FFMPEG and MJPEG codec.

I have looked in:
https://ffmpeg.org/ffmpeg-all.html
https://ffmpeg.org/ffmpeg-formats.html
https://ffmpeg.org/doxygen/trunk/structAVCodecContext.html

Reviewed By: ISS, sergey, sybren

Differential Revision: https://developer.blender.org/D6507

===================================================================

M	source/blender/imbuf/intern/indexer.c

===================================================================

diff --git a/source/blender/imbuf/intern/indexer.c b/source/blender/imbuf/intern/indexer.c
index dd2edebedff..76717bef537 100644
--- a/source/blender/imbuf/intern/indexer.c
+++ b/source/blender/imbuf/intern/indexer.c
@@ -472,10 +472,6 @@ static struct proxy_output_ctx *alloc_proxy_output_ffmpeg(
   char fname[FILE_MAX];
   int ffmpeg_quality;
 
-  /* JPEG requires this */
-  width = round_up(width, 8);
-  height = round_up(height, 8);
-
   rv->proxy_size = proxy_size;
   rv->anim = anim;



More information about the Bf-blender-cvs mailing list