<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 20, 2022 at 21:29 Michal Prívozník <<a href="mailto:mprivozn@redhat.com">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On 1/20/22 16:48, Ani Sinha wrote:<br>
> <br>
> <br>
<br>
> <br>
> AKA kicking the can one more time 🙃<br>
<br>
Well, I should have been more careful and not merge the patch in the<br>
first place. Changing API behavior is something we should never do.<br>
<br>
Looking at the code closer, it looks like all callers of this function<br>
would need to ignore the reported error so that their behavior is not<br>
changed. At this point, does it make sense to report an error in the<br>
function?</blockquote><div dir="auto"><br></div><div dir="auto">The callers can decide what do with the error raised by the function. We should not write functions that cannot fail. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)" dir="auto"></blockquote></div></div>