Don't underflow if there are no embedded frames

Closes #513

This isn't tracked as part of the elementID anymore

Split chapter data and embedded frame data

This will allow us to test parsing them separately

Don't require users to include a padding byte explicitly

This makes it where the natural construction can be used of something
like:

new ChapterFrame("ID", ... )

Closes #514

Also test second constructor

Update signature and docs to be more in-line with TagLib's style

Merge pull request #544 from TsudaKageyu/advance

Use std::advance rather than a loop and increment.

It doesn't make sense to set the factory after construction

Closes #259

Make this macro work with current Qt versions

Closes #499

Add astylerc

Use std::advance rather than a loop and increment.

Merge pull request #521 from TsudaKageyu/gcc-warning

Fix a GCC/Clang warning about singed/unsigned comparison.

Merge pull request #542 from TsudaKageyu/comment-typos

Fix some typos in comment.

Fix some typos in comment.

Merge pull request #535 from FestusHagen/fh1.m_UintAmbiguity

Silenced uint ambiguity error.

Merge pull request #538 from gogglesmm/optimize-mid-to-uint

 bytevector.mid(pos,4).toUInt() => bytevector.toUInt(pos)

Merge pull request #536 from FestusHagen/fh1.m_CMakeSubProjectFix

Fix for using Taglib as a CMake sub directory project.

Merge pull request #533 from TsudaKageyu/bytevector-resize

Fix the wrong padding of ByteVector::resize().

Fix code styling

replace use of bytevector.mid(pos,4).toUInt() with more optimized bytevector.toUInt(pos)

Revert the last two commits.
But leave the tests unchanged, and add some comments.

Reduce redundant memset when resizing ByteVector.

Expand the internal buffer of ByteVector only if really needed.
Add tests for all execution paths of ByteVector::resize().

Fix for using Taglib as a CMake sub directory project.

Silenced uint ambiguity error.

Fix the wrong padding of ByteVector::resize().

The expanded area will be filled with garbage instead of correct padding in some corner cases.

Merge pull request #532 from TsudaKageyu/bytevector-detach-test

A little robuster tests for ByteVector iterators after detaching.

A little robuster tests for ByteVector iterators after detaching.

Merge pull request #531 from TsudaKageyu/bytevector-detach

Fix ByteVector to return correct iterators after detached.

Fix ByteVector to return correct iterators after detached.

Merge pull request #527 from TsudaKageyu/wma-guid

Fix saving WMA files with some GUID fields.

Store any GUID fields in Metadata Library Object.

Fix saving WMA files with some GUID fields.

Fix a GCC/Clang warning about singed/unsigned comparison.

Merge pull request #492 from TsudaKageyu/duplicate-id3v2-aiff-wav

Check AIFF/WAV files for duplicate tags.

Merge pull request #511 from TsudaKageyu/tag-prop

Fix a bug that Tag::setProperties() clears the date instead of the track number

Fix a bug that Tag::setProperties() clears the date instead of the track number.

Merge pull request #479 from TsudaKageyu/fuzzed-mp4

Fix an infinite loop when parsing MP4 files.

Fix infinite loops when parsing MP4 files.

Check AIFF/WAV files for duplicate tags.
AIFF/WAV files can have duplicate tags and it leads to memory leak.

Merge pull request #505 from TsudaKageyu/fuzzed-ape-tests

Separate the tests for fuzzed APE files.

Separate the tests for fuzzed APE files.

Merge pull request #496 from TsudaKageyu/fuzzed-mpc

Some fixes for fuzzed MPC files.

Check for EOF to fix a segfault while parsing fuzzed MPC files.

Check the packet size to fix a segfault error while parsing fuzzed MPC files.

Check for EOF to fix an infinite loop while parsing fuzzed MPC files.

Fix an out-of-bounds access and consequent errors while parsing fuzzed MPC files.
Consequent errors may vary: segfault, zerodiv and so forth.

Merge pull request #502 from TsudaKageyu/fix-save-wav

Fix saving ID3v2/INFO tags of WAV files.

Fix saving ID3v2/INFO tags of WAV files.
The old tag won't be removed when the new tag is empty.

Merge pull request #494 from TsudaKageyu/fix-zlib

