Merge pull request #685 from g-coder/patch-2

Added ccache support for faster compilation

Update CMakeLists.txt

Added ccache support for faster compilation

ccache is a compiler cache. It speeds up recompilation by caching previous compilations and detecting when the same compilation is being done again. It provides support for C/C++ languages.
It makes compilation fast if it is available on the machine.
https://ccache.samba.org/

Small cosmetic fixes.

Fix instance references to a static member function.

Small cosmetic fix.

Merge pull request #676 from TsudaKageyu/string-hide-private

Hide private things from a public header.

Hide some private functions from a public header.

Hide a private static variable.

This is so-called Scott Mayers' singleton pattern.

Remove duplicate tags when saving AIFF files.

Just the same way as WAV already does.

Reduce redundant ref()/deref() operations.

Fix an instance reference to a static data member.

Silence some GCC warnings about no return statement.

Avoid using obsolete XiphComment::removeField().

Update NEWS.

Merge pull request #681 from TsudaKageyu/xiph-remove-fields

Add alternative methods to XiphComment::removeField() that may cause …

Merge branch 'genversion' of https://github.com/ufleisch/taglib into ufleisch-genversion

# Conflicts:
# CMakeLists.txt

Make FileRef::FileTypeResolver work properly.

Style fixes in fileref.cpp.

Fix a typo in a comment.

Update NEWS.

Style fixes in fileref.cpp.

Merge pull request #528 from chouquette/master

FileRef: Allow an IOStream to be used

Unify common functions for finding tags.

Several classes have exactly identical functions for finding tags.
This also hides the functions from public headers.

Add alternative methods to XiphComment::removeField().

Using XiphComment::removeField() may lead to a linkage error, however we can't fix the method itself without breaking the ABI or changing its behavior.
So we added some alternative method and marked the old one deprecated.

Update NEWS.

Merge pull request #654 from TsudaKageyu/xiph-setcomment

Fix XiphComment::setComment() for the case that a Vorbis comment has …

Avoid using String::null where an empty string is required.

String::null is not necessarily be empty or remains the same instance.
Using it in a public header may lead to a linkage error.

Avoid using String::isNull() where it is considered to be confused with isEmpty().

Revert "Add a test about handing "COMMENT" and "DESCIPRION" fields in XiphComment."

This reverts commit 8f147034d6af7a4c2830b3113d4b257c1836fabe.

Revert "Use always "COMMENT" field when updating XiphComment."

This reverts commit ba5137bf2d3b1bcca9269e3a7f5f194243c87d23.

Merge pull request #643 from TsudaKageyu/save-asf

Avoid writing duplicate tags when saving ASF files.

Remove a warning from a comment and update NEWS.

Avoid writing duplicate tags when saving ASF files.
Reduce memory reallocations and copies when saving ASF files.

Update NEWS.

Update NEWS.

Merge pull request #642 from TsudaKageyu/strip-and-properties

Fix segfaults when calling File::properties() after strip().

Update NEWS.

Merge pull request #623 from TsudaKageyu/erase-duplicate-id3v2

Skip duplicate ID3v2 tags and treat them as an extra blank of the first one.

Consistent rounding when calculating the MP4 audio length.

Update NEWS.

Efficient lookup for an ID3v2 tag in a MPEG file.

An ID3v2 tag or MPEG frame is most likely be at the beginning of the file.

Efficient lookup for the MP4/ASF field name and ID3v1 genre tables.

Linear lookup is much faster and memory efficient when an array is very small.

Add some tests for String::toInt() with too large values.

Backport shorter versions of some functions from taglib2.

Fix a string conversion bug in tag_c.cpp.

Avoid using ByteVector::null where an empty vector is required.

ByteVector::null is not necessarily be empty or remains the same instance.
Using it in a public header may lead to a linkage error.

Efficient lookup for the ID3v2 frame ID table.

Linear lookup is much faster and memory efficient when an array is very small.

Merge pull request #493 from TsudaKageyu/nested-refcounter

Simplify overly complicated ByteVector::mid() implementation.

Resolve some conflicts before merging.

Consistent notations between ByteVector::data() and at().

