Browse Source

🐛 Use delete [] for new []

Scott Lahteine 4 years ago
parent
commit
13e4e24e21
1 changed files with 4 additions and 4 deletions
  1. 4
    4
      Marlin/src/sd/cardreader.cpp

+ 4
- 4
Marlin/src/sd/cardreader.cpp View File

1219
         #if ENABLED(SDSORT_USES_RAM) && DISABLED(SDSORT_CACHE_NAMES)
1219
         #if ENABLED(SDSORT_USES_RAM) && DISABLED(SDSORT_CACHE_NAMES)
1220
           #if ENABLED(SDSORT_DYNAMIC_RAM)
1220
           #if ENABLED(SDSORT_DYNAMIC_RAM)
1221
             for (uint16_t i = 0; i < fileCnt; ++i) free(sortnames[i]);
1221
             for (uint16_t i = 0; i < fileCnt; ++i) free(sortnames[i]);
1222
-            TERN_(HAS_FOLDER_SORTING, free(isDir));
1222
+            TERN_(HAS_FOLDER_SORTING, delete [] isDir);
1223
           #endif
1223
           #endif
1224
         #endif
1224
         #endif
1225
       }
1225
       }
1245
   void CardReader::flush_presort() {
1245
   void CardReader::flush_presort() {
1246
     if (sort_count > 0) {
1246
     if (sort_count > 0) {
1247
       #if ENABLED(SDSORT_DYNAMIC_RAM)
1247
       #if ENABLED(SDSORT_DYNAMIC_RAM)
1248
-        delete sort_order;
1248
+        delete [] sort_order;
1249
         #if ENABLED(SDSORT_CACHE_NAMES)
1249
         #if ENABLED(SDSORT_CACHE_NAMES)
1250
           LOOP_L_N(i, sort_count) {
1250
           LOOP_L_N(i, sort_count) {
1251
             free(sortshort[i]); // strdup
1251
             free(sortshort[i]); // strdup
1252
             free(sortnames[i]); // strdup
1252
             free(sortnames[i]); // strdup
1253
           }
1253
           }
1254
-          delete sortshort;
1255
-          delete sortnames;
1254
+          delete [] sortshort;
1255
+          delete [] sortnames;
1256
         #endif
1256
         #endif
1257
       #endif
1257
       #endif
1258
       sort_count = 0;
1258
       sort_count = 0;

Loading…
Cancel
Save