summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTobias Klauser <tklauser@distanz.ch>2014-06-02 13:59:39 +0200
committerTobias Klauser <tklauser@distanz.ch>2014-06-02 13:59:39 +0200
commit96cbba90a5ccf29c2124b8d4d46ed534861ccf44 (patch)
tree24a17fbeb56eb7813c1307ddc2841acaa58384f3
parentc288c3a32659db207db09fdb8beb6ee00e3eaff0 (diff)
mausezahn: Don't use ternary operator to decide which function to call
Replace an odd use of the ternary operator with a more readable if/else. This fixes Debian Bug #750077. Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
-rw-r--r--staging/modifications.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/staging/modifications.c b/staging/modifications.c
index 3dc2abf..08e9fb0 100644
--- a/staging/modifications.c
+++ b/staging/modifications.c
@@ -638,7 +638,11 @@ int print_frame_details()
dum1 = (unsigned char*) &tx.ip_src_h;
dum2 = (unsigned char*) &tx.ip_dst_h;
- (mode==IP) ? (void) bs2str(tx.ip_payload, pld, tx.ip_payload_s) : strcpy(pld, "[see next layer]");
+ if (mode==IP) {
+ (void) bs2str(tx.ip_payload, pld, tx.ip_payload_s);
+ } else {
+ strcpy(pld, "[see next layer]");
+ }
if (ipv6_mode) {
char src6[64]; char dst6[64];