<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif">I just noticed that between Cairo release 1.17.2 and 1.17.4, a suspicious looking always return line was added in function _cairo_clip_reduce_to_boxes() on line 526 in file cairo-clip-boxes.c :</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">1.17.2:</div><div class="gmail_default" style="font-family:tahoma,sans-serif">static cairo_clip_t *<br>_cairo_clip_reduce_to_boxes (cairo_clip_t *clip)<br>{<br>    struct reduce r;<br>    cairo_clip_path_t *clip_path;<br>    cairo_status_t status;<br><br>    if (clip->path == NULL)<br>    return clip;<br>...<br><br>1.17.4<br>static cairo_clip_t *<br>_cairo_clip_reduce_to_boxes (cairo_clip_t *clip)<br>{<br>    struct reduce r;<br>    cairo_clip_path_t *clip_path;<br>    cairo_status_t status;<br><br>  return clip;<br>    if (clip->path == NULL)<br>        return clip;<br>...</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">The additional return before the if condition practically skips the whole function.</div><div class="gmail_default" style="font-family:tahoma,sans-serif">Is this intentional?</div><div class="gmail_default" style="font-family:tahoma,sans-serif">Best,</div><div class="gmail_default" style="font-family:tahoma,sans-serif">Toby</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div></div>