more API review

git-svn-id: svn://anonsvn.kde.org/home/kde/trunk/kdesupport/taglib@774207 283d02a7-25f6-0310-bc7c-ecb5cbfe19da

taglib/ape/apeitem.cpp

@@ -147,6 +147,11 @@ StringList APE::Item::toStringList() const
   return d->text;
 }
 
+StringList APE::Item::values() const
+{
+  return d->text;
+}
+
 String APE::Item::toString() const
 {
   return isEmpty() ? String::null : d->text.front();

taglib/ape/apeitem.h

@@ -61,6 +61,7 @@ namespace TagLib {
       /*!
        * Constructs an item with \a key and \a value.
        */
+      // BIC: Remove this, StringList has a constructor from a single string
       Item(const String &key, const String &value);
 
       /*!
@@ -143,10 +144,16 @@ namespace TagLib {
       String toString() const;
 
       /*!
-       * Returns the value as a string list.
+       * \deprecated
+       * \see values
        */
       StringList toStringList() const;
 
+      /*!
+       * Returns the list of values.
+       */
+      StringList values() const;
+
       /*!
        * Render the item to a ByteVector.
        */

taglib/mpeg/id3v2/id3v2header.h

@@ -73,6 +73,10 @@ namespace TagLib {
        * Set the the major version number to \a version.  (Note: This is
        * the 4, not the 2 in ID3v2.4.0.  The 2 is implied.)
        * \see majorVersion()
+       *
+       * \note This is used by the internal parser; this will not change the
+       * version which is written and in general should not be called by API
+       * users.
        */
       void setMajorVersion(uint version);