[flac-dev] [PATCH] Add missing functions to SeekTable class
Josh Coalson
xflac at yahoo.com
Sun May 6 16:18:46 PDT 2012
> From: Cristian Rodríguez <crrodriguez at opensuse.org>
> Sent: Sunday, May 6, 2012 8:24 AM
>
> El 05/05/12 02:23, Bastiaan Timmer escribió:
>> The attached patch adds the missing FLAC__metadata_object_seektable_*()
>> functions from FLAC's metadata object methods (FLAC/metadata.h) to
>> FLAC++'s SeekTable class. Of the 11 functions in the C API, only 4 are
>> currently in the C++ API, this patch adds the missing 7.
>>
>> If this patch is ok, VorbisComment will be next. A quick look tells me only
>> 5 out of 13 FLAC__metadata_object_vorbiscomment*() functions from
>> FLAC/metadata.h exist in FLAC++'s VorbisComment class.
>>
>> Bas Timmer
>>
> This patch looks fine to me.. if applied we either need to raise the C++
> library SOVERSION and/or add a linker versioning script for *nix systems
> though.
Looks good to me to. Erik, are you keeping up with patches? Lately I've only
been able to follow along.
The soname is a good point: the flac release numbers are tied to the format
and I rely on the sonames to manage binary compatibility. Some maintainers
don't like it but I think it's important for users:
http://flac.sourceforge.net/faq.html#api__release_versioning
More information about the flac-dev
mailing list