Re: [PING 2] [PATCH] libstdc++: basic_filebuf: don't flush more often than necessary.

2022-10-11 Thread Charles-François Natali via Gcc-patches
On Thu, Oct 6, 2022, 17:56 Jonathan Wakely wrote: > > I actually just copy-pasted the header from another test, would it be > simpler if i just removed it? > > > Yes, that's probably the simplest solution, and then add a > Signed-off-by: tag in your patch email, to state you're contributing > it

Re: [PING 2] [PATCH] libstdc++: basic_filebuf: don't flush more often than necessary.

2022-10-06 Thread Jonathan Wakely via Gcc-patches
On Thu, 6 Oct 2022 at 17:33, Charles-François Natali wrote: > > On Thu, Oct 6, 2022, 14:29 Jonathan Wakely wrote: >> >> Sorry for the lack of review. I've been trying to remember (and find) >> some previous discussions related to this topic, but haven't managed >> to find it yet. > > > No worries

Re: [PING 2] [PATCH] libstdc++: basic_filebuf: don't flush more often than necessary.

2022-10-06 Thread Charles-François Natali via Gcc-patches
On Thu, Oct 6, 2022, 14:29 Jonathan Wakely wrote: > Sorry for the lack of review. I've been trying to remember (and find) > some previous discussions related to this topic, but haven't managed > to find it yet. > No worries! > The patch does look sensible (and is the same as the one attached

Re: [PING 2] [PATCH] libstdc++: basic_filebuf: don't flush more often than necessary.

2022-10-06 Thread Jonathan Wakely via Gcc-patches
Sorry for the lack of review. I've been trying to remember (and find) some previous discussions related to this topic, but haven't managed to find it yet. The patch does look sensible (and is the same as the one attached to PR 63746) so I'll make sure to review it in time for the GCC 13 cut-off.

Re: [PING 2] [PATCH] libstdc++: basic_filebuf: don't flush more often than necessary.

2022-10-06 Thread Charles-François Natali via Gcc-patches
On Thu, Sep 22, 2022, 17:51 Charles-François Natali wrote: > > On Mon, Sep 5, 2022, 23:51 Charles-Francois Natali > wrote: > >> `basic_filebuf::xsputn` would bypass the buffer when passed a chunk of >> size 1024 and above, seemingly as an optimisation. >> >> This can have a significant performan