[libgens] EmuContext: Mark overridden functions correctly.
authorDavid Korth <gerbilsoft@gerbilsoft.com>
Sat, 5 Sep 2015 03:51:39 +0000 (23:51 -0400)
committerDavid Korth <gerbilsoft@gerbilsoft.com>
Sat, 5 Sep 2015 03:51:39 +0000 (23:51 -0400)
Changed all overridden functions in EmuMD and EmuPico to final.
I'm pretty sure EmuMCD and Emu32X won't be deriving from EmuMD
directly, especially since EmuPico isn't.

This was found by running a build with clang-3.7.0, which printed
warnings similar to the following:

[ 33%] Building CXX object src/libgens/CMakeFiles/gens.dir/cpu/M68K_Mem.cpp.o
In file included from src/libgens/cpu/M68K_Mem.cpp:42:
src/libgens/EmuContext/EmuMD.hpp:51:7: warning: 'softReset' overrides a member function
      but is not marked 'override' [-Winconsistent-missing-override]
                int softReset(void);
                    ^
src/libgens/EmuContext/EmuContext.hpp:82:15: note: overridden virtual function is here
                virtual int softReset(void) = 0;
                            ^
In file included from /home/d

src/libgens/EmuContext/EmuMD.hpp
src/libgens/EmuContext/EmuPico.hpp

index 34e981c..0570c01 100644 (file)
@@ -48,37 +48,37 @@ class EmuMD : public EmuContext
                 * Perform a soft reset.
                 * @return 0 on success; non-zero on error.
                 */
-               int softReset(void);
+               virtual int softReset(void) final;
 
                /**
                 * Perform a hard reset.
                 * @return 0 on success; non-zero on error.
                 */
-               int hardReset(void);
+               virtual int hardReset(void) final;
 
                /**
                 * Set the region code.
                 * @param region Region code.
                 * @return 0 on success; non-zero on error.
                 */
-               int setRegion(SysVersion::RegionCode_t region);
+               virtual int setRegion(SysVersion::RegionCode_t region) final;
 
                /**
                 * Save SRam/EEPRom.
                 * @return 1 if SRam was saved; 2 if EEPRom was saved; 0 if nothing was saved. (TODO: Enum?)
                 */
-               int saveData(void) final;
+               virtual int saveData(void) final;
 
                /**
                 * AutoSave SRam/EEPRom.
                 * @param frames Number of frames elapsed, or -1 for paused. (force autosave)
                 * @return 1 if SRam was saved; 2 if EEPRom was saved; 0 if nothing was saved. (TODO: Enum?)
                 */
-               int autoSaveData(int framesElapsed) final;
+               virtual int autoSaveData(int framesElapsed) final;
 
                /** Frame execution functions. **/
-               void execFrame(void);
-               void execFrameFast(void);
+               virtual void execFrame(void) final;
+               virtual void execFrameFast(void) final;
 
                /**
                 * Load the current state from a ZOMG file.
@@ -86,7 +86,7 @@ class EmuMD : public EmuContext
                 * @return 0 on success; non-zero on error.
                 * TODO: Error code constants.
                 */
-               virtual int zomgLoad(const char *filename) override;
+               virtual int zomgLoad(const char *filename) final;
 
                /**
                 * Save the current state to a ZOMG file.
@@ -94,7 +94,7 @@ class EmuMD : public EmuContext
                 * @return 0 on success; non-zero on error.
                 * TODO: Error code constants.
                 */
-               virtual int zomgSave(const char *filename) const override;
+               virtual int zomgSave(const char *filename) const final;
 
        protected:
                /**
index 0b50d21..a1bb356 100644 (file)
@@ -48,37 +48,37 @@ class EmuPico : public EmuContext
                 * Perform a soft reset.
                 * @return 0 on success; non-zero on error.
                 */
-               int softReset(void);
+               virtual int softReset(void) final;
 
                /**
                 * Perform a hard reset.
                 * @return 0 on success; non-zero on error.
                 */
-               int hardReset(void);
+               virtual int hardReset(void) final;
 
                /**
                 * Set the region code.
                 * @param region Region code.
                 * @return 0 on success; non-zero on error.
                 */
-               int setRegion(SysVersion::RegionCode_t region);
+               virtual int setRegion(SysVersion::RegionCode_t region) final;
 
                /**
                 * Save SRam/EEPRom.
                 * @return 1 if SRam was saved; 2 if EEPRom was saved; 0 if nothing was saved. (TODO: Enum?)
                 */
-               int saveData(void) final;
+               virtual int saveData(void) final;
 
                /**
                 * AutoSave SRam/EEPRom.
                 * @param frames Number of frames elapsed, or -1 for paused. (force autosave)
                 * @return 1 if SRam was saved; 2 if EEPRom was saved; 0 if nothing was saved. (TODO: Enum?)
                 */
-               int autoSaveData(int framesElapsed) final;
+               virtual int autoSaveData(int framesElapsed) final;
 
                /** Frame execution functions. **/
-               void execFrame(void);
-               void execFrameFast(void);
+               virtual void execFrame(void) final;
+               virtual void execFrameFast(void) final;
 
                /**
                 * Load the current state from a ZOMG file.
@@ -86,7 +86,7 @@ class EmuPico : public EmuContext
                 * @return 0 on success; non-zero on error.
                 * TODO: Error code constants.
                 */
-               virtual int zomgLoad(const char *filename) override;
+               virtual int zomgLoad(const char *filename) final;
 
                /**
                 * Save the current state to a ZOMG file.
@@ -94,7 +94,7 @@ class EmuPico : public EmuContext
                 * @return 0 on success; non-zero on error.
                 * TODO: Error code constants.
                 */
-               virtual int zomgSave(const char *filename) const override;
+               virtual int zomgSave(const char *filename) const final;
 
        protected:
                /**