cosmetic and consistency fixes

Author: cristidragomir97

Diff for: examples/InternalStoragePartitioning/InternalStoragePartitioning.ino

@@ -40,7 +40,7 @@ std::vector<Partition> partitioningScheme  = {
         {1024, FS_FAT}, // 1 MB for certificates
         {5120, FS_FAT}, // 5 MB for OTA firmware updates
         {8192, FS_LITTLEFS} // 8 MB for user data
-    };
+};
 
 // Function to test writing to a file in the specified storage partition
 void testWriting(Arduino_UnifiedStorage *storage) {
@@ -81,7 +81,6 @@ void testAllPartitions(std::vector<Partition> partitions) {
 }
 
 
-
 void listPartitions(){
     std::vector<Partition> partitions = InternalStorage::readPartitions();
     int partitionIndex = 0;

Diff for: examples/SimpleStorageWriteRead/SimpleStorageWriteRead.ino

@@ -30,12 +30,6 @@
 #include "Arduino_UnifiedStorage.h"
 
 
-
-// Redirect Serial.print*() output to GDB instead of SerialUSB where it would conflict with ThreadDebug.
-// NOTE: Calls to Serial.print*() will block waiting for GDB to be connected so only useful to use this redefine
-//       when actively debugging the program.
-
-
 void printFolderContents(Folder dir, int indentation = 0) {
   std::vector<Folder> directories = dir.getFolders();
   std::vector<UFile> files = dir.getFiles();
@@ -80,8 +74,6 @@ void setup() {
   if(!internalStorage.begin()){
     Serial.println("Error mounting storage device.");
   }
-
-
 
   // Create a root directory in storage device
   Folder root = internalStorage.getRootFolder();

Diff for: extras/tests/TestExisting/TestExisting.ino

@@ -10,10 +10,10 @@ void setup() {
     /* UNCOMMENT THIS PART IF YOU WANT TO ENABLE FORMATTING*/
 
 
-    #if defined(ARDUINO_PORTENTA_C33) || defined(ARDUINO_PORTENTA_H7_M7)
+    #if defined(HAS_SERIAL)
         Serial.begin(115200);
         while(!Serial);
-    #elif defined(ARDUINO_OPTA)
+    #elif defined(HAS_RS485)
         beginRS485(115200);
     #endif 
 

Diff for: extras/tests/TestFileOperations/TestFileOperations.ino

@@ -303,11 +303,10 @@ void runTests(Arduino_UnifiedStorage * storage, String storageType) {
 }
 
 void setup(){
- 
-    #if defined(ARDUINO_PORTENTA_C33) || defined(ARDUINO_PORTENTA_H7_M7)
+    #if defined(HAS_SERIAL)
         Serial.begin(115200);
         while(!Serial);
-    #elif defined(ARDUINO_OPTA)
+    #elif defined(HAS_RS485)
         beginRS485(115200);
     #endif 
 

Diff for: extras/tests/TestFolderOperations/TestFolderOperations.ino

@@ -120,13 +120,14 @@ void runTests(Arduino_UnifiedStorage * storage, String storageType) {
 }
 
 void setup(){
-    #if defined(ARDUINO_PORTENTA_C33) || defined(ARDUINO_PORTENTA_H7_M7)
+    #if defined(HAS_SERIAL)
         Serial.begin(115200);
         while(!Serial);
-    #elif defined(ARDUINO_OPTA)
+    #elif defined(HAS_RS485)
         beginRS485(115200);
     #endif 
 
+
     // toggle this to enable debugging output
     Arduino_UnifiedStorage::debuggingModeEnabled = false;
 

Diff for: extras/tests/TestRepeatedFormatMount/TestRepeatedFormatMount.ino

@@ -45,12 +45,13 @@ void runRepeatedMountTest(Arduino_UnifiedStorage * storage, String storageType,
 
 
 void setup(){
-    #if defined(ARDUINO_PORTENTA_C33) || defined(ARDUINO_PORTENTA_H7_M7)
+    #if defined(HAS_SERIAL)
         Serial.begin(115200);
         while(!Serial);
-    #elif defined(ARDUINO_OPTA)
+    #elif defined(HAS_RS485)
         beginRS485(115200);
     #endif 
+
 
     // toggle this to enable debugging output
     Arduino_UnifiedStorage::debuggingModeEnabled = false;

Diff for: library.properties

@@ -2,7 +2,7 @@ name=Arduino_UnifiedStorage
 version=1.0.0
 author=Arduino
 maintainer=Arduino <[email protected]>
-sentence=Simplify cross-device storage management on Portenta platforms with a single library supporting &sd, Flash, and USB storage access.
+sentence=Simplify cross-device storage management on Portenta platforms with a single library supporting SD, Flash, and USB storage access.
 paragraph=With this versatile library, you can seamlessly handle various storage types without the need for separate, device-specific libraries. Develop data logging, file storage, or data transfer applications, with a consistent interface. All while reducing development time and ensuring compatibility across the Portenta C33, Portenta H7, and Portenta Machine Control platforms.
 category=Data Storage
 url=https://github.com/arduino-libraries/Arduino_UnifiedStorage

Diff for: src/Utils.h

@@ -45,6 +45,7 @@
 
 
 [[gnu::unused]] static const char* createPartitionName(int number) {
+    // check if the number can be represented by a letter of the alphabet
     if (number < 1 || number > 26) {
         // Handle out-of-range numbers or errors as needed
         return nullptr;