Remove redundant lower-bound assert for parsing state (#205)

TomasGalbickaNXP · web-flow · commit 281a624f09c8 · 2026-04-10T15:30:31.000-07:00
Some toolchains (e.g. IAR with warnings-as-errors) treat the enum range as constrained, so parsingState &gt;= HTTP_PARSING_NONE (0) is always true and triggers a diagnostic.

Signed-off-by: Tomas Galbicka &lt;tomas.galbicka@nxp.com&gt;
diff --git a/source/core_http_client.c b/source/core_http_client.c
@@ -1994,8 +1994,7 @@ static HTTPStatus_t getFinalResponseStatus( HTTPParsingState_t parsingState,
 {
     HTTPStatus_t returnStatus = HTTPSuccess;
 
-    assert( parsingState >= HTTP_PARSING_NONE &&
-            parsingState <= HTTP_PARSING_COMPLETE );
+    assert( parsingState <= HTTP_PARSING_COMPLETE );
     assert( totalReceived <= pResponse->bufferLen );
 
     /* If no parsing occurred, that means network data was never received. */

Original file line number	Diff line number	Diff line change
`@@ -1994,8 +1994,7 @@ static HTTPStatus_t getFinalResponseStatus( HTTPParsingState_t parsingState,`
`1994`	`1994`	`{`
`1995`	`1995`	`HTTPStatus_t returnStatus = HTTPSuccess;`
`1996`	`1996`
`1997`		`- assert( parsingState >= HTTP_PARSING_NONE &&`
`1998`		`- parsingState <= HTTP_PARSING_COMPLETE );`
	`1997`	`+ assert( parsingState <= HTTP_PARSING_COMPLETE );`
`1999`	`1998`	`assert( totalReceived <= pResponse->bufferLen );`
`2000`	`1999`
`2001`	`2000`	`/* If no parsing occurred, that means network data was never received. */`