spiffs seek(..., SeekEnd) doesn't work

[brianjmurrell]

Basic Infos

Hardware

Hardware: NodeMCU
Core Version: 2.4.0 I believe

Description

f.seek(80, SeekEnd); doesn't actually seek from the end

Settings in IDE

Module: NodeMCU 1.0
Flash Size: 4MB/3MB
CPU Frequency: 80Mhz
Flash Mode: not sure
Flash Frequency: not sure
Upload Using: SERIAL
Reset Method: not sure

Sketch

#include <FS.h>

void setup() {
  SPIFFS.begin();
  File f = SPIFFS.open("/stats.txt", "r");
  f.seek(80, SeekEnd);
  Serial.println(f.size());
  Serial.println(f.position());
  f.seek(f.size() - 80);
  Serial.println(f.position());
}

void loop() {
// NA
}

Debug Messages

2041660
2041660
2041580

As you can see the seek(80, SeekEnd) didn't actually work and seeked to the end, not 80 bytes from the end. The seek(f.size() - 80) did what the seek(80, SeekEnd) should have done.

[igrr]

Probably duplicate of #2597.

…

On Sat, Feb 3, 2018, 23:54 Brian J. Murrell ***@***.***> wrote: Basic Infos Hardware Hardware: NodeMCU Core Version: 2.4.0 I believe Description f.seek(80, SeekEnd); doesn't actually seek from the end Settings in IDE Module: NodeMCU 1.0 Flash Size: 4MB/3MB CPU Frequency: 80Mhz Flash Mode: not sure Flash Frequency: not sure Upload Using: SERIAL Reset Method: not sure Sketch #include <FS.h> void setup() { SPIFFS.begin(); File f = SPIFFS.open("/stats.txt", "r"); f.seek(80, SeekEnd); Serial.println(f.size()); Serial.println(f.position()); f.seek(f.size() - 80); Serial.println(f.position()); } void loop() {// NA } Debug Messages 2041660 2041660 2041580 As you can see the seek(80, SeekEnd) didn't actually work and seeked to the end, not 80 bytes from the end. The seek(f.size() - 80) did what the seek(80, SeekEnd) should have done. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#4285>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEJceiMowvvYSVeEdhXu8Sv-UFM00b6Nks5tRIEbgaJpZM4R4Nrl> .

[devyte]

It is. I'll pull the other one into 2.4.1 and make a PR.
Closing this as duplicate.

[devyte]

@brianjmurrell please try #4287 .