diff options
author | Nguyễn Thái Ngọc Duy <pclouds@gmail.com> | 2018-04-14 19:19:43 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2018-04-16 13:54:53 +0900 |
commit | d2bff22c2369289eac4d17654c464e392e726b3e (patch) | |
tree | 5ae45677a9cc495b7348ffeb2056c7c09da9d4e6 /mergetools/winmerge | |
parent | Git 2.16.3 (diff) | |
download | tgif-d2bff22c2369289eac4d17654c464e392e726b3e.tar.xz |
connect.c: mark die_initial_contact() NORETURN
There is a series running in parallel with this one that adds code
like this
switch (...) {
case ...:
die_initial_contact();
case ...:
There is nothing wrong with this. There is no actual falling
through. But since gcc is not that smart and gcc 7.x introduces
-Wimplicit-fallthrough, it raises a false alarm in this case.
This class of warnings may be useful elsewhere, so instead of
suppressing the whole class, let's try to fix just this code. gcc is
smart enough to realize that no execution can continue after a
NORETURN function call and no longer raises the warning.
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'mergetools/winmerge')
0 files changed, 0 insertions, 0 deletions