Merge pull request #23 from LeeLeahy2/enable-disable

Replace sempSetPrint* with sempEnable*Output and sempDisable*Output

Author: nseidle

Examples/Base_Test/Base_Test.ino

@@ -121,7 +121,7 @@ void setup()
     Serial.printf("Parser Name: %s\r\n", sempGetTypeName(parse, parse->type));
 
     // Obtain a raw data stream from somewhere
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     Serial.printf("Raw data stream: %d bytes\r\n", RAW_DATA_BYTES);
 
     // The raw data stream is passed to the parser one byte at a time

Examples/Mixed_Parser/Mixed_Parser.ino

@@ -178,7 +178,7 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", rawDataBytes);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     for (dataIndex = 0; dataIndex < DATA_STREAM_ENTRIES; dataIndex++)
     {
         for (int offset = 0; offset < dataStream[dataIndex].length; offset++)

Examples/Multiple_Parsers/Multiple_Parsers.ino

@@ -193,8 +193,8 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", rawDataBytes);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(nmeaParser, &Serial);
-    sempSetPrintDebug(ubloxParser, &Serial);
+    sempEnableDebugOutput(nmeaParser);
+    sempEnableDebugOutput(ubloxParser);
     for (dataIndex = 0; dataIndex < DATA_STREAM_ENTRIES; dataIndex++)
     {
         for (int offset = 0; offset < dataStream[dataIndex].length; offset++)

Examples/NMEA_Test/NMEA_Test.ino

@@ -104,7 +104,7 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", RAW_DATA_BYTES);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     for (dataOffset = 0; dataOffset < RAW_DATA_BYTES; dataOffset++)
         // Update the parser state based on the incoming byte
         sempParseNextByte(parse, rawDataStream[dataOffset]);

Examples/RTCM_Test/RTCM_Test.ino

@@ -99,7 +99,7 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", RAW_DATA_BYTES);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     for (dataOffset = 0; dataOffset < RAW_DATA_BYTES; dataOffset++)
         // Update the parser state based on the incoming byte
         sempParseNextByte(parse, rawDataStream[dataOffset]);

Examples/UBLOX_Test/UBLOX_Test.ino

@@ -141,7 +141,7 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", RAW_DATA_BYTES);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     for (dataOffset = 0; dataOffset < RAW_DATA_BYTES; dataOffset++)
         // Update the parser state based on the incoming byte
         sempParseNextByte(parse, rawDataStream[dataOffset]);

Examples/Unicore_Test/Unicore_Test.ino

@@ -147,7 +147,7 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", RAW_DATA_BYTES);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     for (dataOffset = 0; dataOffset < RAW_DATA_BYTES; dataOffset++)
         // Update the parser state based on the incoming byte
         sempParseNextByte(parse, rawDataStream[dataOffset]);

Examples/User_Parser/User_Parser.ino

@@ -167,7 +167,7 @@ void setup()
     Serial.printf("Raw data stream: %d bytes\r\n", RAW_DATA_BYTES);
 
     // The raw data stream is passed to the parser one byte at a time
-    sempSetPrintDebug(parse, &Serial);
+    sempEnableDebugOutput(parse);
     for (dataOffset = 0; dataOffset < RAW_DATA_BYTES; dataOffset++)
     {
         uint8_t data;

src/SparkFun_Extensible_Message_Parser.cpp

@@ -192,18 +192,28 @@ const char * sempGetStateName(const SEMP_PARSE_STATE *parse)
     return "Unknown state";
 }
 
-// Enable or disable debug output.  Specify nullptr to disable output.
-void sempSetPrintDebug(SEMP_PARSE_STATE *parse, Print *printDebug)
+// Disable debug output
+void sempDisableDebugOutput(SEMP_PARSE_STATE *parse)
 {
-    // Set the class to print debug output
-    parse->printDebug = printDebug;
+    parse->printDebug = nullptr;
 }
 
-// Enable or disable error output.  Specify nullptr to disable output.
-void sempSetPrintError(SEMP_PARSE_STATE *parse, Print *printError)
+// Enable debug output
+void sempEnableDebugOutput(SEMP_PARSE_STATE *parse, Print *print)
 {
-    // Set the class to print error output
-    parse->printError = printError;
+    parse->printDebug = print;
+}
+
+// Disable error output
+void sempDisableErrorOutput(SEMP_PARSE_STATE *parse)
+{
+    parse->printError = nullptr;
+}
+
+// Enable error output
+void sempEnableErrorOutput(SEMP_PARSE_STATE *parse, Print *print)
+{
+    parse->printError = print;
 }
 
 //----------------------------------------

src/SparkFun_Extensible_Message_Parser.h

@@ -212,11 +212,13 @@ const char * sempGetStateName(const SEMP_PARSE_STATE *parse);
 // Translate the type value into an ASCII type name
 const char * sempGetTypeName(SEMP_PARSE_STATE *parse, uint16_t type);
 
-// Enable or disable debug output.  Specify nullptr to disable output.
-void sempSetPrintDebug(SEMP_PARSE_STATE *parse, Print *printDebug);
+// Enable or disable debug output
+void sempEnableDebugOutput(SEMP_PARSE_STATE *parse, Print *print = &Serial);
+void sempDisableDebugOutput(SEMP_PARSE_STATE *parse);
 
-// Enable or disable error output.  Specify nullptr to disable output.
-void sempSetPrintError(SEMP_PARSE_STATE *parse, Print *printError);
+// Enable or disable error output
+void sempEnableErrorOutput(SEMP_PARSE_STATE *parse, Print *print = &Serial);
+void sempDisableErrorOutput(SEMP_PARSE_STATE *parse);
 
 // The parser routines within a parser module are typically placed in
 // reverse order within the module.  This lets the routine declaration