>I wasn't claiming there's no reason behind it, just asking if there are>any, for documentation's sakei see many patches going in without documentation (or patches falling of the list without documentation) so I can imaginewhy he sent in without documentation... .oOp wo 26 jan. 2022 om 00:44 schreef Zebediah Figura (she/her) <zfigura@codeweavers.com>:On 1/25/22 17:22, Benjamin Shefte wrote:
> this mostdefinitaly has affected multiple server applications I use for the
> old WON.Net Backend servers, so you cant say this is a patch without any
> reason befind it
I wasn't claiming there's no reason behind it, just asking if there are
any, for documentation's sake.
The more salient problem with the patch is the lack of tests. It should
be easy to prove this change's correctness.