FS library position() to return (size_t) -1 on error (espressif#10002)

drmpf · pre-commit-ci-lite[bot] · web-flow · commit 0ab2c58b6c14 · 2024-07-10T13:50:09.000+03:00
* position_fix Fix for error return from position() Issue espressif#9992 * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
diff --git a/libraries/FS/src/FS.cpp b/libraries/FS/src/FS.cpp
@@ -105,7 +105,7 @@ bool File::seek(uint32_t pos, SeekMode mode) {
 
 size_t File::position() const {
   if (!*this) {
-    return 0;
+    return (size_t)-1;
   }
 
   return _p->position();
diff --git a/libraries/FS/src/FS.h b/libraries/FS/src/FS.h
@@ -64,7 +64,7 @@ class File : public Stream {
   bool seek(uint32_t pos) {
     return seek(pos, SeekSet);
   }
-  size_t position() const;
+  size_t position() const;  // returns (size_t)-1 on error
   size_t size() const;
   bool setBufferSize(size_t size);
   void close();

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ bool File::seek(uint32_t pos, SeekMode mode) {`
`105`	`105`
`106`	`106`	`size_t File::position() const {`
`107`	`107`	`if (!*this) {`
`108`		`- return 0;`
	`108`	`+ return (size_t)-1;`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`return _p->position();`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ class File : public Stream {`
`64`	`64`	`bool seek(uint32_t pos) {`
`65`	`65`	`return seek(pos, SeekSet);`
`66`	`66`	`}`
`67`		`- size_t position() const;`
	`67`	`+ size_t position() const; // returns (size_t)-1 on error`
`68`	`68`	`size_t size() const;`
`69`	`69`	`bool setBufferSize(size_t size);`
`70`	`70`	`void close();`