Changes according to review
This commit is contained in:
parent
8479ed0022
commit
ca3829360e
|
@ -120,8 +120,8 @@ static displayPort_t *cmsDisplayPortSelectNext(void)
|
||||||
//
|
//
|
||||||
|
|
||||||
#define LEFT_MENU_COLUMN 1
|
#define LEFT_MENU_COLUMN 1
|
||||||
#define RIGHT_MENU_COLUMN(p) ((uint8_t)(p->cols - 8))
|
#define RIGHT_MENU_COLUMN(p) ((p)->cols - 8)
|
||||||
#define MAX_MENU_ITEMS(p) (p->rows - 2)
|
#define MAX_MENU_ITEMS(p) ((p)->rows - 2)
|
||||||
|
|
||||||
static bool cmsInMenu = false;
|
static bool cmsInMenu = false;
|
||||||
|
|
||||||
|
|
|
@ -69,7 +69,7 @@ typedef struct
|
||||||
|
|
||||||
#define IS_PRINTVALUE(p) ((p)->flags & PRINT_VALUE)
|
#define IS_PRINTVALUE(p) ((p)->flags & PRINT_VALUE)
|
||||||
#define SET_PRINTVALUE(p) { (p)->flags |= PRINT_VALUE; }
|
#define SET_PRINTVALUE(p) { (p)->flags |= PRINT_VALUE; }
|
||||||
#define CLR_PRINTVALUE(p) { (p)->flags &= (uint8_t)~PRINT_VALUE; }
|
#define CLR_PRINTVALUE(p) { (p)->flags &= ~PRINT_VALUE; }
|
||||||
|
|
||||||
#define IS_PRINTLABEL(p) ((p)->flags & PRINT_LABEL)
|
#define IS_PRINTLABEL(p) ((p)->flags & PRINT_LABEL)
|
||||||
#define SET_PRINTLABEL(p) { (p)->flags |= PRINT_LABEL; }
|
#define SET_PRINTLABEL(p) { (p)->flags |= PRINT_LABEL; }
|
||||||
|
|
|
@ -17,9 +17,12 @@
|
||||||
|
|
||||||
#pragma once
|
#pragma once
|
||||||
|
|
||||||
// type conversion warnings. -Wconversion can be turned on to enable the process of eliminating these warnings
|
// type conversion warnings.
|
||||||
//#pragma GCC diagnostic error "-Wconversion"
|
// -Wconversion can be turned on to enable the process of eliminating these warnings
|
||||||
|
//#pragma GCC diagnostic warning "-Wconversion"
|
||||||
#pragma GCC diagnostic ignored "-Wsign-conversion"
|
#pragma GCC diagnostic ignored "-Wsign-conversion"
|
||||||
|
// -Wpadded can be turned on to check padding of structs
|
||||||
|
//#pragma GCC diagnostic warning "-Wpadded"
|
||||||
|
|
||||||
//#define SCHEDULER_DEBUG // define this to use scheduler debug[] values. Undefined by default for performance reasons
|
//#define SCHEDULER_DEBUG // define this to use scheduler debug[] values. Undefined by default for performance reasons
|
||||||
#define DEBUG_MODE DEBUG_NONE // change this to change initial debug mode
|
#define DEBUG_MODE DEBUG_NONE // change this to change initial debug mode
|
||||||
|
|
Loading…
Reference in New Issue