Opened 9 years ago

Closed 7 years ago

#784 closed defect (fixed)

confusing part_mode initialization

Reported by: sorin Owned by:
Priority: minor Milestone: HM-13.0
Component: HM Version: HM-8.0
Keywords: part_mode Cc: fbossen, ksuehring, davidf, jct-vc@…

Description (last modified by ksuehring)

static const UChar 
INIT_PART_SIZE[3][NUM_PART_SIZE_CTX] =   
{
  { 154,  139,  CNU,  CNU, },  
  { 154,  139,  CNU,  CNU, },  
  { 184,  CNU,  CNU,  CNU, },  
};

binIdx=2 is used. Should be:

static const UChar 
INIT_PART_SIZE[3][NUM_PART_SIZE_CTX] =   
{
  { 154,  139,  154,  CNU, },  
  { 154,  139,  154,  CNU, },  
  { 184,  CNU,  CNU,  CNU, },  
};

CNU happens to be 154, so functionally it is correct, just confusing.

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 8 years ago by ksuehring

  • Description modified (diff)

comment:3 Changed 7 years ago by fbossen

  • Milestone set to HM-13.0
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r3774

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, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • Sorin Cismas(Reporter)