3d_print_resources/Marlin
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Pretty up, modernize, etc.

Browse source
This commit is contained in:
Scott Lahteine 2020-11-01 01:57:21 -05:00
parent 4eaa200113
commit a8228bde5d
4 changed files with 113 additions and 129 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Marlin/Configuration_adv.h
View file

@ -1256,15 +1256,15 @@
// This allows hosts to request long names for files and folders with M33 // This allows hosts to request long names for files and folders with M33
//#define LONG_FILENAME_HOST_SUPPORT //#define LONG_FILENAME_HOST_SUPPORT
//M20 Configuration // Default M20 File Listing behavior
//#define M20_Reports_Directory_Names // Enable this to report each directory on the SD card in addition to any GCODE files
#if ENABLED(LONG_FILENAME_HOST_SUPPORT) #if ENABLED(LONG_FILENAME_HOST_SUPPORT)
//LONG_FILENAME_HOST_SUPPORT allows M20 to return LONG or DOS8.3 filenames to the host. Default setting can be setup here. -> If both are disabled, M20 defaults to the DOS names #define M20_REPORT_DOS_FILENAMES // Include DOS 8.3 filenames in listings by default
//If both are enabled, then M20 will report back this format for each item: [DOS8.3 Name] --> [Long_FileName] [Size in bytes] //#define M20_REPORT_LONG_FILENAMES // Include long filenames in listings by default
#define M20_Reports_DOS_FileNames
//#define M20_Reports_LONG_FileNames
#endif #endif
// List folders ahead of the .gcode file listing
//#define M20_REPORT_DIRECTORY_NAMES
// Enable this option to scroll long filenames in the SD card menu // Enable this option to scroll long filenames in the SD card menu
//#define SCROLL_LONG_FILENAMES //#define SCROLL_LONG_FILENAMES

36
Marlin/src/gcode/sd/M20.cpp
View file

