id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
858	Missing close brace and conditions in palette_coding ()	yhchao		"1. In the first while loop while( PaletteScanPos < maxSubPos ), the close brace is missing for the condition:

  if( PaletteScanPos > 0 ) {	
    xcPrev = x0 + TraverseScanOrder[ log2CbWidth ][ log2CbHeight ][ PaletteScanPos − 1 ][ 0 ] 	
    ycPrev = y0 + TraverseScanOrder[ log2CbWidth ][ log2CbHeight ][ PaletteScanPos − 1 ][ 1 ]	

  It should be:

  if( PaletteScanPos > 0 ) {	
    xcPrev = x0 + TraverseScanOrder[ log2CbWidth ][ log2CbHeight ][ PaletteScanPos − 1 ][ 0 ] 	
    ycPrev = y0 + TraverseScanOrder[ log2CbWidth ][ log2CbHeight ][ PaletteScanPos − 1 ][ 1 ]	
  }




2. In the condition of copy_above_palette_indices_flag parsing:
 
  if( MaxPaletteIndex > 0  &&  !RunCopyMap[ xC ][ yC ] ) {
    if( ( ( !palette_transpose_flag  &&  yC > 0 )  | |  ( palette_transpose_flag  &&  xC > 0 ) ) &&  CopyAboveIndicesFlag[ xcPrev  ][ ycPrev ]  = =  0 ) {
	copy_above_palette_indices_flag
	CopyAboveIndicesFlag[ xC ][ yC ] = copy_above_palette_indices_flag
    }
    PreviousRunType = CopyAboveIndicesFlag[ xC ][ yC ]
    PreviousRunPosition = PaletteScanPos
  } else
    CopyAboveIndicesFlag[ xC ][ yC ] = CopyAboveIndicesFlag[ xcPrev ][ ycPrev ]


  Should add two condition PaletteScanPos > 0 as follows:


 if( MaxPaletteIndex > 0  &&  !RunCopyMap[ xC ][ yC ] ) {
    if( ( ( !palette_transpose_flag  &&  yC > 0 )  | |  ( palette_transpose_flag  &&  xC > 0 ) ) &&  ( CopyAboveIndicesFlag[ xcPrev  ][ ycPrev ]  = =  0 && PaletteScanPos > 0 ) ) {
	copy_above_palette_indices_flag
	CopyAboveIndicesFlag[ xC ][ yC ] = copy_above_palette_indices_flag
    }
    PreviousRunType = CopyAboveIndicesFlag[ xC ][ yC ]
    PreviousRunPosition = PaletteScanPos
  } elseif ( PaletteScanPos > 0 ) 
    CopyAboveIndicesFlag[ xC ][ yC ] = CopyAboveIndicesFlag[ xcPrev ][ ycPrev ]


, since the variables xcPrev and ycPrev are not defined if PaletteScanPos = = 0.
"	defect	closed	minor	VVC D7 vC	spec	VVC D8 vB	fixed		ksuehring bbross XiangLi fbossen jvet@…