aboutsummaryrefslogtreecommitdiffstats
path: root/utils/cec-compliance/cec-test-audio.cpp
diff options
context:
space:
mode:
authorRosen Penev <rosenp@gmail.com>2020-04-17 14:15:24 -0700
committerHans Verkuil <hverkuil-cisco@xs4all.nl>2020-04-20 14:40:14 +0200
commitce8ed12c95eeb628febfff04abe9fae4d6f410bf (patch)
tree698d0ccf0dbbc229d1410b2842c5b30ae6ef421a /utils/cec-compliance/cec-test-audio.cpp
parent06dc2b0c0e9705d5d4607c89dfc7decf714a68d6 (diff)
treewide: do not use return after else
Found with clang-tidy's readability-else-after-return Signed-off-by: Rosen Penev <rosenp@gmail.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> [hverkuil-cisco@xs4all.nl: split '} if () {' into two lines] [hverkuil-cisco@xs4all.nl: split 'if () continue; else return -1;' into two lines]
Diffstat (limited to 'utils/cec-compliance/cec-test-audio.cpp')
-rw-r--r--utils/cec-compliance/cec-test-audio.cpp8
1 files changed, 4 insertions, 4 deletions
diff --git a/utils/cec-compliance/cec-test-audio.cpp b/utils/cec-compliance/cec-test-audio.cpp
index af472bf4..576d59c6 100644
--- a/utils/cec-compliance/cec-test-audio.cpp
+++ b/utils/cec-compliance/cec-test-audio.cpp
@@ -224,11 +224,11 @@ static int arc_initiate_rx(struct node *node, unsigned me, unsigned la, bool int
announce("The device supports ARC but is not ready to initiate.");
return 0;
}
- else {
- warn("Device responded Feature Abort with unexpected abort reason. Assuming no ARC support.\n");
- unsupported = true;
- }
+
+ warn("Device responded Feature Abort with unexpected abort reason. Assuming no ARC support.\n");
+ unsupported = true;
}
+
if (unsupported) {
fail_on_test_v2(node->remote[la].cec_version, node->remote[la].source_has_arc_rx);
return OK_NOT_SUPPORTED;

Privacy Policy