fix include order and unify local include placement

In commit https://github.com/linux-can/can-utils/commit/b153fe3f3591
("clang-tidy: sort includes alphabetically") the position of the "lib.h"
include has been moved in a way that struct canfd_frame has not been
defined anymore and lead to a warning when compiling cangen.c

This patch reverts that specific move and unifies the location of local
includes at the end of the include list.

Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
pull/252/head
Oliver Hartkopp 2020-10-13 10:03:19 +02:00
parent bda4b0ad8c
commit 5d5ccf38ee
4 changed files with 7 additions and 3 deletions

View File

@ -44,7 +44,6 @@
*
*/
#include "canframelen.h"
#include <arpa/inet.h>
#include <assert.h>
#include <stdbool.h>
@ -52,6 +51,8 @@
#include <stdint.h>
#include <string.h>
#include "canframelen.h"
/**
* Functions and types for CRC checks.
*

View File

@ -61,10 +61,11 @@
#include <sys/types.h>
#include <sys/uio.h>
#include "lib.h"
#include <linux/can.h>
#include <linux/can/raw.h>
#include "lib.h"
#define DEFAULT_GAP 200 /* ms */
#define DEFAULT_BURST_COUNT 1

View File

@ -55,11 +55,12 @@
#include <sys/socket.h>
#include <sys/types.h>
#include "terminal.h"
#include <linux/can.h>
#include <linux/can/raw.h>
#include <linux/sockios.h>
#include "terminal.h"
#define NO_CAN_ID 0xFFFFFFFFU
const char fc_info [4][9] = { "CTS", "WT", "OVFLW", "reserved" };

View File

@ -21,6 +21,7 @@
#include <linux/socket.h>
#include "libj1939.h"
#define J1939_MAX_ETP_PACKET_SIZE (7 * 0x00ffffff)
#define JCAT_BUF_SIZE (1000 * 1024)