Implemented - it does look nicer. Thanks for the feedback.

Some questions.
1/ I'm unsure of the advantage of having it compiled or preprocessed (who cares?)

2/ Why would I limit the #define use to mapping macros, functions etc?
this would seem perfectly legit.

#define NUM_OF_RULES_PER_RECIPE 10


I explicitly init'd the enums as I personally find it easier to remember then, and also if I change the start index, it still looks clear.

typedef enum {
    actionEmpty = 0,
    actionOn,
    actionOff,
    actionRegulate,
    actionPulse
} recipe_action;

typedef enum {
    startEmpty = 0,
    startAbsolute,
    startPriorRule,
    startAnyRule
} recipe_start;

typedef enum {
    feedbackEmpty = 0, 
    feedbackExternalIO,
    feedbackTimeOn, 
    feedbackTimeOff 
} recipe_feedback;

typedef enum {
    progressNotStarted = 0,
    progressInProgress,
    progressCompleted
} recipe_progress;


struct strct_rules{
    unsigned char out_point;
    recipe_action action;
    recipe_start start_condition;
    unsigned char start_time;
    unsigned char stop_time;
    recipe_feedback feedback_rule;
    unsigned char feedback_point;
};




 

Regards
Marshall