Re: [PATCH] getline.3: wfix.

Alejandro Colomar
2022-08-01T12:56:39Z

[-- Attachment #1.1: Type: text/plain, Size: 1013 bytes --]

Hi enh,

On 7/29/22 22:22, enh wrote:
> Clarify in the RETURN VALUE section that it's necessary to free the
> buffer, since at least one person missed this up at the start of the
> DESCRIPTION.
>
> Signed-off-by: Elliott Hughes <enh@google.com>
> ---
> man3/getline.3 | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/man3/getline.3 b/man3/getline.3
> index 8b7357825..bad30956c 100644
> --- a/man3/getline.3
> +++ b/man3/getline.3
> @@ -99,6 +99,10 @@ condition).
> In the event of a failure,
> .I errno
> is set to indicate the error.

.PP

Since the added text has nothing to do with the previous, let's separate
them into separate paragraphs.

> +If
> +.I "*lineptr"

As Branden said, please remove the unnecessary quotes.

> +was set to NULL before the call, then the buffer should be freed by the
> +user program even on failure.
> .SH ERRORS
> .TP
> .B EINVAL

Cheers,

Alex

--
Alejandro Colomar
<http://www.alejandro-colomar.es/>

[-- Attachment #2: OpenPGP digital signature --]