]> granicus.if.org Git - taglib/commit
Consistently handle invalid and deprecated tags in setProperties()
authorMichael Helmling <supermihi@web.de>
Wed, 26 Dec 2012 21:46:37 +0000 (22:46 +0100)
committerMichael Helmling <supermihi@web.de>
Wed, 26 Dec 2012 21:46:37 +0000 (22:46 +0100)
commitf1d723077f2ee346182982d89febee0494a5b89e
tree782b35f4d31986c998c5570505a596d7676daeda
parenta095c468b2643f5a7c645d78fef91784092c30c9
Consistently handle invalid and deprecated tags in setProperties()

This commit reverts the use of strip() in setProperties() because the
latter function should not change the file before save() is called.
Instead, the following policy is now consistently applied for file formats
with multiple tag types:
- the recommended tag type is created, if it does not exist
- deprecated tags are updated, if they exist, but not created
- illegal tag types are ignored by setProperties(), but used in properties()
  if no others exist.

The only tag types considered "illegal" so far are APEv2 in MPEG and ID3 in FLAC.
taglib/ape/apefile.cpp
taglib/ape/apefile.h
taglib/flac/flacfile.cpp
taglib/flac/flacfile.h
taglib/mpc/mpcfile.cpp
taglib/mpc/mpcfile.h
taglib/mpeg/mpegfile.cpp
taglib/mpeg/mpegfile.h
taglib/wavpack/wavpackfile.cpp
taglib/wavpack/wavpackfile.h