
<html>

<head>

<meta http-equiv="content-type" content="text/html; charset=UTF-8">

</head>

<body><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Oops sorry, I noticed your mail after I sent the patches, I'm rebasing and resending</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Le jeudi 2 septembre 2021, 23:29:55 CEST Zebediah Figura (she/her) a écrit :</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Wow. That is some of the bizarrest behaviour I've seen. I'd call it a</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> bug, but I can't even figure out what behaviour they were *trying* to</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> implement.</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Well you know... windows being a bug per se... ¯\_(ツ)_/¯</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> I kept reading through your test, trying to describe the behaviour in</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> simpler (or saner) terms, and failing. So congratulations, I think</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> you've hit the nail on the head :D</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Thank you that's kind! :)</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Yeah, that looks about right, nothing else comes to mind now.</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> In terms of some informal review on those, I saw that one and the one</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> with the comments changed. I think in both cases the comments are kind</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> of redundant—reading the code tells me as much and is actually kind of</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> easier. Where comments could help (and the way I usually tend to write</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> them) is in listing the conclusions that each following chunk of code</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> actually proves. That's a bit vague, but I think it would help make the</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> tests make sense a bit more readily.</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Ok, I'll add some explanatory notes.</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Also, a nitpick—there's a C99 variable declaration in the for loop</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> initializer; unfortunately we have to avoid those.</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Oops, fixing!</p>

</body>

</html>