diff options
| author | Hans Ulrich Niedermann <hun@n-dimensional.de> | 2021-10-05 15:50:32 +0200 |
|---|---|---|
| committer | Hans Ulrich Niedermann <hun@n-dimensional.de> | 2021-10-05 16:07:20 +0200 |
| commit | 0dbeaf7cbf30ab88ac7aea3e7d3f24df8dc0309f (patch) | |
| tree | 5473eb1125e904a86e479180689167280a2e81dc /gphoto2/gphoto2-library.h | |
| parent | bacd4128617e1dd613998c0268ded8304cef6f19 (diff) | |
| download | libgphoto2-0dbeaf7cbf30ab88ac7aea3e7d3f24df8dc0309f.tar.gz | |
Use consistently named non-reserved CPP macros for header files
C preprocessor macros beginning/ending with underscores are
reserved for system use as well as macros containing a double
underscore anywhere, so application software macros are
supposed to be named differently.
For a header file gphoto2-abilities-list.h, this
consistently uses a macro name
LIBGPHOTO2_GPHOTO2_ABILITIES_LIST_H
for the usual
#ifndef LIBGPHOTO2_GPHOTO2_ABILITIES_LIST_H
#define LIBGPHOTO2_GPHOTO2_ABILITIES_LIST_H
... content of the header file ...
#endif /* !defined(LIBGPHOTO2_GPHOTO2_ABILITIES_LIST_H) */
Found using "clang -Weverything".
This should work, unless when run with an ancient C preprocessor
which might only the first 8 or 10 characters to distinguish
macro names and ignores the remainder of the name.
Diffstat (limited to 'gphoto2/gphoto2-library.h')
| -rw-r--r-- | gphoto2/gphoto2-library.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gphoto2/gphoto2-library.h b/gphoto2/gphoto2-library.h index b0f8c530f..3c93fbbdb 100644 --- a/gphoto2/gphoto2-library.h +++ b/gphoto2/gphoto2-library.h @@ -22,8 +22,8 @@ * Boston, MA 02110-1301 USA */ -#ifndef __GPHOTO2_LIBRARY_H__ -#define __GPHOTO2_LIBRARY_H__ +#ifndef LIBGPHOTO2_GPHOTO2_LIBRARY_H +#define LIBGPHOTO2_GPHOTO2_LIBRARY_H #include <gphoto2/gphoto2-abilities-list.h> #include <gphoto2/gphoto2-camera.h> @@ -77,4 +77,4 @@ int camera_init (Camera *camera, GPContext *context); } #endif /* __cplusplus */ -#endif /* __GPHOTO2_LIBRARY_H__ */ +#endif /* !defined(LIBGPHOTO2_GPHOTO2_LIBRARY_H) */ |