On 5/19/20 1:03 PM, Martin Sebor wrote:
I'm having trouble with the commit hook that tries to enforce
ChangeLog contents.  It fails with an error that doesn't make
sense to me: the file it complains isn't mentioned clearly is
listed there and I can't tell what about how it's mentioned
the hook is having a problem with.

Got it!  The file name corresponding to cp-tree.h is, of course,
tree.c and not cp-tree.c.  This is not to be confused with the C
front end where it would be c-tree.c (if it existed there).

Thanks
Martin

$ git push
Enumerating objects: 23, done.
Counting objects: 100% (23/23), done.
Delta compression using up to 16 threads
Compressing objects: 100% (12/12), done.
Writing objects: 100% (12/12), 2.41 KiB | 2.41 MiB/s, done.
Total 12 (delta 11), reused 0 (delta 0)
remote: *** ChangeLog format failed:
remote: ERR: file not changed in a patch:"gcc/cp/cp-tree.c"
remote: ERR: changed file not mentioned in a ChangeLog:"gcc/cp/tree.c"
remote:
remote: Please see: https://gcc.gnu.org/codingconventions.html#ChangeLogs
remote:
remote: error: hook declined to update refs/heads/master
To git+ssh://gcc.gnu.org/git/gcc.git
  ! [remote rejected]         master -> master (hook declined)
error: failed to push some refs to 'git+ssh://mse...@gcc.gnu.org/git/gcc.git'

$ head gcc/cp/ChangeLog
2020-05-18  Martin Sebor  <mse...@redhat.com>

     PR c++/94923
     * call.c ((maybe_warn_class_memaccess): Use is_byte_access_type.
     * cp-tree.h (is_dummy_object): Return bool.
     (is_byte_access_type): Declare new function.
     * cp-tree.c (is_dummy_object): Return bool.
     (is_byte_access_type): Define new function.

2020-05-19  Patrick Palka  <ppa...@redhat.com>

Reply via email to