Re: [PATCH] Cygwin: dsp: Implement SNDCTL_DSP_SETFRAGMENT ioctl().

2023-01-31 Thread Corinna Vinschen
On Jan 31 20:20, Takashi Yano wrote: > On Tue, 31 Jan 2023 10:28:40 +0100 > Corinna Vinschen wrote: > > On Jan 30 22:09, Takashi Yano wrote: > > > Previously, SNDCTL_DSP_SETFRAGMENT was just a fake. In this patch, > > > it has been implemented to allow latency control in some apps. > > > > > >

Re: [PATCH] Cygwin: dsp: Implement SNDCTL_DSP_SETFRAGMENT ioctl().

2023-01-31 Thread Corinna Vinschen
On Jan 31 20:18, Takashi Yano wrote: > On Tue, 31 Jan 2023 10:28:40 +0100 > Corinna Vinschen wrote: > > LGTM. Given how much I *don't* use the audio stuff in Cygwin, > > would you just like to take over maintainership for this code? > > Thanks. I could take care of it if you don't mind? Not at

Re: [PATCH] Cygwin: dsp: Implement SNDCTL_DSP_SETFRAGMENT ioctl().

2023-01-31 Thread Takashi Yano
On Tue, 31 Jan 2023 10:28:40 +0100 Corinna Vinschen wrote: > On Jan 30 22:09, Takashi Yano wrote: > > Previously, SNDCTL_DSP_SETFRAGMENT was just a fake. In this patch, > > it has been implemented to allow latency control in some apps. > > > > Reviewed-by: Corinna Vinschen > > Signed-off-by:

Re: [PATCH] Cygwin: dsp: Implement SNDCTL_DSP_SETFRAGMENT ioctl().

2023-01-31 Thread Takashi Yano
On Tue, 31 Jan 2023 10:28:40 +0100 Corinna Vinschen wrote: > LGTM. Given how much I *don't* use the audio stuff in Cygwin, > would you just like to take over maintainership for this code? Thanks. I could take care of it if you don't mind? -- Takashi Yano

Re: [PATCH] Cygwin: dsp: Implement SNDCTL_DSP_SETFRAGMENT ioctl().

2023-01-31 Thread Corinna Vinschen
On Jan 30 22:09, Takashi Yano wrote: > Previously, SNDCTL_DSP_SETFRAGMENT was just a fake. In this patch, > it has been implemented to allow latency control in some apps. > > Reviewed-by: Corinna Vinschen > Signed-off-by: Takashi Yano > --- > winsup/cygwin/fhandler/dsp.cc | 78

[PATCH] Cygwin: dsp: Implement SNDCTL_DSP_SETFRAGMENT ioctl().

2023-01-30 Thread Takashi Yano
Previously, SNDCTL_DSP_SETFRAGMENT was just a fake. In this patch, it has been implemented to allow latency control in some apps. Reviewed-by: Corinna Vinschen Signed-off-by: Takashi Yano --- winsup/cygwin/fhandler/dsp.cc | 78 -