cleanup warnings

Author: facchinm

cores/arduino/main.cpp

@@ -133,7 +133,10 @@ extern "C" {
 }
 
 //Declared weak in Arduino.h to allow user redefinitions.
-int atexit(void (*func)()) { return 0; }
+int atexit(void (*func)()) {
+   (void)func;
+   return 0;
+}
 
 namespace __gnu_cxx {
    void __verbose_terminate_handler() { }

libraries/BlockDevices/BlockDevice.h

@@ -41,10 +41,10 @@ typedef pin_size_t pin_t;
 
 class BlockDevice {
 private:
-   virtual int open() {}
-   virtual int close() {}
+   virtual int open() { return 0; }
+   virtual int close() { return 0; }
    virtual int write(const void *buffer, bd_addr_t addr, bd_size_t size) {
-      program(buffer, addr, size);
+      return program(buffer, addr, size);
    }
 
 
@@ -67,13 +67,20 @@ class BlockDevice {
    /* Write blocks */
    virtual int program(const void *buffer, bd_addr_t addr, bd_size_t size) = 0;
    virtual int erase(bd_addr_t addr, bd_size_t size) = 0;
-   virtual int trim(bd_addr_t addr, bd_size_t size) { return 0; }
+   virtual int trim(bd_addr_t addr, bd_size_t size) {
+      (void)addr;
+      (void)size;
+      return 0;
+   }
 
 
    virtual bd_size_t get_read_size() const = 0;
    virtual bd_size_t get_program_size() const = 0;
    virtual bd_size_t get_erase_size() const { return get_program_size();}
-   virtual bd_size_t get_erase_size(bd_addr_t addr) const { return get_erase_size(); }
+   virtual bd_size_t get_erase_size(bd_addr_t addr) const {
+      (void)addr;
+      return get_erase_size();
+   }
 
 
    virtual int get_erase_value() const { return -1; }

libraries/Storage/FileHandle.cpp

@@ -14,6 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "FileHandle.h"
 
 //namespace mbed {

libraries/Storage/FileHandle.h

@@ -148,6 +148,7 @@ class FileHandle : private NonCopyable<FileHandle> {
      */
     virtual int truncate(off_t length)
     {
+        (void)length;
         return -EINVAL;
     }
 
@@ -189,6 +190,7 @@ class FileHandle : private NonCopyable<FileHandle> {
      */
     virtual int enable_input(bool enabled)
     {
+        (void)enabled;
         return -EINVAL;
     }
 
@@ -207,6 +209,7 @@ class FileHandle : private NonCopyable<FileHandle> {
      */
     virtual int enable_output(bool enabled)
     {
+        (void)enabled;
         return -EINVAL;
     }
 
@@ -223,6 +226,7 @@ class FileHandle : private NonCopyable<FileHandle> {
     virtual short poll(short events) const
     {
         // Possible default for real files
+        (void)events;
         return POLLIN | POLLOUT;
     }
 
@@ -268,6 +272,7 @@ class FileHandle : private NonCopyable<FileHandle> {
     virtual void sigio(void (*func)())
     {
         //Default for real files. Do nothing for real files.
+        (void)func;
     }
 };
 

libraries/Storage/FileSystem.cpp

@@ -20,6 +20,8 @@
 #include "FileSystem.h"
 #include <errno.h>
 
+#pragma GCC diagnostic ignored "-Wunused-parameter"
+
 //namespace mbed {
 
 FileSystem::FileSystem(const char *name)

libraries/Storage/FileSystemHandle.cpp

@@ -17,6 +17,8 @@
 
 #include "FileSystemHandle.h"
 
+#pragma GCC diagnostic ignored "-Wunused-parameter"
+
 //namespace mbed {
 
 int FileSystemHandle::open(DirHandle **dir, const char *path)

libraries/Storage/storage_common.h

@@ -77,15 +77,17 @@ static inline void debug_mem(uint8_t *b, uint32_t _size)
 #else
 
 static inline void debug_if(int condition, const char *format, ...) {
-
+    (void)condition;
+    (void)format;
 }
 
 static inline void debug(const char *format, ...) {
-    
+    (void)format;
 }
 
 static inline void debug_mem(uint8_t *b, uint32_t _size) {
-
+    (void)b;
+    (void)_size;
 }
 
 #endif // STORAGE_DEBUG

libraries/Storage/storage_retarget.cpp

@@ -28,6 +28,8 @@
 #include <errno.h>
 #include "storage_retarget.h"
 
+#pragma GCC diagnostic ignored "-Wunused-parameter"
+
 static PlatformMutex _mutex;
 
 /* DIR is typedeffed to struct DIR_impl in header */