diff options
author | Tobias Klauser <tklauser@distanz.ch> | 2014-06-02 13:59:39 +0200 |
---|---|---|
committer | Tobias Klauser <tklauser@distanz.ch> | 2014-06-02 13:59:39 +0200 |
commit | 96cbba90a5ccf29c2124b8d4d46ed534861ccf44 (patch) | |
tree | 24a17fbeb56eb7813c1307ddc2841acaa58384f3 | |
parent | c288c3a32659db207db09fdb8beb6ee00e3eaff0 (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.c | 6 |
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]; |