Completely remove `StripAll` from the API

I'd imagined it being useful for calls to `strip()`, but it's not
actually used there since that's an OR-ed together set of flags
representing which tags to strip.

Unsaved (and incorrect looking) field

StripAll should be treated as equivalent to StripOthers in `save()`

Remove DSF and DSDIFF from master to a feature branch

These can be merged back into master once they're in a more mature state.

Unify File::save(...) APIs between file formats that support ID3v2

Closes #922

Revert switch to other static size method

This was based on a misread of the header:  at present there is no
non-static size() method, so removing the argument makes the behavior
incorrect.

Missing header that should have been added in b8dc105

Mark deprected methods and remove internal usage

This does not put the deprecated marker on methods that will or could resolve
to the same overload, e.g.:

void foo(bool bar = true); // <-- not marked
void foo(Bar bar) // <-- since this will have a default argument in the new version

Deprecate calls to MPEG::File::save(...) that use boolean params

This uses explicit enums for e.g. the ID3v2 version, making calls more
readable:

  file.save(ID3v1 | ID3v2, StripOthers, ID3v2::v4, Duplicate);

Instead of:

  file.save(ID3v1 | ID3v2, true, 4, true);

Needs to be ported to other types, per #922

Add docs for this method

No tabs in TagLib

Require at least CMake 3

Add -lz to taglib.pc and taglib-config when built with zlib

Closes #872

Clear valid flag for invalid Speex files

This matches the corresponding code in vorbisfile.cpp, opusfile.cpp and flagfile.cpp, and fixes taglib/taglib#902.

Merge pull request #883 from ufleisch/riff-padding

