Revert "Remove more useless inline specifiers."

This reverts commit dadfe7979953dda7ae04ca1c3638495943dedf6d.

diff --git a/taglib/asf/asfutils.h b/taglib/asf/asfutils.h
index 4bca5fa61c3138f13182cce36df78c4219f85521..a8ecd70dc7f888da925dfb68702a11d30065f25a 100644 (file)
--- a/taglib/asf/asfutils.h
+++ b/taglib/asf/asfutils.h
@@ -37,7 +37,7 @@ namespace TagLib
     namespace
     {
 
-      unsigned short readWORD(File *file, bool *ok = 0)
+      inline unsigned short readWORD(File *file, bool *ok = 0)
       {
         const ByteVector v = file->readBlock(2);
         if(v.size() != 2) {
@@ -48,7 +48,7 @@ namespace TagLib
         return v.toUShort(false);
       }
 
-      unsigned int readDWORD(File *file, bool *ok = 0)
+      inline unsigned int readDWORD(File *file, bool *ok = 0)
       {
         const ByteVector v = file->readBlock(4);
         if(v.size() != 4) {
@@ -59,7 +59,7 @@ namespace TagLib
         return v.toUInt(false);
       }
 
-      long long readQWORD(File *file, bool *ok = 0)
+      inline long long readQWORD(File *file, bool *ok = 0)
       {
         const ByteVector v = file->readBlock(8);
         if(v.size() != 8) {
@@ -70,7 +70,7 @@ namespace TagLib
         return v.toLongLong(false);
       }
 
-      String readString(File *file, int length)
+      inline String readString(File *file, int length)
       {
         ByteVector data = file->readBlock(length);
         unsigned int size = data.size();
@@ -86,7 +86,7 @@ namespace TagLib
         return String(data, String::UTF16LE);
       }
 
-      ByteVector renderString(const String &str, bool includeLength = false)
+      inline ByteVector renderString(const String &str, bool includeLength = false)
       {
         ByteVector data = str.data(String::UTF16LE) + ByteVector::fromShort(0, false);
         if(includeLength) {

diff --git a/taglib/mpeg/mpegutils.h b/taglib/mpeg/mpegutils.h
index cc484f2593ec3c3fd26eb21bd18d40f60965b671..e35f752fb28be14f95bbab8f72586f452a6a9203 100644 (file)
--- a/taglib/mpeg/mpegutils.h
+++ b/taglib/mpeg/mpegutils.h
@@ -42,12 +42,12 @@ namespace TagLib
        * first byte is easy to check for, however checking to see if the second byte
        * starts with \e 111 is a bit more tricky, hence these functions.
        */
-      bool firstSyncByte(unsigned char byte)
+      inline bool firstSyncByte(unsigned char byte)
       {
         return (byte == 0xFF);
       }
 
-      bool secondSynchByte(unsigned char byte)
+      inline bool secondSynchByte(unsigned char byte)
       {
         // 0xFF is possible in theory, but it's very unlikely be a header.
 

diff --git a/taglib/riff/riffutils.h b/taglib/riff/riffutils.h
index 4078385bfb4a87fd5bfa9e547b82563d398adfd6..ecb985a4b5b7c576337bf3cde8cad612e345b779 100644 (file)
--- a/taglib/riff/riffutils.h
+++ b/taglib/riff/riffutils.h
@@ -37,7 +37,7 @@ namespace TagLib
     namespace
     {
 
-      bool isValidChunkName(const ByteVector &name)
+      inline bool isValidChunkName(const ByteVector &name)
       {
         if(name.size() != 4)
           return false;

diff --git a/taglib/toolkit/tutils.h b/taglib/toolkit/tutils.h
index 0a3e14dbfa2241455674bd991ad99a33f077840e..e3e4f6c36e94b68fbf9f670cc3cf102d0e44d410 100644 (file)
--- a/taglib/toolkit/tutils.h
+++ b/taglib/toolkit/tutils.h
@@ -61,7 +61,7 @@ namespace TagLib
       /*!
        * Reverses the order of bytes in an 16-bit integer.
        */
-      unsigned short byteSwap(unsigned short x)
+      inline unsigned short byteSwap(unsigned short x)
       {
 #if defined(HAVE_BOOST_BYTESWAP)
 
@@ -97,7 +97,7 @@ namespace TagLib
       /*!
        * Reverses the order of bytes in an 32-bit integer.
        */
-      unsigned int byteSwap(unsigned int x)
+      inline unsigned int byteSwap(unsigned int x)
       {
 #if defined(HAVE_BOOST_BYTESWAP)
 
@@ -136,7 +136,7 @@ namespace TagLib
       /*!
        * Reverses the order of bytes in an 64-bit integer.
        */
-      unsigned long long byteSwap(unsigned long long x)
+      inline unsigned long long byteSwap(unsigned long long x)
       {
 #if defined(HAVE_BOOST_BYTESWAP)
 
@@ -180,7 +180,7 @@ namespace TagLib
        * Returns a formatted string just like standard sprintf(), but makes use of
        * safer functions such as snprintf() if available.
        */
-      String formatString(const char *format, ...)
+      inline String formatString(const char *format, ...)
       {
         // Sufficient buffer size for the current internal uses.
         // Consider changing this value when you use this function.
@@ -228,7 +228,7 @@ namespace TagLib
        * We took the trouble to define this one here, since there are some
        * incompatible variations of case insensitive strcmp().
        */
-      bool equalsIgnoreCase(const char *s1, const char *s2)
+      inline bool equalsIgnoreCase(const char *s1, const char *s2)
       {
         while(*s1 != '\0' && *s2 != '\0' && ::tolower(*s1) == ::tolower(*s2)) {
           s1++;
@@ -252,7 +252,7 @@ namespace TagLib
       /*!
        * Returns the integer byte order of the system.
        */
-      ByteOrder systemByteOrder()
+      inline ByteOrder systemByteOrder()
       {
         union {
           int  i;
@@ -269,7 +269,7 @@ namespace TagLib
       /*!
        * Returns the IEEE754 byte order of the system.
        */
-      ByteOrder floatByteOrder()
+      inline ByteOrder floatByteOrder()
       {
         union {
           double d;