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.

Revert useless changes accidentally committed.

Add missing consts.

Combine two internal functions which are always used together.

Ignore fake MPEG frame headers when seeking them.

Efficient lookup for an ID3v2 tag in MPEG files with garbage.

This looks for an ID3v2 tag until reaching the first valid MPEG frame in O(n) time.

Update NEWS.

Merge pull request #791 from TsudaKageyu/flac-empty-seektable

Be tolerant of empty FLAC seektable blocks.

Initialize d-pointers in class member initializer list.

Always use parentheses with new. It's a bit safer.

Initialize all the data members of ASF::Attribute.

Be tolerant of empty FLAC seektable blocks.

Remove some TODO comments which are no longer necessary.

Fix #789 (typo in member doc)

Merge pull request #787 from rshanmu/master

Renamed unsupported-extension.xxx and modified test

Renamed unsupported-extension.xxx and modified test

The file name unsupported-extension.xxx causes issue when unpacked
taglib-xxx.gz over an NFS partition. The file extension ".xxx" is the
one NFS uses for its own purpose and hence it will not allow creation
of this file.

Hence renamed the file and modified the test cases that checks for this
file.

Signed-off-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@bp.renesas.com>

Remove the CMake check for Boost I missed out on.

Remove all the optional dependencies on Boost.

Fix the CMake check for std::atomic_int.

std::atomic_int of Visual C++ 2012 cannot be constructed with integer.

Safer conversion of boolean values in ASF attributes.

Technically, boolean values in Extended Content Description Object is not necessarily be 0 or 1.

Make use of increment/decrement operators of std::atomic.

Remove optional dependencies on Boost's dynamic libraries.

Using precompiled Boost libraries can lead to depending on external dynamic libraries.

Merge pull request #784 from haoxi911/master

Fix #667: Compiled TagLib framework for OS X fails at codesign.

Fix #667: Compiled TagLib framework for OS X fails at codesign.

Rewrite ByteVector::replace() to run in O(n) time.

Add a test to check if ByteVector is detached correctly when being replaced.

#include guards in CMake generated headers.

Update NEWS.

Fix possible Ogg packet losses.

Prevent the segment table of Ogg pages from exceeding the size limit.

Remove Utils::floatByteOrder() and use systemByteOrder() instead.

We can safely assume that the integer and float byte orders are the same on IEEE754 compliant systems.

Add a test for #743/#779.

Small fix in style.

Merge pull request #779 from supermihi/master

Fixes #743 by not overwriting existing Xiph comment in FLAC::File::save

Fixes #743 by not overwriting existing Xiph comment in FLAC::File::save

A bit more tolerant check to return itself in String::substr().

Merge pull request #778 from martin-flaska/optimization

String::substr optimization

tstring: String::substr optimization when returning itself as a substring

Use copy ctor to return in a case whole string is being returned.

The intention was to optimize String::stripWhiteSpace for no-strip case
(without any leading or trailing white space removal).

copyFromUTF16 was used in any case previously and allocated duplicate
buffer for the same string - no implicit sharing.

Signed-off-by: Martin Flaska <martin.flaska@legrand.us>

test_string: Make 'stripWhiteSpace' test more complex

No string without leading/trailing spaces was used in the test.

Signed-off-by: Martin Flaska <martin.flaska@legrand.us>

Suppress MSVC warnings in test.

Merge pull request #776 from mathbunnyru/small_improvements

Small improvements

Small fixes

Increment fixes

Update NEWS.

Merge pull request #772 from TsudaKageyu/vorbis-fields

Fix handling of lowercase 'metadata_block_picture' field

Fix handling of lowercase 'metadata_block_picture' fields in Vorbis comments.

Also refactored some redundant code for parsing pictures.