Skip to content

Commit bac04cc

Browse files
authored
Merge pull request #93 from sparkfun/Fix_compiler_warnings
Fix compiler warnings
2 parents 3649283 + 4c35c92 commit bac04cc

2 files changed

Lines changed: 1 addition & 14 deletions

File tree

src/Parse_SBF.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,6 @@ bool sempSbfPreamble2(SEMP_PARSE_STATE *parse, uint8_t data)
230230
sempPrintStringLn(output, ") bytes, invalid preamble2");
231231
}
232232

233-
SEMP_SBF_VALUES *scratchPad = (SEMP_SBF_VALUES *)parse->scratchPad;
234233
sempInvalidDataCallback(parse);
235234
parse->state = sempFirstByte;
236235
return false;

src/SparkFun_Extensible_Message_Parser.cpp

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,6 @@ void sempErrorOutputEnable(SEMP_PARSE_STATE *parse, SEMP_OUTPUT output)
373373
//----------------------------------------
374374
bool sempFirstByte(SEMP_PARSE_STATE *parse, uint8_t data)
375375
{
376-
int index;
377376
SEMP_PARSER_DESCRIPTION *parseDescripion;
378377

379378
if (parse)
@@ -388,7 +387,7 @@ bool sempFirstByte(SEMP_PARSE_STATE *parse, uint8_t data)
388387
for (parse->type = 0; parse->type < parse->parserCount; parse->type++)
389388
{
390389
// Determine if this parser is able to parse this stream
391-
parseDescripion = parse->parsers[index];
390+
parseDescripion = parse->parsers[parse->type];
392391
if (parseDescripion->preamble(parse, data))
393392
// This parser claims to be able to parse this stream
394393
return true;
@@ -509,7 +508,6 @@ float sempGetF4(const SEMP_PARSE_STATE *parse, size_t offset)
509508
float sempGetF4NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
510509
{
511510
float value;
512-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
513511
memcpy(&value, &parse->buffer[offset], sizeof(value));
514512
return value;
515513
}
@@ -531,7 +529,6 @@ double sempGetF8(const SEMP_PARSE_STATE *parse, size_t offset)
531529
double sempGetF8NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
532530
{
533531
double value;
534-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
535532
memcpy(&value, &parse->buffer[offset], sizeof(value));
536533
return value;
537534
}
@@ -553,7 +550,6 @@ int8_t sempGetI1(const SEMP_PARSE_STATE *parse, size_t offset)
553550
int8_t sempGetI1NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
554551
{
555552
int8_t value;
556-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
557553
value = *(int8_t *)&parse->buffer[offset];
558554
return value;
559555
}
@@ -575,7 +571,6 @@ int16_t sempGetI2(const SEMP_PARSE_STATE *parse, size_t offset)
575571
int16_t sempGetI2NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
576572
{
577573
int16_t value;
578-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
579574
memcpy(&value, &parse->buffer[offset], sizeof(value));
580575
return value;
581576
}
@@ -607,7 +602,6 @@ int32_t sempGetI4(const SEMP_PARSE_STATE *parse, size_t offset)
607602
int32_t sempGetI4NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
608603
{
609604
int32_t value;
610-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
611605
memcpy(&value, &parse->buffer[offset], sizeof(value));
612606
return value;
613607
}
@@ -639,7 +633,6 @@ int sempGetI8Digits(int64_t value)
639633
int64_t sempGetI8NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
640634
{
641635
int64_t value;
642-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
643636
memcpy(&value, &parse->buffer[offset], sizeof(value));
644637
return value;
645638
}
@@ -676,7 +669,6 @@ const char * sempGetString(const SEMP_PARSE_STATE *parse, size_t offset)
676669
//----------------------------------------
677670
const char * sempGetStringNoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
678671
{
679-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
680672
return (const char *)&parse->buffer[offset];
681673
}
682674

@@ -714,7 +706,6 @@ uint8_t sempGetU1(const SEMP_PARSE_STATE *parse, size_t offset)
714706
uint8_t sempGetU1NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
715707
{
716708
uint8_t value;
717-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
718709
value = *(uint8_t *)&parse->buffer[offset];
719710
return value;
720711
}
@@ -736,7 +727,6 @@ uint16_t sempGetU2(const SEMP_PARSE_STATE *parse, size_t offset)
736727
uint16_t sempGetU2NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
737728
{
738729
uint16_t value;
739-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
740730
memcpy(&value, &parse->buffer[offset], sizeof(value));
741731
return value;
742732
}
@@ -766,7 +756,6 @@ int sempGetU4Digits(uint32_t value)
766756
uint32_t sempGetU4NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
767757
{
768758
uint32_t value;
769-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
770759
memcpy(&value, &parse->buffer[offset], sizeof(value));
771760
return value;
772761
}
@@ -806,7 +795,6 @@ int sempGetU8Digits(uint64_t value)
806795
uint64_t sempGetU8NoOffset(const SEMP_PARSE_STATE *parse, size_t offset)
807796
{
808797
uint64_t value;
809-
SEMP_PARSER_DESCRIPTION *parserDescription = parse->parsers[parse->type];
810798
memcpy(&value, &parse->buffer[offset], sizeof(value));
811799
return value;
812800
}

0 commit comments

Comments
 (0)