On Thu, Mar 05, 2020 at 08:19:02PM +0100, Tim Duesterhus wrote:
> The single line merge would have happened in the missing 7th patch.

OK, but as I started to address the build warning, I really saw that
what was left there was just useless.

> I've
> rebased it, resolved the merge conflict you created and added back the
> `unlikely` that you dropped.

If you want. I dropped it because it's useless there and doesn't
improve readability.

> The other change I made was using `b_putist` instead of my manual loop as
> suggested.

OK!

> It compiles and passes reg-tests, I did not verify that tracing works.

It looks good, applied now. Thank you and I won't bother you anymore for
today :-)

Cheers,
Willy

Reply via email to