MPEG: Hide an internal function from the public header.

MPEG: Remove unused formal parameters.

MPEG: Fix warnings about signed/unsigned mismatch on some compilers.

MPEG: No need to get the length of an ID3v2 tag twice.

MPEG: Properties::xingHeader() should return null if a VBR header is not found.

MPEG: AudioProperties improvements

Add lengthInSeconds(), lengthInMilliseconds() properties. (#503)
Support VBRI header in addition to Xing. (#136)
Fix MPEG frame seeker functions. (maybe #190)
Calculate MPEG frame length accurately.
Remove some data members which are not needed to carry.
Add some tests for audio properties.
Add some supplementary comments.

Merge pull request #611 from TsudaKageyu/fix-bufsize

Fix mismatched file I/O buffer sizes.

Fix mismatched file I/O buffer sizes.

Merge pull request #607 from TsudaKageyu/unused-function

Remove some unused private function prototypes.

Removed an unused data member from MPE::File.
It seems to be related to scan().

Remove some unused private function prototypes.

Merge pull request #594 from TsudaKageyu/typos-in-comment

Fix some typos in comments.

Fix some typos in comments.

Add const to docs

Merge pull request #591 from TsudaKageyu/wmemcpy

Use wmemcpy() rather than memcpy() and sizeof(wchar_t).

Use wmemcpy() rather than memcpy() and sizeof(wchar_t).

Prefix #ifndef with TAGLIB_ in tpropertymap.h

Merge pull request #577 from TsudaKageyu/frameoffset

Fix MPEG::File::firstFrameOffset() and lastFrameOffset(). (#190)

Merge pull request #578 from TsudaKageyu/test-temp

A lot of zero-sized temporary files are left after a test session on …

A lot of zero-sized temporary files are left after a test session on Windows.

Merge pull request #574 from TsudaKageyu/skip-dup-chunks

Skip duplicate chunks when reading AIFF/WAV files.

Fix MPEG::File::firstFrameOffset() and lastFrameOffset(). (#190)

Skip duplicate chunks when reading AIFF/WAV files.

Similar to #492.
There is no good reason to use the last chunk rather than the first one.

Merge pull request #572 from TsudaKageyu/map-erase2

Map::erase() can take a key directly so no need to call find() before…

Map::erase() can take a key directly so no need to call find() beforehand.

Merge pull request #571 from TsudaKageyu/fuzzed-wv

Avoid an infinite loop when reading fuzzed WavPack files. (#482)

Merge pull request #570 from TsudaKageyu/audioprop-typo

Fix a typo in comment.

Avoid an infinite loop when reading fuzzed WavPack files. (#482)

Fix a typo in comment.

Merge pull request #569 from TsudaKageyu/map-erase

std::map::erase() can take a key directly and has no-throw guarantee.

Merge pull request #484 from TsudaKageyu/fix-rfind

Fix File::rfind() for small files.

std::map::erase() can take a key directly and has no-throw guarantee.

TagLib::XM::Properties is a public class that should be exported

Merge pull request #550 from TsudaKageyu/fix-test

Fix test code to work on some environments.

Fix test code to work on some environments.
const char * is more preferable than string for ifstream constructor.

Bring the API more in line with the rest of TagLib

Like in #255, this also makes it possible to read values from the
tag in a const function.

Merge pull request #549 from TsudaKageyu/win32-test

Modify the test code to work on MSVC/Windows.

Modify the test code to work on MSVC/Windows.

This should also work Windows and has less duplicated code

Don't use tempnam on UNIX

This silences the huge stream of warnings when building the tests.

I think I didn't break the Windows version in the process (though
it may make sense to use the built in Windows functions there
instead), but I don't have a Windows build environment here, so
I can't test.

Merge pull request #546 from TsudaKageyu/test-surrogate-pair

Add a test for strings that contains surrogate pairs.

Merge pull request #548 from TsudaKageyu/render-twice

No need to call ID3v2::Frame::render() twice when saving an ID3v2 tag.

No need to call ID3v2::Frame::render() twice when saving an ID3v2 tag.

Quote path names including user-provided variables

This should make these work even if the value contains spaces

Closes #344

Add a test for strings that contains surrogate pairs.

Remove *file argument to private members

This is already covered by d->file, so there's no reason to pass a pointer
to the member dozens of times.

Moar const fixes

1001 Const fixes (plus iterator rename)

Remove unnecessary checks for null before delete

Closes #343

Merge pull request #515 from TsudaKageyu/empty-id3v2-frame

Skip empty ID3v2 frames when saving an ID3v2 tag.

Add accessors to manipulate MP4 tags without modifying the internal structure

This brings the MP4 API in line closer to other tag formats and makes it
possible to access the tag data from const functions.

"ItemListMap" has been renamed to "ItemMap" (with the old version
deprecated).  It seems that the "ListMap" notion was copied probably
from Allan's ApeTag implementation, which incorrectly copied the term
from the XiphTag.  Notably, in XiphTag, because a field can have multiple
values, the "ListMap" is a map of lists.  Calling things a "ListMap" where
there can be only one value doesn't fit.

Closes #255

Add isEmpty() to MP4

Closes #457

Show something useful for ChapterFrame::toString()

Closes #517

Fix File::rfind() for small files to work just like ByteVector::rfind().

Update signature and docs

Specifically this allows the frame to be constructed and then to have
the children and embedded frames set later.

Remove the null termination stuff from the ToC frames too

We've moved away from including the null byte in the returned value

This does change previous behavior, but the previous behavior was
particularly stupid and inconsistent with everything else in TagLib.

It should be possible to mitigate this by putting the same safety
guards in the TableOfContents

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 a test to check if an empty ID3v2 frame is really skipped.

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.

Take into account the frame header version when skipping an empty frame.

Discard empty ID3v2 frames instead of adding a dummy null byte.

Add a dummy byte to an empty ID3v2 frame to stick to the ID3v2 spec.

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.