From a23eb24f9519629b001d7acdd0760acbb17f4eb4 Mon Sep 17 00:00:00 2001 From: Oliver Hartkopp Date: Sun, 1 Nov 2020 16:51:40 +0100 Subject: [PATCH] cangen: remove local assignment of Classical CAN frame pointer As we have introduced struct can_frame *ccf = (struct can_frame *)&frame; on top level we can remove this local assignment inside the if statement. Signed-off-by: Oliver Hartkopp --- cangen.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/cangen.c b/cangen.c index e5a13f9..5c86f26 100644 --- a/cangen.c +++ b/cangen.c @@ -423,12 +423,10 @@ int main(int argc, char **argv) frame.len = random() & 0xF; if (frame.len > CAN_MAX_DLEN) { - if (len8_dlc) { - struct can_frame *ccf = (struct can_frame *)&frame; - - /* generate Classic CAN len8 DLCs */ + /* generate Classic CAN len8 DLCs? */ + if (len8_dlc) ccf->len8_dlc = frame.len; - } + frame.len = 8; /* for about 50% of the frames */ } }