I wasn't claiming there's no reason behind it, just asking if there are any, for documentation's sake
i see many patches going in without documentation (or patches falling of the list without documentation) so I can imagine why he sent in without documentation... .
o
Op 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.