Bug 105783

Summary: mesa-18.0.0/src/gallium/drivers/vc5/vc5_draw.c:589: duplicate expression ?
Product: Mesa Reporter: dcb314
Component: OtherAssignee: Eric Anholt <eric>
Status: RESOLVED DUPLICATE QA Contact: mesa-dev
Severity: normal    
Priority: medium    
Version: 17.3   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description dcb314 2018-03-28 11:18:58 UTC
[mesa-18.0.0/src/gallium/drivers/vc5/vc5_draw.c:589]: (style) Same expression on both sides of '||'.

Source code is

      if (surf->format == PIPE_FORMAT_B4G4R4A4_UNORM ||
          surf->format == PIPE_FORMAT_B4G4R4A4_UNORM) {

Suggest either simplify expression or use some other constant.
Comment 1 Eric Engestrom 2018-03-28 12:09:09 UTC
This was introduced in 2e3c7beb1e60a47e1f5dd "broadcom/vc5: Pack clear colors according to the TLB internal format/type."
Comment 2 Grazvydas Ignotas 2018-03-28 12:17:44 UTC

*** This bug has been marked as a duplicate of bug 104626 ***

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.