Opened 12 years ago

Closed 12 years ago

#550 closed defect (fixed)

Context init table for merge_flag is incorrect

Reported by: pieterkapsenberg Owned by:
Priority: minor Milestone: HM-7.1
Component: HM Version: HM-7.0
Keywords: context init table merge flag Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

It appears that the context init table for merge flag is incorrect. In ContextTables.h:

static const UChar 
INIT_MERGE_FLAG_EXT[3][NUM_MERGE_FLAG_EXT_CTX] = 
{
#if SLICE_TYPE_ORDER
  { CNU, }, 
  { 110, }, 
  { 154, }, 
#else
  { 154, }, 
  { 110, }, 
  { CNU, }, 
#endif
};

This implies the B slice merge flag init is unused, and I slice is used.

Change History (2)

comment:1 Changed 12 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 12 years ago by ksuehring

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

Good spotting. When implementing the slice_type numbering change I made an editing mistake: The #if and #else branches should be exchanged. I fixed this in r2426

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

  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • Pieter Kapsenberg(Reporter)