Opened 4 years ago

Closed 4 years ago

#1328 closed defect (fixed)

Capitalization of Min( ) and Max( )

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vE
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

In the following 10 instances, "min" and "max" are undefined and should be replaced by "Min" and "Max", respectively.

Section 7.4.3.3

maxSublayerNeeded = min( NumSubLayersInLayerInOLS[ i ][ m ],

maxSublayerNeeded = min( NumSubLayersInLayerInOLS[ i ][ m ],

Section 8.5.6.4

dILimit = ( 1 << max( 13, BitDepth + 1 ) ) (950)

Table 38

max( nTbW, nTbH )

Section 8.8.3.4

For xEdge = 0..min( ( nCbW / 8 ) − 1, numSbX − 1), y = 0..nCbH − 1:

For yEdge = 0..min( ( nCbH / 8 ) − 1, numSbY − 1 ), x = 0..nCbW − 1:

Section C.7

subpicWidthInLumaSamples = min( ( sps_subpic_ctu_top_left_x[ spIdx ] + (1589)

subpicHeightInLumaSamples = min( ( sps_subpic_ctu_top_left_y[ spIdx ] + (1590)

if( vbX > ( subpicX * CtbSizeY / 8 ) && vbX < min( ( subpicX + (1599)

if( vbY > ( subpicY * CtbSizeY / 8 ) && vbY < min( ( subpicY + (1600)

Change history (2)

comment:1 Changed 4 years ago by bheng

Also, there are 2 instances where "abs" should be replaced by "Abs"

Section 8.5.3.3

abs(pL0[ x+2+ dX ][ 2*y+ 2+ dY ]-pL1[ x+2-dX ][ 2*y+2-dY ])

Section 8.5.7.3

sType = abs( motionIdx ) < 32 ? 2 : ( motionIdx <= 0 ? ( 1 − isFlip ) : isFlip )

comment:2 Changed 4 years ago by yk

  • Resolution set to fixed
  • Status changed from new to closed

Good catches! Thanks. Will be fixed in JVET-S2001-vH.

Note: See TracTickets for help on using tickets.