Opened 9 years ago

Closed 9 years ago

#724 closed defect (invalid)

Loop in RasterScan to TileScan lacks an early exit

Reported by: pieterkapsenberg Owned by: bbross
Priority: minor Milestone: D8
Component: Text Version: D8 (J1003) d7
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In 6.5.1 Coding tree block raster and tile scanning conversion process, the derivation process for ctbAddrRStoTS is lacking an exit out of the loop when determining tileX and tileY. As a result, tileX and tileY always end up with the maximum value. Since there is no defined way of early exit (no equivalent to the C "break"), this can be solved by running the loop backward like so:

for( i = num_tile_columns_minus1; i >= 0; i-- )
    if( tbX >= colBd[ i ] )
        tileX = i

Alternatively:

    tileX = -1
    for( i = 0; i <= num_tile_columns_minus1; i++ )
        if( ( tileX < 0 ) && ( tbX >= colBd[ i ] ) )
            tileX = i

Change History (2)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by pieterkapsenberg

  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Owner, Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Pieter Kapsenberg(Reporter, Participant)
  • Woo-Jin Han(Subscriber)