From: Benjamin Shefte Subject: Re: [PATCH] Increased winsock2 maximum sockets Message-Id: Date: Thu, 27 Jan 2022 12:06:02 -0600 In-Reply-To: References: <20220125142623.1188619-1-shefben@gmail.com> My bad, had kind of a bad day yesterday so my responses may have been a bit short-tempered. I'll add a list of programs and documentations and possibly examples of things that I have used for the limit is too low for my purposes. On Tue, Jan 25, 2022, 6:07 PM Louis Lenders wrote: > >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. >> >>
My bad, had kind of a bad day yesterday so my responses may have been a bit short-tempered.
I'll add a list of programs and documentations and possibly examples of things that I have used for the limit is too low for my purposes.

On Tue, Jan 25, 2022, 6:07 PM Louis Lenders <xerox.xerox2000x@gmail.com> wrote:
>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.