Hi Lorenzo, On Tue, Aug 19, 2025 at 09:37:39PM +0100, Lorenzo Stoakes wrote: > > > diff --git a/man/man2/mremap.2 b/man/man2/mremap.2 > > > index 6d14bf627..53d4eda29 100644 > > > --- a/man/man2/mremap.2 > > > +++ b/man/man2/mremap.2 > > > @@ -47,8 +47,35 @@ The > > > .B MREMAP_DONTUNMAP > > > flag may also be specified. > > > .P > > > -If the operation is not > > > -simply moving mappings, > > > +Equally, if the operation performs a shrink, I've changed s/Equally/Similarly/ > > > +that is if > > > > Missing comma. > > Could you fix that up? Thnks! > > > > > > +.I old_size > > > +is greater than > > > +.IR new_size , > > > +then > > > +.I old_size > > > +may also span multiple mappings > > > +which do not have to be > > > +adjacent to one another. > > > > I'm wondering if there's a missing comma or not before 'which'. > > The meaning of the sentence would be different. > > > > So, I should ask: > > > > Does old_size > new_size mean that old_size may span multiple mappings > > and you're commenting that multiple mappings need not be adjacent? > > Yes. > > > > > Or are multiple mappings always allowed and old_size > new_size allows > > non-adjacent ones? > > No. > > > > > I suspect it's the former, right? Then, it's missing a comma, right? > > Yes could you fix that up? Yup. > > > > > > > Other than this, the patch looks good. > > Thanks! Thanks for the patch! I've applied it with those amendments. <https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=21b1c7188ce1d66ef294eb1681361315a35e8070> Have a lovely day! Alex -- <https://www.alejandro-colomar.es/>
Attachment:
signature.asc
Description: PGP signature
