colin/NetworkManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

lgtm: suppress lgtm[cpp/duplicate-include-guard] warning in ethtool header

Browse Source 
lgtm.com doesn't like this:

  Query pack:com.lgtm/cpp-queries
  Query ID:cpp/duplicate-include-guard

  Using the same include guard macro in more than one header file may
  cause unexpected behavior from the compiler.

both for src/libnm-base/nm-ethtool-utils-base.h and
src/libnm-client-public/nm-ethtool-utils.h. But this is intentional,
because these two files are supposed to be identical (but compiled
twice, under different context).

Suppress the warning.
This commit is contained in:
Thomas Haller
2021-06-01 17:48:26 +02:00
parent dd4cccbe66
commit bae22a45d8
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/libnm-base/nm-ethtool-utils-base.h
View File

@@ -3,7 +3,7 @@
* Copyright (C) 2018 Red Hat, Inc.
*/
#ifndef __NM_ETHTOOL_UTILS_H__
#ifndef __NM_ETHTOOL_UTILS_H__ /* lgtm[cpp/duplicate-include-guard] */
#define __NM_ETHTOOL_UTILS_H__
G_BEGIN_DECLS

2
src/libnm-client-public/nm-ethtool-utils.h
View File

@@ -3,7 +3,7 @@
* Copyright (C) 2018 Red Hat, Inc.
*/
#ifndef __NM_ETHTOOL_UTILS_H__
#ifndef __NM_ETHTOOL_UTILS_H__ /* lgtm[cpp/duplicate-include-guard] */
#define __NM_ETHTOOL_UTILS_H__
G_BEGIN_DECLS