Hi Alexandre,

Alexandre Ratchov wrote on Fri, Nov 27, 2020 at 07:05:27PM +0100:

> this wording is shorter and more precise and complete.

This looks good mdoc(7)-wise, so go ahead, but please consider
the two nits below when committing.

Yours,
  Ingo


> Index: sio_open.3
> ===================================================================
> RCS file: /cvs/src/lib/libsndio/sio_open.3,v
> retrieving revision 1.51
> diff -u -p -r1.51 sio_open.3
> --- sio_open.3        20 Nov 2020 12:09:45 -0000      1.51
> +++ sio_open.3        27 Nov 2020 18:02:16 -0000
> @@ -387,17 +387,17 @@ bitmasks should always be used.
>  .Ss Starting and stopping the device
>  The
>  .Fn sio_start
> -function puts the device in a waiting state:
> -the device will wait for playback data to be provided
> -(using the
> -.Fn sio_write
> -function).
> -Once enough data is queued to ensure that play buffers
> -will not underrun, actual playback is started automatically.
> -If record mode only is selected, then recording starts
> -immediately.
> +function prepares the devices to start.

As Erico already noticed, i think you need to go back to the singular
here, s/devices/device/.  Otherwise, if changing to the plural is
intentional, a clarification might make sense which other device(s)
apart from .Fa hdl the text intends to talk about.

> +Once the play buffer is full, i.e.

This line should be:

  Once the play buffer is full, i.e.\&

If a dot at the end of an input line is not a full stop,
it needs escaping to prevent excessive horizontal spacing.

> +.Fa sio_par.bufsz
> +samples are queued with
> +.Fn sio_write ,
> +playback starts automatically.
> +If record mode only is selected, then
> +.Fn sio_start
> +starts recording immediately.
>  In full-duplex mode, playback and recording will start
> -synchronously as soon as enough data to play is available.
> +synchronously as soon as the play buffer is full.
>  .Pp
>  The
>  .Fn sio_stop

Reply via email to