@ -32,36 +32,12 @@
*/ */
void GcodeSuite::M20() { void GcodeSuite::M20() {
if (card.flag.mounted) { if (card.flag.mounted) {
if ( parser.seen('S' ) and parser.seen('L' ) ) { card.ls(
card.ls(TRUE,TRUE); parser.boolval('S', TERN1(M20_REPORT_LONG_FILENAMES)),
parser.boolval('L', ENABLED(M20_REPORT_DOS_FILENAMES))
} else if ( parser.seen('S' ) ) { );
card.ls(TRUE,FALSE); // Request Short (default DSoc 8,3) filenames if user used 'M20 S' if (DISABLED(LONG_FILENAME_HOST_SUPPORT) && parser.boolval('L'))
SERIAL_ECHOLNPGM("ERROR: LONG_FILENAME_HOST_SUPPORT Not Enabled!");
} else if ( parser.seen('L' ) ) {
#if ENABLED(LONG_FILENAME_HOST_SUPPORT)
card.ls(FALSE,TRUE); // Request Short (default DSoc 8,3) filenames if user used 'M20 S'
#else
SERIAL_ECHOLNPGM("ERROR: Long_FileName_Host_Support Not Enabled!");
#endif
} else {
#if ENABLED(LONG_FILENAME_HOST_SUPPORT)
#if ( ENABLED(M20_Reports_LONG_FileNames) and ENABLED(M20_Reports_DOS_FileNames) )
card.ls(TRUE,TRUE);
#elif ENABLED(M20_Reports_LONG_FileNames)
card.ls(FALSE,TRUE);
#elif ENABLED(M20_Reports_DOS_FileNames)
card.ls(TRUE,FALSE);
#endif
#else
card.ls(TRUE,FALSE);
#endif
}
} }
else else
SERIAL_ECHO_MSG(STR_NO_MEDIA); SERIAL_ECHO_MSG(STR_NO_MEDIA);

179
Marlin/src/sd/cardreader.cpp
View file

@ -57,7 +57,7 @@
#include "../core/debug_out.h" #include "../core/debug_out.h"
#include "../libs/hex_print.h" #include "../libs/hex_print.h"
#include <strings.h> //Required for strcasecmp command to compile without warnings in VS_Code #include <strings.h> // Suppress VSCode warnings about 'strcasecmp'
// public: // public:
@ -175,7 +175,7 @@ char *createFilename(char * const buffer, const dir_t &p) {
bool CardReader::is_dir_or_gcode(const dir_t &p) { bool CardReader::is_dir_or_gcode(const dir_t &p) {
//uint8_t pn0 = p.name[0]; //uint8_t pn0 = p.name[0];
if ( (p.attributes & DIR_ATT_HIDDEN) // Hidden by attribute if ((p.attributes & DIR_ATT_HIDDEN) // Hidden by attribute
// When readDir() > 0 these must be false: // When readDir() > 0 these must be false:
//|| pn0 == DIR_NAME_FREE || pn0 == DIR_NAME_DELETED // Clear or Deleted entry //|| pn0 == DIR_NAME_FREE || pn0 == DIR_NAME_DELETED // Clear or Deleted entry
//|| pn0 == '.' || longFilename[0] == '.' // Hidden file //|| pn0 == '.' || longFilename[0] == '.' // Hidden file
@ -225,63 +225,63 @@ void CardReader::selectByIndex(SdFile dir, const uint8_t index) {
// //
// Get file/folder info for an item by name // Get file/folder info for an item by name
// //
void CardReader::selectByName(SdFile dir, const char * const match, boolean debug/*=false*/) { void CardReader::selectByName(SdFile dir, const char * const match, const bool debug/*=false*/) {
dir_t p; dir_t p;
for (uint8_t cnt = 0; dir.readDir(&p, longFilename) > 0; cnt++) { for (uint8_t cnt = 0; dir.readDir(&p, longFilename) > 0; cnt++) {
if (is_dir_or_gcode(p)) { if (is_dir_or_gcode(p)) {
createFilename(filename, p); createFilename(filename, p);
#if ENABLED(LONG_FILENAME_HOST_SUPPORT) #if ENABLED(LONG_FILENAME_HOST_SUPPORT)
// Check for match using Long_File_Name or short filename // Check for match using Long_File_Name or short filename
if ( (strcasecmp(match, longFilename) == 0) or (strcasecmp(match, filename) == 0) ) { if (strcasecmp(match, longFilename) == 0 || strcasecmp(match, filename) == 0) {
#if ENABLED(DEBUG_CARDREADER) #if ENABLED(DEBUG_CARDREADER)
//debug = true; //debug = true;
if (debug) { if (debug) {
DEBUG_ECHOLN(" "); DEBUG_ECHOLNPGM(" ");
DEBUG_ECHO(" DEBUG -SelectByName- LONG FileName: "); DEBUG_ECHOPGM(" DEBUG -SelectByName- LONG FileName: ");
DEBUG_ECHO(longFilename); DEBUG_ECHO(longFilename);
DEBUG_ECHO("-- Created FileName: "); DEBUG_ECHOPGM("-- Created FileName: ");
DEBUG_ECHO(filename); DEBUG_ECHO(filename);
DEBUG_ECHO(" -- Search Term: "); DEBUG_ECHOPGM(" -- Search Term: ");
DEBUG_ECHO(match); DEBUG_ECHO(match);
DEBUG_ECHOLN(" -- Match Success: TRUE"); DEBUG_ECHOLNPGM(" -- Match Success: TRUE");
} }
#endif #endif
//This is a workaround if the filename IS the actual longfilename. This avoids reporting incorrect filenames such as "CURVE_~1.GCO" turning into "firmware.bin" //This is a workaround if the filename IS the actual longfilename. This avoids reporting incorrect filenames such as "CURVE_~1.GCO" turning into "firmware.bin"
if ( ! (strncasecmp(filename, longFilename,3) == 0) ) { if (strncasecmp(filename, longFilename, 3) != 0) {
#if ENABLED(DEBUG_CARDREADER) #if ENABLED(DEBUG_CARDREADER)
//debug = true; //debug = true;
if ( debug) DEBUG_ECHOLNPAIR("DEBUG: MisMatch Found! Returning LongFileName as FileName"); if (debug) DEBUG_ECHOLNPAIR("DEBUG: MisMatch Found! Returning LongFileName as FileName");
if ( debug) DEBUG_ECHOLNPAIR("DEBUG: filename = ", filename); if (debug) DEBUG_ECHOLNPAIR("DEBUG: filename = ", filename);
if ( debug) DEBUG_ECHOLNPAIR("DEBUG: longFilename = ", longFilename); if (debug) DEBUG_ECHOLNPAIR("DEBUG: longFilename = ", longFilename);
#endif #endif
strcpy(longFilename, filename); strcpy(longFilename, filename);
} }
return; return;
}
} else { else {
//Match Not Found -> Wipe out the filenames to avoid returning a path to a different file // Match Not Found -> Wipe out the filenames to avoid returning a path to a different file
//if ( debug) DEBUG_ECHOLN("FALSE"); //if (debug) DEBUG_ECHOLN("FALSE");
strcpy(filename, ""); strcpy(filename, "");
strcpy(longFilename, ""); strcpy(longFilename, "");
} }
#else #else
if (strcasecmp(match, filename) == 0) { if (strcasecmp(match, filename) == 0) {
if ( debug) DEBUG_ECHOLN(" DEBUG SelectByName - MatchFound"); if (debug) DEBUG_ECHOLN(" DEBUG SelectByName - MatchFound");
return; return;
} else {
if ( debug) DEBUG_ECHOLN("FALSE");
} }
else if (debug)
DEBUG_ECHOLN("FALSE");
#endif #endif
} }
} }
} }
#if ENABLED(M20_Reports_Directory_Names) #if ENABLED(M20_REPORT_DIRECTORY_NAMES)
// //
// Recursive method to list all directories within a folder // Recursive method to list all directories within a folder
// //
void CardReader::printDirListing(SdFile parent, boolean ReturnDOSFileNames, boolean ReturnLongFileNames, const char * const prepend/*=nullptr*/) { void CardReader::printDirListing(SdFile parent, const bool print_dos_names, const bool print_long_names, const char * const prepend/*=nullptr*/) {
dir_t p; dir_t p;
while (parent.readDir(&p, longFilename) > 0) { while (parent.readDir(&p, longFilename) > 0) {
if (DIR_IS_SUBDIR(&p)) { if (DIR_IS_SUBDIR(&p)) {
@ -292,8 +292,8 @@ void CardReader::selectByName(SdFile dir, const char * const match, boolean debu
// Allocate enough stack space to do the work ( full path to a folder, trailing slash, and nul ) // Allocate enough stack space to do the work ( full path to a folder, trailing slash, and nul )
const bool prepend_is_empty = (!prepend || prepend[0] == '\0'); const bool prepend_is_empty = (!prepend || prepend[0] == '\0');
const int lenPrePend = (prepend_is_empty ? 1 : strlen(prepend)) ; const size_t lenPrePend = prepend_is_empty ? 1 : strlen(prepend),
const int len = lenPrePend + strlen(dosFilename) + 1 + 1; len = lenPrePend + strlen(dosFilename) + 1 + 1;
char path[len]; char path[len];
// Append the FOLDERNAME12/ to the passed string. // Append the FOLDERNAME12/ to the passed string.
@ -304,15 +304,15 @@ void CardReader::selectByName(SdFile dir, const char * const match, boolean debu
strcat(path, "/"); // 1 character strcat(path, "/"); // 1 character
#if DISABLED(LONG_FILENAME_HOST_SUPPORT) #if DISABLED(LONG_FILENAME_HOST_SUPPORT)
SERIAL_ECHOLN( path ); // All this work is already done! SERIAL_ECHOLN(path); // All this work is already done!
#else #else
const int plen = lenPrePend + strlen(longFilename) + 2; const size_t plen = lenPrePend + strlen(longFilename) + 2;
// Generate the name of the file path then display // Generate the name of the file path then display
char Fpath[plen]; char Fpath[plen];
strcpy(Fpath, prepend_is_empty ? "/" : prepend); // root slash if prepend is empty strcpy(Fpath, prepend_is_empty ? "/" : prepend); // root slash if prepend is empty
strcat(Fpath, longFilename) ; strcat(Fpath, longFilename);
strcat(Fpath, "/"); strcat(Fpath, "/");
SERIAL_ECHOLN( Fpath ); // Echo Result SERIAL_ECHOLN(Fpath); // Echo Result
#endif #endif
// Get a new directory object using the full path // Get a new directory object using the full path
@ -322,7 +322,7 @@ void CardReader::selectByName(SdFile dir, const char * const match, boolean debu
SERIAL_ECHO_START(); SERIAL_ECHO_START();
SERIAL_ECHOLNPAIR(STR_SD_CANT_OPEN_SUBDIR, dosFilename); SERIAL_ECHOLNPAIR(STR_SD_CANT_OPEN_SUBDIR, dosFilename);
} }
printDirListing(child, ReturnDOSFileNames, ReturnLongFileNames, path); printDirListing(child, print_dos_names, print_long_names, path);
} }
} }
} }
@ -331,7 +331,7 @@ void CardReader::selectByName(SdFile dir, const char * const match, boolean debu
// //
// Recursive method to list all files within a folder // Recursive method to list all files within a folder
// //
void CardReader::printListing(SdFile parent, boolean ReturnDOSFileNames, boolean ReturnLongFileNames, const char * const prepend/*=nullptr*/) { void CardReader::printListing(SdFile parent, const bool print_dos_names, const bool print_long_names, const char * const prepend/*=nullptr*/) {
dir_t p; dir_t p;
while (parent.readDir(&p, longFilename) > 0) { while (parent.readDir(&p, longFilename) > 0) {
if (DIR_IS_SUBDIR(&p)) { if (DIR_IS_SUBDIR(&p)) {
@ -342,7 +342,7 @@ void CardReader::printListing(SdFile parent, boolean ReturnDOSFileNames, boolean
// Allocate enough stack space for the full path to a folder, trailing slash, and nul // Allocate enough stack space for the full path to a folder, trailing slash, and nul
const bool prepend_is_empty = (!prepend || prepend[0] == '\0'); const bool prepend_is_empty = (!prepend || prepend[0] == '\0');
const int len = (prepend_is_empty ? 1 : strlen(prepend)) + strlen(dosFilename) + 1 + 1; const size_t len = (prepend_is_empty ? 1 : strlen(prepend)) + strlen(dosFilename) + 1 + 1;
char path[len]; char path[len];
// Append the FOLDERNAME12/ to the passed string. // Append the FOLDERNAME12/ to the passed string.
@ -361,48 +361,48 @@ void CardReader::printListing(SdFile parent, boolean ReturnDOSFileNames, boolean
SERIAL_ECHO_START(); SERIAL_ECHO_START();
SERIAL_ECHOLNPAIR(STR_SD_CANT_OPEN_SUBDIR, dosFilename); SERIAL_ECHOLNPAIR(STR_SD_CANT_OPEN_SUBDIR, dosFilename);
} }
printListing(child, ReturnDOSFileNames, ReturnLongFileNames, path); printListing(child, print_dos_names, print_long_names, path);
// close() is done automatically by destructor of SdFile // close() is done automatically by destructor of SdFile
} }
else if (is_dir_or_gcode(p)) { else if (is_dir_or_gcode(p)) {
createFilename(filename, p); //Parse the string and retrieve a DOS 8.3 FileName+ createFilename(filename, p); // Parse the string and retrieve a DOS 8.3 FileName+
#if ENABLED(LONG_FILENAME_HOST_SUPPORT) #if ENABLED(LONG_FILENAME_HOST_SUPPORT)
if (ReturnDOSFileNames) {//Print the DOS FileName & Path if (print_dos_names) { // Print the path element and DOS 8.3 filename
if (prepend) { SERIAL_ECHO(prepend); } if (prepend) SERIAL_ECHO(prepend);
SERIAL_ECHO(filename); SERIAL_ECHO(filename);
} }
if (ReturnLongFileNames) {// Print the Long FileName if (print_long_names) { // Print the Long FileName
if (ReturnDOSFileNames) SERIAL_ECHO(" --> "); if (print_dos_names) SERIAL_ECHOPGM(" --> ");
if (prepend) { if (prepend) {
//Allocate memory to concat everything before passing it into the PrintLongPath Routine // Allocate stack to join strings before passing to printLongPath
const int len = strlen(prepend) + strlen(filename) + 3; const size_t plen = strlen(prepend),
char Fpath[len]; len = plen + strlen(filename) + 1;
// Generate the name of the file path then display // Generate the name of the file path then display
strcpy(Fpath, prepend) ;
strcat(Fpath, filename) ;
printLongPath (Fpath, false);
}else {
//File exists on root
const int len = strlen(filename) + 3;
char Fpath[len]; char Fpath[len];
strcpy(Fpath, "/") ; strcpy(Fpath, prepend);
strcat(Fpath, filename) ; strcpy(&Fpath[plen], filename);
printLongPath(Fpath, false) ; printLongPath(Fpath);
}
else {
// File is in the root directory
const int len = 1 + strlen(filename) + 1; // slash, filename, terninating nul
char Fpath[len];
Fpath[0] = '/';
strcpy(&Fpath[1], filename);
printLongPath(Fpath);
} }
} }
#else #else
//If Long_FileName_Host_Support is disabled -> Print the DOS8.3 filenames // If Long_FileName_Host_Support is disabled -> Print the DOS8.3 filenames
if (prepend) { SERIAL_ECHO(prepend); } if (prepend) SERIAL_ECHO(prepend);
SERIAL_ECHO(filename); SERIAL_ECHO(filename);
#endif #endif
//Print the FileSize // Print the FileSize
SERIAL_CHAR(' '); SERIAL_ECHOLNPAIR(" ", p.fileSize, " bytes");
SERIAL_ECHO( p.fileSize );
SERIAL_ECHOLN(" bytes");
} }
} }
} }
@ -412,7 +412,7 @@ void CardReader::printListing(SdFile parent, boolean ReturnDOSFileNames, boolean
// //
// Get a long pretty path based on a DOS 8.3 path // Get a long pretty path based on a DOS 8.3 path
// //
void CardReader::printLongPath(char * const path, boolean PrintLineReturn) { void CardReader::printLongPath(char * const path, const bool print_eol/*=false*/) {
int i, pathLen = strlen(path); int i, pathLen = strlen(path);
@ -447,7 +447,7 @@ void CardReader::printListing(SdFile parent, boolean ReturnDOSFileNames, boolean
// If the filename was printed then that's it // If the filename was printed then that's it
if (!flag.filenameIsDir) break; if (!flag.filenameIsDir) break;
// SERIAL_ECHOPGM("Opening dir: "); SERIAL_ECHOLN(segment); //SERIAL_ECHOPGM("Opening dir: "); SERIAL_ECHOLN(segment);
// Open the sub-item as the new dive parent // Open the sub-item as the new dive parent
SdFile dir; SdFile dir;
@ -463,31 +463,39 @@ void CardReader::printListing(SdFile parent, boolean ReturnDOSFileNames, boolean
} // while i<pathLen } // while i<pathLen
if (PrintLineReturn) SERIAL_EOL(); if (print_eol) SERIAL_EOL();
} }
#endif // LONG_FILENAME_HOST_SUPPORT #endif // LONG_FILENAME_HOST_SUPPORT
// //
// List all files on the SD card // List all files on the SD card
// //
void CardReader::ls(boolean ReturnDOSFileNames, boolean ReturnLongFileNames) { void CardReader::ls(const bool print_dos_names, const bool print_long_names) {
if (flag.mounted) { if (flag.mounted) {
#if ENABLED(M20_Reports_Directory_Names) #if ENABLED(M20_REPORT_DIRECTORY_NAMES)
SERIAL_ECHOLN("Begin Directory Listing") ; SERIAL_ECHOLNPGM("Begin Directory Listing");
root.rewind(); root.rewind();
card.printDirListing(root, ReturnDOSFileNames,ReturnLongFileNames); card.printDirListing(root, print_dos_names, print_long_names);
SERIAL_ECHOLN("End Directory Listing") ; SERIAL_ECHOLNPGM("End Directory Listing");
SERIAL_ECHOLN(" ") ; SERIAL_ECHOLNPGM(" ");
#endif #endif
SERIAL_ECHOLNPGM(STR_BEGIN_FILE_LIST); SERIAL_ECHOLNPGM(STR_BEGIN_FILE_LIST);
root.rewind(); root.rewind();
printListing(root, ReturnDOSFileNames, ReturnLongFileNames); printListing(root, print_dos_names, print_long_names);
SERIAL_ECHOLNPGM(STR_END_FILE_LIST); SERIAL_ECHOLNPGM(STR_END_FILE_LIST);
} }
} }
//
// List all files on the SD card
//
void CardReader::ls() {
if (flag.mounted) {
root.rewind();
printListing(root);
}
}
// //
// Echo the DOS 8.3 filename (and long filename, if any) // Echo the DOS 8.3 filename (and long filename, if any)
@ -713,7 +721,7 @@ void CardReader::openFileRead(char * const path, const uint8_t subcall_type/*=0*
endFilePrint(); endFilePrint();
SdFile *diveDir; SdFile *diveDir;
const char * const fname = diveToFile(true, diveDir, path, false); const char * const fname = diveToFile(true, diveDir, path);
if (!fname) return; if (!fname) return;
if (file.open(diveDir, fname, O_READ)) { if (file.open(diveDir, fname, O_READ)) {
@ -773,7 +781,7 @@ void CardReader::openFileWrite(char * const path) {
bool CardReader::fileExists(const char * const path) { bool CardReader::fileExists(const char * const path) {
if (!isMounted()) return false; if (!isMounted()) return false;
SdFile *diveDir = nullptr; SdFile *diveDir = nullptr;
const char * const fname = diveToFile(false, diveDir, path, false); const char * const fname = diveToFile(false, diveDir, path);
if (fname) { if (fname) {
diveDir->rewind(); diveDir->rewind();
selectByName(*diveDir, fname); selectByName(*diveDir, fname);
@ -791,7 +799,7 @@ void CardReader::removeFile(const char * const name) {
//endFilePrint(); //endFilePrint();
SdFile *curDir; SdFile *curDir;
const char * const fname = diveToFile(false, curDir, name, false); const char * const fname = diveToFile(false, curDir, name);
if (!fname) return; if (!fname) return;
#if ENABLED(SDCARD_READONLY) #if ENABLED(SDCARD_READONLY)
@ -935,7 +943,7 @@ uint16_t CardReader::countFilesInWorkDir() {
* *
* A nullptr result indicates an unrecoverable error. * A nullptr result indicates an unrecoverable error.
*/ */
const char* CardReader::diveToFile(const bool update_cwd, SdFile*& diveDir, const char * const path, const bool NewFile/*=false*/) { const char* CardReader::diveToFile(const bool update_cwd, SdFile*& diveDir, const char * const path, const bool is_new_file/*=false*/) {
// Track both parent and subfolder // Track both parent and subfolder
static SdFile newDir1, newDir2; static SdFile newDir1, newDir2;
SdFile *sub = &newDir1, *startDir; SdFile *sub = &newDir1, *startDir;
@ -947,8 +955,8 @@ const char* CardReader::diveToFile(const bool update_cwd, SdFile*& diveDir, cons
if (path[0] == '/') { // Starting at the root directory? if (path[0] == '/') { // Starting at the root directory?
DEBUG_ECHOLN("Starting from ROOT"); DEBUG_ECHOLN("Starting from ROOT");
root.rewind();
diveDir = &root; diveDir = &root;
diveDir->rewind(); // Redundant rewind, but no harm
item_name_adr++; item_name_adr++;
DEBUG_ECHOLNPAIR("diveToFile: CWD to root: ", hex_address((void*)diveDir)); DEBUG_ECHOLNPAIR("diveToFile: CWD to root: ", hex_address((void*)diveDir));
if (update_cwd) workDirDepth = 0; // The cwd can be updated for the benefit of sub-programs if (update_cwd) workDirDepth = 0; // The cwd can be updated for the benefit of sub-programs
@ -976,14 +984,17 @@ const char* CardReader::diveToFile(const bool update_cwd, SdFile*& diveDir, cons
DEBUG_ECHOLNPAIR("diveToFile: sub = ", hex_address((void*)sub)); DEBUG_ECHOLNPAIR("diveToFile: sub = ", hex_address((void*)sub));
// Open diveDir (closing first) // Close before dive or select
sub->close(); sub->close();
#if ENABLED(LONG_FILENAME_HOST_SUPPORT) #if ENABLED(LONG_FILENAME_HOST_SUPPORT)
// Select the folder by name and ensure the DOS8.3 name is used // Select the folder by name and ensure the DOS8.3 name is used
DEBUG_ECHOLNPAIR("Search for folder: ", dosSubdirname); DEBUG_ECHOLNPAIR("Search for folder: ", dosSubdirname);
selectByName(*diveDir,dosSubdirname, false) ; selectByName(*diveDir, dosSubdirname);
if (strlen(filename) > 0 ) strcpy(dosSubdirname,filename); if (filename[0]) strcpy(dosSubdirname, filename);
#endif #endif
// Open diveDir (closing first, above)
if (!sub->open(diveDir, dosSubdirname, O_READ)) { if (!sub->open(diveDir, dosSubdirname, O_READ)) {
openFailed(dosSubdirname); openFailed(dosSubdirname);
item_name_adr = nullptr; item_name_adr = nullptr;
@ -1022,23 +1033,19 @@ const char* CardReader::diveToFile(const bool update_cwd, SdFile*& diveDir, cons
TERN_(SDCARD_SORT_ALPHA, presort()); TERN_(SDCARD_SORT_ALPHA, presort());
} }
#if ENABLED(LONG_FILENAME_HOST_SUPPORT) #if ENABLED(LONG_FILENAME_HOST_SUPPORT)
// Select the file by name and ensure the DOS8.3 name is returned if (!is_new_file) {
if (NewFile) {
return item_name_adr; // Expecting the start location of a new file. Nothing else to do here.
}else {
// Expecting a file that already exists - Search and select // Expecting a file that already exists - Search and select
diveDir->rewind(); diveDir->rewind();
DEBUG_ECHOLNPAIR("Search for file: ", item_name_adr); DEBUG_ECHOLNPAIR("Search for file: ", item_name_adr);
selectByName(*diveDir, item_name_adr, false) ; selectByName(*diveDir, item_name_adr);
DEBUG_ECHOLNPAIR("diveToFile: Return String = ", filename); DEBUG_ECHOLNPAIR("diveToFile: Return String = ", filename);
return filename; return filename;
} }
#else
DEBUG_ECHOLNPAIR("diveToFile: Return String = ", item_name_adr);
return item_name_adr;
#endif #endif
DEBUG_ECHOLNPAIR("diveToFile: Return String = ", item_name_adr);
return item_name_adr;
} }
void CardReader::cd(const char * relpath) { void CardReader::cd(const char * relpath) {

13
Marlin/src/sd/cardreader.h
View file

@ -81,7 +81,7 @@ public:
static void mount(); static void mount();
static void release(); static void release();
static inline bool isMounted() { return flag.mounted; } static inline bool isMounted() { return flag.mounted; }
static void ls(boolean ReturnDOSFileNames,boolean ReturnLongFileNames); static void ls(const bool print_dos_names, const bool print_long_names);
// Handle media insert/remove // Handle media insert/remove
static void manage_media(); static void manage_media();
@ -103,8 +103,9 @@ public:
static void removeFile(const char * const name); static void removeFile(const char * const name);
static inline char* longest_filename() { return longFilename[0] ? longFilename : filename; } static inline char* longest_filename() { return longFilename[0] ? longFilename : filename; }
#if ENABLED(LONG_FILENAME_HOST_SUPPORT) #if ENABLED(LONG_FILENAME_HOST_SUPPORT)
static void printLongPath(char * const path, boolean PrintLineReturn); // Used by M33 static void printLongPath(char * const path, const bool print_eol=false); // Used by M33
#endif #endif
// Working Directory for SD card menu // Working Directory for SD card menu
@ -135,7 +136,7 @@ public:
static inline uint8_t percentDone() { return (isFileOpen() && filesize) ? sdpos / ((filesize + 99) / 100) : 0; } static inline uint8_t percentDone() { return (isFileOpen() && filesize) ? sdpos / ((filesize + 99) / 100) : 0; }
// Helper for open and remove // Helper for open and remove
static const char* diveToFile(const bool update_cwd, SdFile*& curDir, const char * const path, const bool NewFile=false); static const char* diveToFile(const bool update_cwd, SdFile*& curDir, const char * const path, const bool is_new_file=false);
#if ENABLED(SDCARD_SORT_ALPHA) #if ENABLED(SDCARD_SORT_ALPHA)
static void presort(); static void presort();
@ -273,9 +274,9 @@ private:
static bool is_dir_or_gcode(const dir_t &p); static bool is_dir_or_gcode(const dir_t &p);
static int countItems(SdFile dir); static int countItems(SdFile dir);
static void selectByIndex(SdFile dir, const uint8_t index); static void selectByIndex(SdFile dir, const uint8_t index);
static void selectByName(SdFile dir, const char * const match, boolean debug=false); static void selectByName(SdFile dir, const char * const match, const bool debug=false);
static void printListing(SdFile parent, boolean ReturnDOSFileNames, boolean ReturnLongFileNames, const char * const prepend=nullptr); static void printListing(SdFile parent, const bool print_dos_names, const bool print_long_names, const char * const prepend=nullptr);
static void printDirListing(SdFile parent, boolean ReturnDOSFileNames, boolean ReturnLongFileNames, const char * const prepend=nullptr); static void printDirListing(SdFile parent, const bool print_dos_names, const bool print_long_names, const char * const prepend=nullptr);
#if ENABLED(SDCARD_SORT_ALPHA) #if ENABLED(SDCARD_SORT_ALPHA)
static void flush_presort(); static void flush_presort();