Change 'sprintf' to 'snprintf' to Fix Compiler Warning #63
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TL;DR
This PR fixes the compiler warning that occurs due to the following line of code:
sprintf(maErrorStr, "No packet received. %s.", maErrorStr);
Problem
Simply adding "No packet received. %.*s" to maErrorStr results in a warning, regardless of how large the static-array member variable is.
Warning Issues
Initially I attempted to rectify the warning by changing to the following:
snprintf(maErrorStr, 1060, "No packet received. %s.", maErrorStr);
However, this resulted in the following output:
Building with Ubuntu Terminal