SD Lib in IDE v1.6.13 ?

International Deutsch
#1

Hallo,

da ich mit jeder neuen IDE Version einen kleinen SD Bugfix mit rübernehme, fiel mir jetzt was auf. Irgendwas für mich unklares wurde geändert.

In Funktion Deklarationen wurden Variablen auskommentiert, sodass der Datentyp alleine da steht. Ich vergleiche das mit notepadd++ und dem compare plugin.

Zeile 236 vorher:

boolean callback_pathExists(SdFile& parentDir, const char *filePathComponent, 
                boolean isLastComponent, void *object) {

aktuell:

boolean callback_pathExists(SdFile& parentDir, const char *filePathComponent, 
                boolean /* isLastComponent */, void * /* object */) {

Bei Zeile 314 gibts noch 2 Funktion worin object auskommentiert wird. Dann steht void* alleine da.

boolean callback_remove(SdFile& parentDir, const char *filePathComponent, 
            boolean isLastComponent, void * /* object */) {

Was ist das für ein Trick? Oder wurde was ausversehen versaut?

#2

Hallo,

die Codestellen sehen mir eher nach den Definitionen der Funktionen aus, nicht nach Deklarationen, wegen der öffnenden Klammern am Ende.

Kann es sein, dass diese Parameter nicht mehr verwendet werden, aber aus Kompatibilitätsgründen (aufrufender Code) weiter vorhanden sein müssen ? Wenn sie keinen Namen mehr haben, kann man sie im Code nicht mehr verwenden ohne eine Fehlermeldung zu bekommen.

#3

Hallo,

ja okay, ist eine Definition. Mal wieder verwechselt.

Die komplette Funktion in SD.cpp sieht zum Bsp. so aus.

boolean callback_pathExists(SdFile& parentDir, const char *filePathComponent, 
                boolean /* isLastComponent */, void * /* object */) {
  
  SdFile child;

  boolean exists = child.open(parentDir, filePathComponent, O_RDONLY);
  
  if (exists) {
     child.close(); 
  }
  
  return exists;
}

Du meinst eine Art Platzhalter der ohne Schaden ins leere führt, verstehe. In der Funktion werden sie nicht verwendet.

#4

Hallo,

ein Problem habe ich noch. Die IDE meckert rum wegen dem nicht initialisierten int pathidx Was open.constprop sein soll weiß ich nicht, die Funktion gibts nicht. Seltsamerweise wird das nicht initialisierte int pathidx erst mit der aktuellen IDE Version angemeckert. Die Funktion steht so auch in der alten schon drin. Es müßte doch reichen wenn ich int pathidx = 0; schreibe?

[u]Edit:[/u] habs geändert, kompiliert erstmal fehlerfrei.

C:\Program Files (x86)\Arduino\libraries\SD\src\SD.cpp: In function 'open.constprop':

C:\Program Files (x86)\Arduino\libraries\SD\src\SD.cpp:442:12: warning: 'pathidx' may be used uninitialized in this function [-Wmaybe-uninitialized]

filepath += pathidx;

^

C:\Program Files (x86)\Arduino\libraries\SD\src\SD.cpp:436:7: note: 'pathidx' was declared here

int pathidx;

^

File SDClass::open(const char *filepath, uint8_t mode) {
  /*

     Open the supplied file path for reading or writing.

     The file content can be accessed via the `file` property of
     the `SDClass` object--this property is currently
     a standard `SdFile` object from `sdfatlib`.

     Defaults to read only.

     If `write` is true, default action (when `append` is true) is to
     append data to the end of the file.

     If `append` is false then the file will be truncated first.

     If the file does not exist and it is opened for writing the file
     will be created.

     An attempt to open a file for reading that does not exist is an
     error.

   */

  int pathidx;

  // do the interative search
  SdFile parentdir = getParentDir(filepath, &pathidx);
  // no more subdirs!

  filepath += pathidx;

  if (! filepath[0]) {
    // it was the directory itself!
    return File(parentdir, "/");
  }

  // Open the file itself
  SdFile file;

  // failed to open a subdir!
  if (!parentdir.isOpen())
    return File();

  // there is a special case for the Root directory since its a static dir
  if (parentdir.isRoot()) {
    if ( ! file.open(root, filepath, mode)) {
      // failed to open the file :(
      return File();
    }
    // dont close the root!
  } else {
    if ( ! file.open(parentdir, filepath, mode)) {
      return File();
    }
    // close the parent
    parentdir.close();
  }

  if (mode & (O_APPEND | O_WRITE)) 
    file.seekSet(file.fileSize());
  return File(file, filepath);
}
#5

Doc_Arduino: Es müßte doch reichen wenn ich int pathidx = 0; schreibe?

Ich würde sagen ja, so wie der Code aussieht, scheint pathidx ja ein reiner Rückgabewert von getParentDir zu sein. Ist das vielleicht auch so ein "überflüssiger" Parameter ?

Muss ich mir mal bei den großen Teensys ansehen, sonst habe ich nichts mit SD-Karte.

#6

Hallo,

spätestens mit filepath addiert wird er zum Rückgabewert. Wenn dann in pathidx zufälligerweise Müll drin steht, kommt Unsinn raus. Hab pathidx erstmal mit 0 definiert. Lernt man auch so. :)

#7