All of lore.kernel.org
 help / color / mirror / Atom feed
* posix_memalign(3)
@ 2023-11-17 22:03 Jonny Grant
  2023-11-24 22:15 ` posix_memalign(3) Alejandro Colomar
  0 siblings, 1 reply; 2+ messages in thread
From: Jonny Grant @ 2023-11-17 22:03 UTC (permalink / raw)
  To: Alejandro Colomar, linux-man

Hello

Some feedback, suggestions.

https://man7.org/linux/man-pages/man3/posix_memalign.3.html
"ENOMEM There was insufficient memory to fulfill the allocation request."

Rather wordy, how about "ENOMEM Out of memory" ? that would match malloc(3)


There are quite a few extra words in the description, how about simplifying:

Orig:
"The function posix_memalign() allocates size bytes and places the
address of the allocated memory in *memptr. "

Simplification:
"posix_memalign() allocates size bytes and places the
allocated memory address in *memptr."

Another one.
Orig:
"The function aligned_alloc() is the same as memalign(), except
for the added restriction that size should be a multiple of
alignment."

simplification:

"aligned_alloc() is the same as memalign(), except
for the added restriction that size should be a multiple of
alignment.

Kind regards, Jonny

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: posix_memalign(3)
  2023-11-17 22:03 posix_memalign(3) Jonny Grant
@ 2023-11-24 22:15 ` Alejandro Colomar
  0 siblings, 0 replies; 2+ messages in thread
From: Alejandro Colomar @ 2023-11-24 22:15 UTC (permalink / raw)
  To: Jonny Grant; +Cc: linux-man

[-- Attachment #1: Type: text/plain, Size: 1222 bytes --]

Hi Jonny,

On Fri, Nov 17, 2023 at 10:03:58PM +0000, Jonny Grant wrote:
> Hello
> 
> Some feedback, suggestions.
> 
> https://man7.org/linux/man-pages/man3/posix_memalign.3.html
> "ENOMEM There was insufficient memory to fulfill the allocation request."
> 
> Rather wordy, how about "ENOMEM Out of memory" ? that would match malloc(3)

LGTM

> 
> 
> There are quite a few extra words in the description, how about simplifying:
> 
> Orig:
> "The function posix_memalign() allocates size bytes and places the
> address of the allocated memory in *memptr. "
> 
> Simplification:
> "posix_memalign() allocates size bytes and places the

LGTM

> allocated memory address in *memptr."

This doesn't convince me.  I've applied patches for all the other
suggestions.

> 
> Another one.
> Orig:
> "The function aligned_alloc() is the same as memalign(), except
> for the added restriction that size should be a multiple of
> alignment."
> 
> simplification:
> 
> "aligned_alloc() is the same as memalign(), except

LGTM

> for the added restriction that size should be a multiple of
> alignment.
> 
> Kind regards, Jonny

Thanks,
Alex

-- 
<https://www.alejandro-colomar.es/>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-11-24 22:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-17 22:03 posix_memalign(3) Jonny Grant
2023-11-24 22:15 ` posix_memalign(3) Alejandro Colomar

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.