Do not ignore non zero RIFF padding if leading to parse error (#882)

Merge pull request #917 from ufleisch/ogg-bitrate

Calculate Ogg bitrate without comment size (#874)

Tests need C++11

Calculate Ogg bitrate without overhead size (#874)

Merge pull request #910 from joerg-krause/patch-1

Drop CMAKE_SYSROOT from taglib-config

Merge pull request #909 from williamjcm/patch-1

Make TagLib seach in its source dir for utf8-cpp.

Merge pull request #912 from whatdoineed2do/m4a-track-year-equal0-bugfix

MP4 - setTrack()/setYear() accepts 0 to remove the tag

Merge pull request #919 from jonaski/spelling

Fix spelling and typos

Fix spelling and typos

Call fflush() before ftruncate() to drop all buffered data (#914)

This avoids stale data presented to caller via a fread()

Current bug due to the buffered data can be seen in stripping mp3s of tags

    f.strip(ID3v1);
    f.strip(APE);

The ID3v1 tag sits at the end of file (strip calls ftruncate()) and the APE
strip performs a readFile() that would return the stream buffered/truncated data
and reinsert

setTrack()/setYear() accepts 0 to remove the tag as per
documentation/functionality across other tpyes (mp3/flac/...); m4a do not honour
this and instead sets the underlying value to 0.

This commit fixes this issue (#911)

Drop CMAKE_SYSROOT from taglib-config

Commit d4c938cbc766ffb400c09efb9e696fce2537d81e is about fixing taglib-config for proper cross-compilation. The fix is right in principle, but wrong about adding `CMAKE_SYSROOT`. The correct prefix path should be set outside of the config file as some embedded Linux distros like OpenWrt or OpenEmbedded install with a different DESTDIR, and dependent packages see a different sysroot.

Make TagLib seach in its source dir for utf8-cpp.

Without this change, TagLib is unusable as part of CMake projects that rely on `add_subdirectory()` for dependencies.

Respect atom type when converting rate tag (#896)

* Respect atom type when converting rate tag

TagLib prior to #818 (commit ff28cf276c60) read and wrote the "rate" tag as
text, and switched to reading it as integer value even when the atom class
is a text type. This breaks reading existing files, and can be avoided by
taking the atom type into account.

This fixes issue #885.

* Respect MP4::Item type when writing the rate tag

TagLib prior to #818 (commit ff28cf276c60) read and wrote the "rate" tag
as text, and switched to writing the integer value of the MP4::Item.
This breaks writing from applications which supply the value as
StringList, which was the previously implemented API. Applications using
an MP4::Item(UInt) are still supported without changes on the application
side.

This is the complementary writing part for issue #885.

Support ID3v2 GRP1 frame introduced with iTunes 12.5.4.42, #903. (#904)

fix taglib-config file for cross compiling (#906)

Update mp4properties.cpp (#893)

When parsing mp4 media header version 1 (mdhd) atoms, the timescale (unit) is parsed as a `LongLong` (8 bytes), but instead should be a `UInt` (4 bytes). This results in an incorrect timescale, and also pushes the offset of the duration (length) off by 4 bytes.

The end result being that the AudioProperties track length for mp4's with mdhd v1 comes back as 0.

See: https://wiki.multimedia.cx/index.php/QuickTime_container

|  Entry | Bytes (v0) | Bytes (v1) |
| :---         |     :---:      | :---: |
| size  | 4  | 4 |
| type  | 4  | 4 |
| version | 1 | 1 |
| flags | 3 | 3 |
| creation time* | 4 | **8** |
| modification time* | 4 | **8** |
| time scale | 4 | 4 |
| duration* | 4 | **8** |
| language | 2 | 2 |
| quality | 2 | 2 |

Minor fix for mp4 media header v0 minimum size check (#895)

Mp4 media header (mdhd) v0 atoms are a minimum of 8 bytes for size & type information, plus 24 bytes for remaining entries (`24 +8`) bytes in total, rather than (`24 + 4`).

See https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/QTFFChap2/qtff2.html#//apple_ref/doc/uid/TP40000939-CH204-25615

Do not ignore non zero RIFF padding if leading to parse error (#882)

Updated NEWS for the latest changes

Added APE, DSF, and DFF formats to the list

Added DSF and DSDIFF authors

Add DSF and DSDIFF file types management (#878)

* Add DSF and DSDIFF file types management

* Fixes for some build chains

* unit64_t replaced by unsigned long long, warning fixes

* Remove C++11 extension incompatible with some build chains (enumeration in a nested name specifier)

* Change typedef types (uint, ulong, ...) to standard types
remove BUILD_FRAMEWORK changes from this pull request

* Replace deprecated String::null and ByteVector::null by String() and ByteVector()
Styling update, thanks to FestusHagen

* Restyling

* Restyling to reduce length of excessively long lines

* Add to detectByExtension

* Added `isSupported(IOStream *stream)` to `DSF::File` and `DSDIFF::File`

Improve CMake VISIBILITY_HIDDEN option handling (#810)

Use standard CMake's CXX_VISIBILITY_PRESET property.

added OS X built files to .gitignore (#828)

Fill `TableOfContentsFrame::toString()`. (#852)

Follow TagLib's brace style

Add support for file descriptor to FileStream (#832)

Add support for file descriptor

Don't list the description twice in `UserTextIdentificationFrame::toString()` (#853)

Fixed OOB read when loading invalid ogg flac file. (#868) (#869)

CVE-2018-11439 is caused by a failure to check the minimum length
of a ogg flac header. This header is detailed in full at:
https://xiph.org/flac/ogg_mapping.html. Added more strict checking
for entire header.

Add support for cmID, purl, egid MP4 atoms (#862). (#863)

Update links

Fix crash when loading an empty mpeg file (#830)

When loading an empty file (empty.txt -> empty.mp3 ), TagLib will crash.
buffer.size() is 0, then buffer.size() - 1 is undefined (unsigned int)

Don't assume TDRC is an instance of TextIdentificationFrame (#831)

If TDRC is encrypted, FrameFactory::createFrame() returns UnknownFrame
which causes problems in rebuildAggregateFrames() when it is assumed
that TDRC is a TextIdentificationFrame

taglib: fix test build failure on powerpc/c++11 (#834)

powerpc is a platform with 'char' == 'unsigned char'.
As a result '-1' is not expressible in char and build fails as:

```
    # '-funsigned-char' to force test build failure on other platforms
    $ cmake .. -DBUILD_TESTS=YES -DCMAKE_CXX_FLAGS="-O2 -funsigned-char" -DCMAKE_C_FLAGS="-O2 -funsigned-char"
    ...
    $ make check
    tests/test_synchdata.cpp: In member function 'void TestID3v2SynchData::testToUIntBroken()':
    tests/test_synchdata.cpp:78:33: error: narrowing conversion of '-1' from 'int' to 'char' inside { } [-Wnarrowing]
     char data[] = { 0, 0, 0, -1 };
                                 ^
```

The fix is to expliticly cast -1 to 'char'.

Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>

Check if mandatory header objects are present when opening ASF files.

Also removes some assignments of "this".
It feels too tricky when it is not absolutely necessary.

Use a macro to pretend virtual functions.

Fix some typos in debug messages.

Update NEWS.

Merge pull request #799 from TsudaKageyu/filetype-detection

Enable FileRef to detect file types by the actual content of a stream.

TableOfContentsFrame depends on ByteVectorList.

Merge pull request #824 from evpobr/fix-createfile2

Fix WinRT configuring

Merge pull request #823 from TsudaKageyu/isolate-3rdparty-lib

Isolate 3rdparty library for easier maintenance.

Fix WinRT configuring

Don't rely on _WIN32_WINNT value to enable WinRT support.

if _WIN32_WINNT is not set manually, it is defaulted to SDK version. So
if you use SDK > 8 you cannot use TagLib under Win7 and lower because of
CreateFile2 function dependency.

PLATFORM_WINRT option (OFF by default) was introduced to enable WinRT
build.

Related issues: https://github.com/Microsoft/vcpkg/issues/1240

Isolate 3rdparty library for easier maintenance.

Update UTF8-CPP to v2.3.5.

Remove an unused include directory from taglib/CMakeLists.txt.

Restore FileRef::create() in order not to change the previous behavior.

Update INSTALL.md

Added a note that with MS Visual Studio 2017 one can directly open the CMake project.

Merge pull request #821 from supermihi/master

Fix Markdown syntax of INSTALL file and rename it to *.md

Fix Markdown syntax of INSTALL file and rename it to *.md

This enables proper formatted display of the file on e.g. GitHub, increasing readability.

Mention that FileRef::create no longer works.

Update NEWS.

Merge pull request #818 from LindyBalboa/add_rate_atom_support

Add direct support for "rate" atom

Fix memory leaks in a test.

Fix MSVC warnings about unreferenced variables.

Fix the Travis-CI testing on OS X.

AppleClang 7.3 doesn't get along with CppUnit by default.

Add direct support for "rate" atom

Resolves #817

Remove an useless UTF-8 BOM.

Allow SYLT frames to have a timestamp with no text.

Thanks to lemonboy999.

FilePrivate is responsible to delete a stream pointer instead of File.

Generally, TagLib leaves the Private classes to delete their members.

Merge branch 'filetype-detection' of https://github.com/TsudaKageyu/taglib into filetype-detection

Rename the functions 'isValidStream' to 'isSupported'.

The name 'isValidStream' is a little misleading because it doesn't check if the stream is really valid. Additionally, 'isSupported' can be naturally overloaded.

Remove some redundant code.

Fix wrong endian of boolean values when saving ASF files.

Restore the layout of the copyright header of test_fileref.cpp.

Enable FileRef to detect file types by the actual content of a stream.

FileRef doesn't work with ByteVectorStream as reported at #796, since ByteVectorStream is not associated with a file name and FileRef detects file types based on file extensions.
This commit makes FileRef to work with ByteVectorStream by enabling it to detect file types based on the actual content of a stream.

Remove obsolete comments.

The bug mentioned in the comments are already fixed.

Ignore warnings about OSAtomicIncrement32Barrier

The warnings suggest moving to std::atomic functions, but those are only
available in C++11.  It would be possible to switch to the C versions of
those functions, which are now provided in stdatoic.h (in C11), but
let's wait until we actually hit problems with this function and are a
few more OS versions into C11 headers being included by default.

Avoid searching the same area twice in MPEG::File::previousFrameOffset().

Add some test cases for invalid UTF-8 sequences.

Fix and add some test cases for invalid surrogate pairs.

Update NEWS.

Remove redundant functions in tstring.cpp.

Two versions of copyFromUTF16() are almost the same.

Remove function bodies of some non-specialized template functions.

The code won't link when a wrong version is used. It's better than showing a debug message.

Add a few more test cases for invalid UTF-8 sequences.

Merge pull request #794 from TsudaKageyu/utf8-library

Replace unicode.h/unicode.cpp by the UTF8-CPP library.

Check an invalid UTF-8 sequence consists of single char.

Single char can be an invalid UTF sequence. For example, { 0x80 } is invalid.

Stop assuming that std::wstring has a contiguous and null-terminated buffer.

Reduce useless memory reallocation in String::upper().

Remove a duplicate test file.

Remove some redundant code in tstring.cpp.

Replace unicode.h/unicode.cpp by the UTF8-CPP library.

unicode.h/unicode.cpp are no longer maintained and incompatible with Debian's guideline.
UTF8-CPP is maintained on GitHub and published under the Boost Software License which is compatible with either LGPL or MPL and should go along with Debian's guideline.

Add a test for unpaired surrogate characters in a UTF-16 string.

Remove some redundant code.

TagUnion::access() does the same thing as FLAC::File::ID3v2Tag().

Prefer isEmpty()/empty() to size() == 0.