Opened 9 years ago

Closed 9 years ago

#308 closed defect (fixed)

Typo in SAO syntax

Reported by: hellman Owned by: bbross
Priority: minor Milestone:
Component: Text Version:
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

Section 7.3.3.4 Sample adaptive offset parameter syntax

The present text says this:


sao_flag_cb
if( sao_flag_cb ) {

sao_split_param( 0, 0, 0, 1 )
sao_split_param( 0, 0, 0, 1 )

}
sao_flag_cr
if( sao_flag_cr ) {

sao_split_param( 0, 0, 0, 2 )
sao_split_param( 0, 0, 0, 2 )

}


The 'sao_split_param' line is repeated twice in each of the two 'if' clauses. The second line should be 'sao_split_offset(...)' in both cases.

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by chihming.fu

It should be modified as follows

	sao_flag_cb
	if( sao_flag_cb ) {
		sao_split_param( 0, 0, 0, 1 )
		sao_offset_param( 0, 0, 0, 1 )
	}
	sao_flag_cr
	if( sao_flag_cr ) {
		sao_split_param( 0, 0, 0, 2 )
		sao_offset_param( 0, 0, 0, 2 )
	}

comment:3 Changed 9 years ago by wjhan

  • Resolution set to fixed
  • 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)
  • Chih-Ming Fu(Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Tim Hellman(Reporter)
  • Woo-Jin Han(Subscriber, Participant)