Hi Joseph,

On Tue, May 20, 2025 at 02:43:55PM +0000, Joseph Myers wrote:
> > Could you please clarify if I need to do anything or if this is already
> > scheduled for review when you have some time?  Also please clarify if
> > you're okay with amending that or if you prefer that I send v23.
> 
> I have it on my list for review.

Thanks!

>  I'd prefer v23 (also with complete 
> ChangeLog entries, please, rather than the present placeholders, so they 
> don't need to be written at commit time).

The ChangeLog entries that I've sent are supposed to be complete.  Did I
miss anything?

I've based on gnulib commits, which I believe follow the same
guidelines.  For example:

        commit 6608062398ef4c983a58b90a1520c39f12fb7ac1
        Author: Paul Eggert <egg...@cs.ucla.edu>
        Date:   Fri Jan 10 10:34:58 2025 -0800

            doc: document some file system portability issues
            
            * doc/glibc-functions/flistxattr.texi:
            * doc/glibc-functions/listxattr.texi:
            * doc/glibc-functions/llistxattr.texi:
            * doc/posix-functions/fchdir.texi, doc/posix-functions/fstat.texi:
            * doc/posix-functions/fstatvfs.texi:
            Document some portability gotchas that Gnulib does not work around.

Now I realize that maybe my changelog misses the trailing ':' for the
entries that have no text (because it's only once at the end)?  So for
example instead of

    gcc/c-family/ChangeLog:
    
            * c-common.h
            * c-common.def
            * c-common.cc (c_countof_type): Add __countof__ operator.

I should do this?

    gcc/c-family/ChangeLog:
    
            * c-common.h:
            * c-common.def:
            * c-common.cc (c_countof_type): Add __countof__ operator.

Or maybe this?

    gcc/c-family/ChangeLog:
    
            * c-common.h:
            * c-common.def:
            * c-common.cc (c_countof_type):
            Add __countof__ operator.

Please let me know.


Cheers,
Alex

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

Attachment: signature.asc
Description: PGP signature

Reply via email to