commit ee0af21256e7a985730ad7b9aa842cbabcf55af7
author Yushin Cho <ycho@mozilla.com> Mon Jan 09 15:46:55 2017 -0800
committer Yushin Cho <ycho@mozilla.com> Tue Jan 10 00:51:43 2017 +0000
tree d16b7da71aa0c055b53a6de7b305914349091843
parent 1a2df5e295eab3ee4de729b253e2e0c1b9a714b6

Correct the misleading codes in encode_inter_mb_segment()

In encode_inter_mb_segment(), when BLOCK_8X4 or BLOCK_4X8 is
passed, the nested loop inside it iterates always twice.
(For BLOCK_4X4, loop iterates only once because encode_inter_mb_segment()
is called for each of 4X4 block.)
Then, the k for 1st iteration is always zero, and the k for 2nd
iteration is always (idy * 2 + idx) with either idy == 1 or idx == 1
depending on the sb_type.

Using "+=" there could mislead readers expecting that
the # of iterations is more.
And probably using simple assignment would be more proper here.

Change-Id: I7a11255eca13403bc090ba4f0cd4785db9f0e541