Removed the unused state of MSTP_RECEIVE_STATE_HEADER_CRC. Hopefully this doesn't break too many implementations!

bacnet-stack/src/mstp.c

@@ -410,7 +410,6 @@ void MSTP_Receive_Frame_FSM(
                         mstp_port->HeaderCRC);
                     mstp_port->HeaderCRCActual = mstp_port->DataRegister;
                     /* don't wait for next state - do it here */
-                    /* MSTP_RECEIVE_STATE_HEADER_CRC */
                     if (mstp_port->HeaderCRC != 0x55) {
                         /* BadCRC */
                         /* indicate that an error has occurred during
@@ -482,13 +481,6 @@ void MSTP_Receive_Frame_FSM(
                 mstp_port->DataAvailable = false;
             }
             break;
-            /* In the HEADER_CRC state, the node validates the CRC on the fixed */
-            /* message header. */
-        case MSTP_RECEIVE_STATE_HEADER_CRC:
-            /* note: we should never get to this state since we shortcut
-               it earlier in the state machine, and never set this state */
-            mstp_port->receive_state = MSTP_RECEIVE_STATE_IDLE;
-            break;
             /* In the DATA state, the node waits for the data portion of a frame. */
         case MSTP_RECEIVE_STATE_DATA:
             /* Timeout */

bacnet-stack/src/mstptext.c

@@ -40,7 +40,6 @@ static INDTEXT_DATA mstp_receive_state_text[] = {
     {MSTP_RECEIVE_STATE_IDLE, "IDLE"},
     {MSTP_RECEIVE_STATE_PREAMBLE, "PREAMBLE"},
     {MSTP_RECEIVE_STATE_HEADER, "HEADER"},
-    {MSTP_RECEIVE_STATE_HEADER_CRC, "HEADER_CRC"},
     {MSTP_RECEIVE_STATE_DATA, "DATA"},
     {0, NULL}
 };