Refactory code: commit_planes can be avoided

intel / IA-Hardware-Composer

35 stars 81 forks source link

Refactory code: commit_planes can be avoided #593

Closed js0701 closed 5 years ago

js0701 commented 5 years ago

DisplayPlaneStateList already hold that. Reduce double vectors can avoid logic complexity

Change-Id: Id8704b6aa8d925ce76708d51dd155fba17decb17 Tests: Android UI normal Tracked-On: None Signed-off-by: Lin Johnson johnson.lin@intel.com

js0701 commented 5 years ago

@shuangwan01 Can you test if video black stripe issue is here? @Shao-Feng @shuangwan01 @yhe39 Let's remove ugly code in this branch. This is is a beginning

shuangwan01 commented 5 years ago

@js0701, just tried this patch in local dev environment, the black stripe issue is not reproduced.

YuanjunHuang commented 5 years ago

nice change

shuangwan01 commented 5 years ago

Reviewed the changes, LGTM :-)

Shao-Feng commented 5 years ago

LGTM too. merged