diff options
| author | Jack Lloyd <[email protected]> | 2016-01-17 19:02:13 -0500 |
|---|---|---|
| committer | Jack Lloyd <[email protected]> | 2016-01-17 19:10:33 -0500 |
| commit | 6fcbd57915d7870c2782bd6e098903353e8756b0 (patch) | |
| tree | c40505fbf8021819dc15d8a632a2d6fb5557f2aa /src/cli/cc_enc.cpp | |
| parent | de22a9eda8bfb087b690feb962fae8313ee526d4 (diff) | |
Add missing overrides and fix -Wpedantic 'extra ;' warnings
Remove -Wsuggest-attribute=noreturn from maintainer mode flags as it
seems like outside of the assertion failure macro any other suggestion
would always be a false positive (an unimplemented function or the like).
Or at least, if such a function needing noreturn to assist with static
analysis is added in the future it will be obvious, by virtue of the
static analyzer warnings which occur due to the missing noreturn
preventing the analyzer from understanding code flow.
Diffstat (limited to 'src/cli/cc_enc.cpp')
| -rw-r--r-- | src/cli/cc_enc.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cli/cc_enc.cpp b/src/cli/cc_enc.cpp index 6f19e546c..9a3256417 100644 --- a/src/cli/cc_enc.cpp +++ b/src/cli/cc_enc.cpp @@ -109,7 +109,7 @@ class CC_Encrypt final : public Command public: CC_Encrypt() : Command("cc_encrypt CC passphrase --tweak=") {} - void go() + void go() override { const uint64_t cc_number = std::stoull(get_arg("CC")); const std::vector<uint8_t> tweak = Botan::hex_decode(get_arg("tweak")); |