Remove ByteVector::null and isNull().

taglib/toolkit/tbytevector.cpp

@@ -327,8 +327,6 @@ public:
 // static members
 ////////////////////////////////////////////////////////////////////////////////
 
-const ByteVector ByteVector::null;
-
 const size_t ByteVector::npos = static_cast<size_t>(-1);
 
 ByteVector ByteVector::fromCString(const char *s, size_t length)
@@ -679,11 +677,6 @@ ByteVector::ConstReverseIterator ByteVector::rend() const
   return v.rbegin() + (v.size() - d->offset);
 }
 
-bool ByteVector::isNull() const
-{
-  return (d == null.d);
-}
-
 bool ByteVector::isEmpty() const
 {
   return (d->length == 0);

taglib/toolkit/tbytevector.h

@@ -263,19 +263,6 @@ namespace TagLib {
      */
     ConstReverseIterator rend() const;
 
-    /*!
-     * Returns true if the vector is null.
-     *
-     * \note A vector may be empty without being null.  So do not use this
-     * method to check if the vector is empty.
-     *
-     * \see isEmpty()
-     *
-     * \deprecated
-     */
-     // BIC: remove
-    bool isNull() const;
-
     /*!
      * Returns true if the ByteVector is empty.
      *
@@ -568,17 +555,6 @@ namespace TagLib {
      */
     void swap(ByteVector &v);
 
-    /*!
-     * A static, empty ByteVector which is convenient and fast (since returning
-     * an empty or "null" value does not require instantiating a new ByteVector).
-     *
-     * \warning Do not modify this variable.  It will mess up the internal state
-     * of TagLib.
-     *
-     * \deprecated
-     */
-    static const ByteVector null;
-
     /*!
     * When used as the value for a \a length or \a patternLength parameter
     * in ByteVector's member functions, means "until the end of the data".

tests/test_bytevector.cpp

@@ -85,7 +85,6 @@ public:
 
     i.clear();
     CPPUNIT_ASSERT(i.isEmpty());
-    CPPUNIT_ASSERT(!i.isNull()); // deprecated, but worth it to check.
   }
 
   void testFind1()