aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorDavid Robillard <d@drobilla.net>2020-11-09 12:24:49 +0100
committerDavid Robillard <d@drobilla.net>2020-11-09 20:03:10 +0100
commitd5f8dc60b7a08de3b6db21679b02fd2e358eb72c (patch)
treee74824aa7fbb6625f2faf64f2d3caac1ed149bb0 /include
parent0f2ec1ed73e68602c92db917fbb131ab61160239 (diff)
downloadserd-d5f8dc60b7a08de3b6db21679b02fd2e358eb72c.tar.gz
serd-d5f8dc60b7a08de3b6db21679b02fd2e358eb72c.tar.bz2
serd-d5f8dc60b7a08de3b6db21679b02fd2e358eb72c.zip
Fix clang-tidy hicpp-signed-bitwise warnings in header
In exchange for suppressing a pointless superficial warning instead, but I dislike that convention since it looks confusing when hex digits are used and it's inconsistent with near-universal floating point suffix convention.
Diffstat (limited to 'include')
-rw-r--r--include/.clang-tidy2
-rw-r--r--include/serd/serd.h30
2 files changed, 16 insertions, 16 deletions
diff --git a/include/.clang-tidy b/include/.clang-tidy
index f8a0d67b..db89f1f9 100644
--- a/include/.clang-tidy
+++ b/include/.clang-tidy
@@ -1,7 +1,7 @@
Checks: >
*,
+ -*-uppercase-literal-suffix,
-clang-diagnostic-unused-macros,
- -hicpp-signed-bitwise,
WarningsAsErrors: '*'
HeaderFilterRegex: '.*'
FormatStyle: file
diff --git a/include/serd/serd.h b/include/serd/serd.h
index c1e8ba8e..514e0e1a 100644
--- a/include/serd/serd.h
+++ b/include/serd/serd.h
@@ -132,14 +132,14 @@ typedef enum {
Flags indicating inline abbreviation information for a statement.
*/
typedef enum {
- SERD_EMPTY_S = 1 << 1, /**< Empty blank node subject */
- SERD_EMPTY_O = 1 << 2, /**< Empty blank node object */
- SERD_ANON_S_BEGIN = 1 << 3, /**< Start of anonymous subject */
- SERD_ANON_O_BEGIN = 1 << 4, /**< Start of anonymous object */
- SERD_ANON_CONT = 1 << 5, /**< Continuation of anonymous node */
- SERD_LIST_S_BEGIN = 1 << 6, /**< Start of list subject */
- SERD_LIST_O_BEGIN = 1 << 7, /**< Start of list object */
- SERD_LIST_CONT = 1 << 8 /**< Continuation of list */
+ SERD_EMPTY_S = 1u << 1u, /**< Empty blank node subject */
+ SERD_EMPTY_O = 1u << 2u, /**< Empty blank node object */
+ SERD_ANON_S_BEGIN = 1u << 3u, /**< Start of anonymous subject */
+ SERD_ANON_O_BEGIN = 1u << 4u, /**< Start of anonymous object */
+ SERD_ANON_CONT = 1u << 5u, /**< Continuation of anonymous node */
+ SERD_LIST_S_BEGIN = 1u << 6u, /**< Start of list subject */
+ SERD_LIST_O_BEGIN = 1u << 7u, /**< Start of list object */
+ SERD_LIST_CONT = 1u << 8u /**< Continuation of list */
} SerdStatementFlag;
/**
@@ -205,8 +205,8 @@ typedef enum {
Flags indicating certain string properties relevant to serialisation.
*/
typedef enum {
- SERD_HAS_NEWLINE = 1, /**< Contains line breaks ('\\n' or '\\r') */
- SERD_HAS_QUOTE = 1 << 1 /**< Contains quotes ('"') */
+ SERD_HAS_NEWLINE = 1u << 0u, /**< Contains line breaks ('\\n' or '\\r') */
+ SERD_HAS_QUOTE = 1u << 1u /**< Contains quotes ('"') */
} SerdNodeFlag;
/**
@@ -270,11 +270,11 @@ typedef struct {
always ASCII).
*/
typedef enum {
- SERD_STYLE_ABBREVIATED = 1, /**< Abbreviate triples when possible. */
- SERD_STYLE_ASCII = 1 << 1, /**< Escape all non-ASCII characters. */
- SERD_STYLE_RESOLVED = 1 << 2, /**< Resolve URIs against base URI. */
- SERD_STYLE_CURIED = 1 << 3, /**< Shorten URIs into CURIEs. */
- SERD_STYLE_BULK = 1 << 4 /**< Write output in pages. */
+ SERD_STYLE_ABBREVIATED = 1u << 0u, /**< Abbreviate triples when possible. */
+ SERD_STYLE_ASCII = 1u << 1u, /**< Escape all non-ASCII characters. */
+ SERD_STYLE_RESOLVED = 1u << 2u, /**< Resolve URIs against base URI. */
+ SERD_STYLE_CURIED = 1u << 3u, /**< Shorten URIs into CURIEs. */
+ SERD_STYLE_BULK = 1u << 4u, /**< Write output in pages. */
} SerdStyle;
/**