Merge pull request #484 from TsudaKageyu/fix-rfind

Fix File::rfind() for small files.

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.

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.