Remove useless call to setContentLength() (#6715)

devyte · web-flow · commit ffa9e3c7c2d8 · 2019-11-05T21:50:52.000-03:00
* Update ESP8266WebServer-impl.h

Remove useless call to setContentLength()

* Fix warning unused param
diff --git a/libraries/ESP8266WebServer/src/ESP8266WebServer-impl.h b/libraries/ESP8266WebServer/src/ESP8266WebServer-impl.h
@@ -533,7 +533,7 @@ template <typename ServerType>
 void ESP8266WebServerTemplate<ServerType>::_streamFileCore(const size_t fileSize, const String &fileName, const String &contentType)
 {
   using namespace mime;
-  setContentLength(fileSize);
+  (void)fileSize;
   if (fileName.endsWith(String(FPSTR(mimeTable[gz].endsWith))) &&
       contentType != String(FPSTR(mimeTable[gz].mimeType)) &&
       contentType != String(FPSTR(mimeTable[none].mimeType))) {

Original file line number	Diff line number	Diff line change
`@@ -533,7 +533,7 @@ template <typename ServerType>`
`533`	`533`	`void ESP8266WebServerTemplate<ServerType>::_streamFileCore(const size_t fileSize, const String &fileName, const String &contentType)`
`534`	`534`	`{`
`535`	`535`	`using namespace mime;`
`536`		`- setContentLength(fileSize);`
	`536`	`+ (void)fileSize;`
`537`	`537`	`if (fileName.endsWith(String(FPSTR(mimeTable[gz].endsWith))) &&`
`538`	`538`	`contentType != String(FPSTR(mimeTable[gz].mimeType)) &&`
`539`	`539`	`contentType != String(FPSTR(mimeTable[none].mimeType))) {`