[x265] Encoder::configure() frame padding

Matt Johnson johnso87 at illinois.edu
Fri Dec 6 15:24:42 CET 2013


Hi all,
	The following code is in Encoder::configure():

//======== set pad size if width is not multiple of the minimum CU size
     uint32_t maxCUDepth = (uint32_t)g_convertToBit[_param->maxCUSize];
     uint32_t minCUDepth = (_param->maxCUSize >> (maxCUDepth - 1));
     if ((_param->sourceWidth % minCUDepth) != 0)
     {
         //pad width to multiple of minCUDepth
     }
     ...
     if ((_param->sourceHeight % minCUDepth) != 0)
     {
	//pad height to multiple of minCUDepth
     }

I had a couple of questions:
* Should minCUDepth be renamed minCUSize?
* Aren't the first two lines a convoluted way of always computing "8"? 
x >> ((log2(x) - 2) - 1) = x / 2^(log2(x) - 3) = 8
* Is there any reason for maxCUDepth or minCUSize to be set via command 
line or some other way instead of being implicitly computed here?

Thanks,
Matt


More information about the x265-devel mailing list