diff options
author | David Robillard <d@drobilla.net> | 2020-09-27 18:17:47 +0200 |
---|---|---|
committer | David Robillard <d@drobilla.net> | 2020-09-27 18:17:47 +0200 |
commit | 9e46d4a8323e9a91480631980a984700b76a09bd (patch) | |
tree | bca7eff6274406c79e430917f7d1729e7dad773d | |
parent | 9b9ca021d6401097d32d96d27de6f906a0252109 (diff) | |
download | lv2-9e46d4a8323e9a91480631980a984700b76a09bd.tar.xz |
Don't use else after return
-rw-r--r-- | .clang-tidy | 1 | ||||
-rw-r--r-- | lv2/midi/midi.h | 8 | ||||
-rw-r--r-- | plugins/eg-params.lv2/state_map.h | 5 | ||||
-rw-r--r-- | plugins/eg-sampler.lv2/uris.h | 12 |
4 files changed, 18 insertions, 8 deletions
diff --git a/.clang-tidy b/.clang-tidy index 4026e2b..e354d02 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -8,7 +8,6 @@ Checks: > -clang-diagnostic-unused-macros, -hicpp-signed-bitwise, -llvm-header-guard, - -readability-else-after-return, WarningsAsErrors: '*' HeaderFilterRegex: '.*' FormatStyle: file diff --git a/lv2/midi/midi.h b/lv2/midi/midi.h index 0abfba1..3a9583b 100644 --- a/lv2/midi/midi.h +++ b/lv2/midi/midi.h @@ -216,11 +216,13 @@ static inline LV2_Midi_Message_Type lv2_midi_message_type(const uint8_t* msg) { if (lv2_midi_is_voice_message(msg)) { return (LV2_Midi_Message_Type)(msg[0] & 0xF0); - } else if (lv2_midi_is_system_message(msg)) { + } + + if (lv2_midi_is_system_message(msg)) { return (LV2_Midi_Message_Type)msg[0]; - } else { - return LV2_MIDI_MSG_INVALID; } + + return LV2_MIDI_MSG_INVALID; } #ifdef __cplusplus diff --git a/plugins/eg-params.lv2/state_map.h b/plugins/eg-params.lv2/state_map.h index 740f0af..9e7b63e 100644 --- a/plugins/eg-params.lv2/state_map.h +++ b/plugins/eg-params.lv2/state_map.h @@ -36,9 +36,12 @@ state_map_cmp(const void* a, const void* b) const StateMapItem* kb = (const StateMapItem*)b; if (ka->urid < kb->urid) { return -1; - } else if (kb->urid < ka->urid) { + } + + if (kb->urid < ka->urid) { return 1; } + return 0; } diff --git a/plugins/eg-sampler.lv2/uris.h b/plugins/eg-sampler.lv2/uris.h index fd80c3f..57e5754 100644 --- a/plugins/eg-sampler.lv2/uris.h +++ b/plugins/eg-sampler.lv2/uris.h @@ -127,10 +127,14 @@ read_set_file(const SamplerURIs* uris, if (!property) { fprintf(stderr, "Malformed set message has no body.\n"); return NULL; - } else if (property->type != uris->atom_URID) { + } + + if (property->type != uris->atom_URID) { fprintf(stderr, "Malformed set message has non-URID property.\n"); return NULL; - } else if (((const LV2_Atom_URID*)property)->body != uris->eg_sample) { + } + + if (((const LV2_Atom_URID*)property)->body != uris->eg_sample) { fprintf(stderr, "Set message for unknown property.\n"); return NULL; } @@ -141,7 +145,9 @@ read_set_file(const SamplerURIs* uris, if (!value) { fprintf(stderr, "Malformed set message has no value.\n"); return NULL; - } else if (value->type != uris->atom_Path) { + } + + if (value->type != uris->atom_Path) { fprintf(stderr, "Set message value is not a Path.\n"); return NULL; } |