ape tag isEmpty override?
lalinsky at gmail.com
Tue Jul 13 17:17:43 CEST 2010
On Tue, Jul 13, 2010 at 5:15 PM, Jim <jmartin92 at comcast.net> wrote:
> Lukáš Lalinský wrote:
>> On Fri, Apr 30, 2010 at 8:24 PM, Jim <jmartin92 at comcast.net> wrote:
>>> I need to check to see if an APE tag is empty (i.e. has no fields at all) but
>>> APE::Tag::isEmpty() seems to only check for the presence of the standard tag
>>> fields (like artist, album, title, etc.). Is this by design or an oversight?
>>> Should APE::Tag have its own isEmpty function or should I be using something
>>> like itemListMap's isEmpty?
>> I'd say it's an oversight. It should have its own implementation of
>> isEmpty(), just like ID3v2 tags have. The one inherited from
>> TagLib::Tag only checks for the standard fields. Could you please file
>> a bug report about it on bugs.kde.org?
> I will add a report for this. Is this also the place I should post wishes for
> 2.0? Or are you keeping a wish list for that somewhere else?
Actually, I've already implemented it. It will be released in 1.7.
> (I have a few minor changes I'd like to see that aren't really major reworks but
> 2.0 still might be a good time to implement them. For example having String to
> int conversion able to handle a leading '+' sign or having an option in
> MPEG::File::save to not copy the info between ID3 v1/v2 tags.)
Somebody already started working on abstract IO for TagLib 2.0, so
perhaps I should set up a new branch for more general 2.0 development.
More information about the taglib-devel