[Mesa-dev] [Bug 104625] semicolon after if

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=104625

Timothy Arceri  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #2 from Timothy Arceri  ---
Fixed by:

author  Thomas Hellstrom 
commit  80c31f7837cd319910d94d780f5048de6cce0adb

svga: Fix a leftover debug hack

Fix what appears to be a leftover debug hack.
The hack would force the driver to take a different blit path; possibly,
although unverified, reverting to software blits.

Tested using piglit tests/quick. No related regressions.

Cc: "17.2 17.3 18.0" 
Fixes: 9d81ab7376 (svga: Relax the format checks for copy_region_vgpu10
somewhat)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104625
Reported-by: Grazvydas Ignotas 
Signed-off-by: Thomas Hellstrom 
Reviewed-by: Brian Paul 

-- 
You are receiving this mail because:
You are the QA Contact for the bug.___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 104625] semicolon after if

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=104625

--- Comment #1 from Thomas Hellström  ---
Thanks. This looks like a leftover debug hack.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 104625] semicolon after if

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=104625

Bug ID: 104625
   Summary: semicolon after if
   Product: Mesa
   Version: git
  Hardware: Other
OS: All
Status: NEW
  Severity: normal
  Priority: medium
 Component: Other
  Assignee: thellst...@vmware.com
  Reporter: nota...@gmail.com
QA Contact: mesa-dev@lists.freedesktop.org

9d81ab73768db6647c116f4710f7a93d8d41e3b1 adds this:

+  if (blit_info->src.resource->format != blit_info->src.format ||
+  blit_info->dst.resource->format != blit_info->dst.format ||
+  !util_is_format_compatible(src_desc, dst_desc));

which looks wrong and does nothing if I read it right. Perhaps something else
was intended?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev