Jonas Bernoulli <[email protected]> writes: > William Casarin <[email protected]> writes: >> LGTM, I have a similar patch here: id:[email protected] > > So if I understood correctly the problem with that was that you were > unable to run the tests and therefore couldn't confirm that you fixed > the issue or even that there was an issue in the first place.
More so I got a bug report from a user that I couldn't replicate. I haven't run the tests. > Well I was able to build and test and I can confirm that there is an > issue and that your commit fixes it. So it should be merged (though > I would merge my own variant because of the extra comment ;-). Sounds good, happy to use this patch instead. Cheers, Will _______________________________________________ notmuch mailing list -- [email protected] To unsubscribe send an email to [email protected]
