FLAC: AudioProperties improvements

Add lengthInSeconds(), lengthInMilliseconds() properties. (#503)
Add bitsPerSample() property besides sampleWidth(). (#360)
Remove some data members which are not needed to carry.
Add some tests for audio properties.
Add some supplementary comments.

Silence some MSVC specific warnings in tests.

Silence some MSVC2015 specific warnings.

MP4 atom length is limited up to 31 bits.
32-bit value will be negative.

Add some comments to tutils.h.

Reorder CMake checks for sprintf() variants.
VS2015 has snprintf(), however sprintf_s() is still recommended.

Remove sudo from .travis.yml.
This allows our tests to run faster on the container-based infrastructure.

Merge pull request #554 from FestusHagen/fh1.m_FriendlyCppUnitSearch

Don't look for CppUnit unless BUILD_TESTS, general cleanup.

Merge pull request #595 from TsudaKageyu/gcc-warning-test

Silence a GCC warning about ignoring a return value in test.

Fix a typo in a debug message.

Merge pull request #625 from TsudaKageyu/checksum-comment

(minor, comment only) Add a supplementary comment to ByteVector::checksum().

Merge pull request #617 from TsudaKageyu/useless-call

Remove an useless function call.

Update NEWS.
Fixed reading FLAC files with zero-sized padding blocks.

Merge pull request #638 from TsudaKageyu/update-news

Update NEWS.

Merge pull request #637 from TsudaKageyu/flac-zero-sized-padding

Make FLAC::File tolerant to zero-sized padding blocks.

Update NEWS.

Avoid creating zero-sized padding blocks.

Make FLAC::File tolerant to zero-sized padding blocks.

Add a supplementary comment to ByteVector::checksum().

Remove an useless function call.

Merge pull request #614 from TsudaKageyu/win32-pkg-config

Enable pkg-config on Windows.

Enable pkg-config on Windows.

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.

Silence a GCC warning about ignoring a return value in test.

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).

Undo comment stacking.

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

Removed missed unnecessary BUILD_TESTS check.

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.

Don't look for CppUnit unless BUILD_TESTS, general cleanup.

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.