Use linear search instead of the Knuth-Morris-Pratt algorithm in ByteVector::find().
In almost all cases, it handles too small data for KMP to work effectively.

Replace DATA macro with more straightforward notations.

Simplify overly complicated ByteVector::mid() implementation.
Especially remove the useless nested RefCounters.

Use the same type name between a List and its iterator.

Merge pull request #661 from ufleisch/podcast-frames

Support for Apple podcast frames

Merge pull request #628 from RyanLucchese/rl-ConstReverseIterator-solaris-bug

return const correct reverse iterator for newer Sun compiler

Merge pull request #629 from RyanLucchese/rl-fix-class-instantiation-solstudio

check SUNPRO_CC version before using Map workaround

Merge pull request #608 from TsudaKageyu/mp4-has-tags

Add a method to check if an MP4 file on disk actually has a tag.

Make use of List iterators.

Use List::isEmpty() rather than size() to check if the list is empty.

std::list::empty() is guaranteed to be an O(1) operation.

Make use of List iterators.

Make use of List iterators and setAutoDelete().

Add a comment to List::isEmpty().

Separate some tests for MP4::File::hasMP4Tag().

Amend the comment on MP4::File::hasMP4Tag().

Reduce duplicate code between String::ctor() and operator=().

It's safer to have only one pair of ref()/deref() than to have a several pairs.

Mark some variables and functions deprecated.

ByteVector::null, ByteVector::isNull(), String::null, String::isNull() have got marked deprecated.
It's dangerous to use them, and they will be removed in taglib2.

Add a test for the CRC checksum of Ogg pages.

Add some notes about ByteVector::isNull() and ByteVector::null.

Add some notes about String::isNull() and String::null.

Add String::clear() method to clear the string.

vsnprintf()/vsprintf() does not necessarily return -1 when failed.

Use std::wstring::empty() rather than size() == 0.

Depending on the implementation, empty() can be more efficient than size().

Fix the usage of boost::endian::endian_reverse().

Skip duplicate ID3v2 tags and treat them as an extra blank of the first one.

This enables all the file formats to handle duplicate ID3v2 tags properly and erase them automatically.

Correct the ID3v2 padding size calculation.

Reduce memory reallocation when rendering an ID3v2 tag.
Prevent an ID3v2 padding from being ridiculously large.

Remove an unused private data member.

Add a method to check if an MP4 file on disk actually has a tag.

Add a test about handing "COMMENT" and "DESCIPRION" fields in XiphComment.

Make use of the Boost Endian library for byte swapping.

It's likely to be better at choosing the most efficient method than our CMake tests.

Silence some MSVC security warnings by replacing strdup() with _strdup().

Backported from taglib2.

Use always "COMMENT" field when updating XiphComment.

The recommended field name for additional comments is "COMMENT".
It's the same behavior as "DATE" or "TRACKNUMBER" field.

Fix XiphComment::setComment() for the case that a Vorbis comment has the "COMMENT" field.

Hide private things from a public header.

Some functions and a variable in tstring.h are not needed to be exposed in a public header.

Fix an instance reference to a static data member.

Small cleanups in CMakeLists.txt.

Small cleanups in audioproperties.cpp.

A bit more accurate calculation of the AIFF audio length.

Actually, it's unlikely to improve the accuracy, but prevents a useless round-trip conversion between double and int.

Skip both ID3v1 and APE tags when seeking the last MPEG frame.

1.10

Use List::isEmpty() than size() > 0.

Small revision of pokowaka's fix. isEmpty() is a little better than size() > 0, since std::list::empty() is guaranteed to be an O(1) operation.

Merge pull request #675 from garima-g/patch-1

Add self-assignment check in operator=

Add self-assignment check in operator=

Method 'operator=' should check its argument with 'this' pointer.

Add self-assignment check in operator=

Method 'operator=' should check its argument with 'this' pointer.

Add self-assignment check in operator=

Method 'operator=' should check its argument with 'this' pointer.

Add self-assignment check in operator=

Method 'operator=' should check its argument with 'this' pointer.

Merge pull request #669 from pokowaka/bad_access

Fixes access violation

Fixes access violation

- Fixes access violation when setting empty stringlist on integer
  properties in mp4 tag
- Add a unit test that validates the fix.