Fix several gcc warnings occurring with -Wextra

Warnings fixed are:

- warning: type qualifiers ignored on function return type [-Wignored-qualifiers]
- warning: unused parameter ... [-Wunused-parameter]
- warning: comparison is always true due to limited range of data type [-Wtype-limits]

git-svn-id: https://matroska.svn.sourceforge.net/svnroot/matroska/trunk/libmatroska@823 a6f86f6d-0131-4f8e-9e7b-e335508773d5

diff --git a/make/linux/Makefile b/make/linux/Makefile
index c678e2869cf850288c7e1b619934869f8bafa370..fd4bf012841eaab54acd56d2e0c4393fb76418e3 100644 (file)
--- a/make/linux/Makefile
+++ b/make/linux/Makefile
@@ -67,7 +67,7 @@ objects:=$(patsubst %$(EXTENSION),%.o,$(sources))
 \r
 objects_so:=$(patsubst %$(EXTENSION),%.lo,$(sources))\r
 \r
-WARNINGFLAGS=-Wall -Wno-unknown-pragmas -ansi -fno-gnu-keywords -D_GNU_SOURCE \\r
+WARNINGFLAGS=-Wall -Wextra -Wno-unknown-pragmas -ansi -fno-gnu-keywords -D_GNU_SOURCE \\r
                -Wshadow\r
 COMPILEFLAGS=$(DEBUGFLAGS) $(CXXFLAGS) $(CPPFLAGS) $(WARNINGFLAGS) $(INCLUDE)\r
 LINKFLAGS=-L. -L$(LIBEBML_LIB_DIR) $(LDFLAGS)\r

diff --git a/matroska/KaxBlock.h b/matroska/KaxBlock.h
index 1fb3c9e9b3cf096809fb7cb0983617b28ea0ba43..4931cbd96fa5ca7624fc5d1f9da8f06945fcd970 100644 (file)
--- a/matroska/KaxBlock.h
+++ b/matroska/KaxBlock.h
@@ -84,7 +84,7 @@ class MATROSKA_DLL_API DataBuffer {
                virtual binary * Buffer() {assert(bValidValue); return myBuffer;}
                virtual uint32   & Size() {return mySize;};
                virtual const binary * Buffer() const {assert(bValidValue); return myBuffer;}
-               virtual const uint32   Size()   const {return mySize;};
+               virtual uint32   Size()   const {return mySize;};
                bool    FreeBuffer(const DataBuffer & aBuffer) {
                        bool bResult = true;
                        if (myBuffer != NULL && bValidValue) {

diff --git a/src/KaxBlock.cpp b/src/KaxBlock.cpp
index e96910e7febab2c0478b1a2df185257016f510cb..b5501e507fc77f015d36803ab22b44f2529b09ee 100644 (file)
--- a/src/KaxBlock.cpp
+++ b/src/KaxBlock.cpp
@@ -161,7 +161,7 @@ LacingType KaxInternalBlock::GetBestLacingType() const {
                return LACING_EBML;
 }
 
-filepos_t KaxInternalBlock::UpdateSize(bool bSaveDefault, bool bForceRender)
+filepos_t KaxInternalBlock::UpdateSize(bool /* bSaveDefault */, bool /* bForceRender */)
 {
        LacingType LacingHere;
     assert(EbmlBinary::GetBuffer() == NULL); // Data is not used for KaxInternalBlock
@@ -238,7 +238,7 @@ KaxBlockVirtual::~KaxBlockVirtual()
         SetBuffer( NULL, 0 ); 
 }
 
-filepos_t KaxBlockVirtual::UpdateSize(bool bSaveDefault, bool bForceRender)
+filepos_t KaxBlockVirtual::UpdateSize(bool /* bSaveDefault */, bool /* bForceRender */)
 {
        assert(TrackNumber < 0x4000);
        binary *cursor = EbmlBinary::GetBuffer();
@@ -268,7 +268,7 @@ filepos_t KaxBlockVirtual::UpdateSize(bool bSaveDefault, bool bForceRender)
        \todo more optimisation is possible (render the Block head and don't copy the buffer in memory, care should be taken with the allocation of Data)
        \todo the actual timecode to write should be retrieved from the Cluster from here
 */
-filepos_t KaxInternalBlock::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault)
+filepos_t KaxInternalBlock::RenderData(IOCallback & output, bool /* bForceRender */, bool /* bSaveDefault */)
 {
        if (myBuffers.size() == 0) {
                return 0;
@@ -1034,7 +1034,7 @@ void KaxBlockBlob::SetBlockGroup( KaxBlockGroup &BlockRef )
        Block.group = &BlockRef;
 }
 
-filepos_t KaxBlockVirtual::ReadData(IOCallback & input, ScopeMode ReadFully)
+filepos_t KaxBlockVirtual::ReadData(IOCallback & input, ScopeMode /* ReadFully */)
 {
     input.setFilePointer(SizePosition + CodedSizeLength(Size, SizeLength, bSizeIsFinite) + Size, seek_beginning);
     return GetSize();

diff --git a/src/KaxSemantic.cpp b/src/KaxSemantic.cpp
index a5b555f55df5472f6abc56138ebf92e875cb33a2..2ebb4ce53841522b1c0456644f90f44917b58fb8 100644 (file)
--- a/src/KaxSemantic.cpp
+++ b/src/KaxSemantic.cpp
@@ -666,97 +666,97 @@ DEFINE_MKX_UINTEGER_DEF(KaxTagDefault, 0x4484, 2, KaxTagSimple, "TagDefault", 1)
 DEFINE_MKX_UNISTRING(KaxTagString, 0x4487, 2, KaxTagSimple, "TagString");\r
 DEFINE_MKX_BINARY (KaxTagBinary, 0x4485, 2, KaxTagSimple, "TagBinary");\r
 \r
-filepos_t KaxBlockVirtual::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxBlockVirtual::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxReferenceVirtual::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxReferenceVirtual::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxSliceFrameNumber::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxSliceFrameNumber::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxSliceBlockAddID::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxSliceBlockAddID::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxSliceDelay::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxSliceDelay::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxSliceDuration::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxSliceDuration::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxEncryptedBlock::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxEncryptedBlock::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxTrackOffset::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxTrackOffset::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxCodecSettings::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxCodecSettings::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxCodecInfoURL::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxCodecInfoURL::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxCodecDownloadURL::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxCodecDownloadURL::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxOldStereoMode::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxOldStereoMode::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxVideoGamma::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxVideoGamma::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxVideoFrameRate::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxVideoFrameRate::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxAudioPosition::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxAudioPosition::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxCueRefCluster::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxCueRefCluster::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxCueRefNumber::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxCueRefNumber::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxCueRefCodecState::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxCueRefCodecState::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r
 \r
-filepos_t KaxFileReferral::RenderData(IOCallback & output, bool bForceRender, bool bSaveDefault) {\r
+filepos_t KaxFileReferral::RenderData(IOCallback & /* output */, bool /* bForceRender */, bool /* bSaveDefault */) {\r
        assert(false); // no you are not allowed to use this element !\r
        return 0;\r
 }\r