Fix a wrong parameter for zlib.

Fix a wrong parameter for zlib.

z_stream.avail_in has to be the length of the input buffer.
It will fail when frameDataLength is smaller than the actual compressed data size.

Merge pull request #491 from TsudaKageyu/aiff-hasid3v2

Implement missing AIFF::File::hasID3v2Tag().

Implement missing AIFF::File::hasID3v2Tag().

Merge pull request #490 from TsudaKageyu/remove-uncompr

Remove an unused file from taglib/CMakeLists.txt.

Remove an unused file from taglib/CMakeLists.txt.
uncompr.c is no longer used since caa53e8

Merge pull request #489 from TsudaKageyu/frame-flag-tests

Add some tests for huge memory allocation due to bad ID3v2 frame header ...

Add some tests for huge memory allocation due to bad ID3v2 frame header flags.

The tests covers #466 and #486.
Also fixes a compilation error on some compilers.

Show a debugging message when we have an error reading

Read the compressed data as a stream

This avoids allocating the complete buffer at first based solely
on the value read from the frame header.  This then does a sanity
check at the end of reading to make sure that the two values match.
At present, it just prints a debugging message if the values do not
match.

Fixes #466

Merge pull request #485 from TsudaKageyu/oga-segfault

Fix a segfault when reading faulty Ogg/FLAC files.

Merge pull request #473 from TsudaKageyu/fix-infotag

Fix an infinite loop when parsing an INFO tag.

Add a check for faulty Ogg/FLAC files.

Separate tests for fuzzed WAV files.

Fix a segfault when reading faulty Ogg/FLAC files.

Fix a segfault when parsing WAV properties.

Add a test for fuzzed WAV files.

Fix an infinite loop when parsing an INFO tag.

Merge pull request #481 from TsudaKageyu/aiff-c

Add support for AIFF-C files.

Fix compilation errors on Clang.

Add support for AIFF-C files.

Merge pull request #476 from TsudaKageyu/fuzzed-ape

Some fixes for fuzzed APE files.

Merge pull request #456 from TsudaKageyu/id3v2-padding

ID3v2 padding won't increase beyond 1% of the file size.

Added some tests for fuzzed APE files.

Fix an excessive loop when parsing an APE file.

Fix a division by zero error when parsing an APE file.

Fix ID3v2 padding size calculation.

Merge remote-tracking branch 'TsudaKageyu/64bit-atom'

Conflicts:
ConfigureChecks.cmake

Remove CMake check for sizeof(long).

Compile without warnings with MSVC

Merge pull request #424 from TsudaKageyu/format

Unified redundant string format functions. (backport from taglib2)

Merge pull request #441 from xhochy/tests-fix-vectorsize

Fix ByteVector sizes in some test cases

Merge pull request #420 from TsudaKageyu/remove-codecvt

Replaced codecvt with Win32 API.

Merge pull request #454 from TsudaKageyu/duplicate-id3v2

Skip duplicate ID3v2 tags in MPEG files.

Merge pull request #458 from achalddave/fix-gcc-errors

Fix more uint/TagLib::uint ambiguities

Fix more uint/TagLib::uint ambiguities

ID3v2 padding won't increase beyond 1% of the file size.

Avoid reading an entire ID3v2 tag when skipping it.

Merge pull request #453 from TsudaKageyu/editorconfig

Suggestion: Use EditorConfig to help us stick to our coding style.

Add a test for duplicate ID3v2 tags.

Comment on a weird workaround for duplicate ID3v2 tags.

Skip duplicate ID3v2 tags in MPEG files.

Use EditorConfig to help us stick to our coding style.
http://editorconfig.org/

Merge pull request #448 from TsudaKageyu/msvc-error

Fix a compilation error on MSVC.

Fix a compilation error on MSVC.

Rebuild TRDC from v2.3 fields

This fixes an issue that was reported to me via email with the recording
date being thrown away from v2.3 tags.

Don't leak

Missing assignment

TableOfContents and ChapterFrame can be added to v2.3 or v2.4 tags

Take ownership of embedded frames, as documented

Previously embedded frames that were created automatically were
never deleted.

Fixes #440

Merge branch 'master' of https://github.com/taglib/taglib