<div dir="ltr"><br><br>On Fri, Dec 11, 2015 at 2:41 PM, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>> wrote:<br>> From OpenGL 4.0 spec, section 4.3.2 "Copying Pixels":<br>> "The pixels corresponding to these buffers are copied from the source<br>>  rectangle bounded by the locations (srcX0, srcY 0) and (srcX1, srcY 1)<br>>  to the destination rectangle bounded by the locations (dstX0, dstY 0)<br>>  and (dstX1, dstY 1). The lower bounds of the rectangle are inclusive,<br>>  while the upper bounds are exclusive."<br>><br>> So, the rectangles sharing just an edge shouldn't overlap.<br>>      -----------<br>>     |           |<br>>      ------- ---<br>>     |       |   |<br>>     |       |   |<br>>      ------- ---<br>><br>> Cc: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.com</a>><br>> Cc: Matt Turner <<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>><br>> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>><br>> ---<br>>  src/mesa/main/blit.c | 8 ++++----<br>>  1 file changed, 4 insertions(+), 4 deletions(-)<br>><br>> diff --git a/src/mesa/main/blit.c b/src/mesa/main/blit.c<br>> index a32f1a4..9c4c0f3 100644<br>> --- a/src/mesa/main/blit.c<br>> +++ b/src/mesa/main/blit.c<br>> @@ -68,16 +68,16 @@ _mesa_regions_overlap(int srcX0, int srcY0,<br>>                        int dstX0, int dstY0,<br>>                        int dstX1, int dstY1)<br>>  {<br>> -   if (MAX2(srcX0, srcX1) < MIN2(dstX0, dstX1))<br>> +   if (MAX2(srcX0, srcX1) <= MIN2(dstX0, dstX1))<br>>        return false; /* dst completely right of src */<br>><br>> -   if (MAX2(dstX0, dstX1) < MIN2(srcX0, srcX1))<br>> +   if (MAX2(dstX0, dstX1) <= MIN2(srcX0, srcX1))<br>>        return false; /* dst completely left of src */<br>><br>> -   if (MAX2(srcY0, srcY1) < MIN2(dstY0, dstY1))<br>> +   if (MAX2(srcY0, srcY1) <= MIN2(dstY0, dstY1))<br>>        return false; /* dst completely above src */<br>><br>> -   if (MAX2(dstY0, dstY1) < MIN2(srcY0, srcY1))<br>> +   if (MAX2(dstY0, dstY1) <= MIN2(srcY0, srcY1))<br>>        return false; /* dst completely below src */<br>><br>>     return true; /* some overlap */<br>> --<br>> 2.5.0<br>><br><br>Anyone want<div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small;display:inline">​​</div> to review these two patches?<br><a href="https://patchwork.freedesktop.org/series/1763/" target="_blank">https://patchwork.freedesktop.org/series/1763/</a></div>