<div dir="ltr">Thanks for the answer.<div><br></div><div>I'll look into addFileTypeResolver.</div><div><br></div><div>About the debug callback set function - since I use it from C in a different DLL, it is easier to keep it outside the namespace from a linking perspective. That's why I opted to do it like this (couldn't get it to link otherwise). Agreed about the naming convention.</div>
<div><br></div><div>Tal<br><br><div class="gmail_quote">2011/8/16 Lukáš Lalinský <span dir="ltr"><<a href="mailto:lalinsky@gmail.com">lalinsky@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Tue, Aug 16, <a href="tel:2011" value="+9722011">2011</a> at 3:09 PM, Tal Sliwowicz <<a href="mailto:tal@tversity.com">tal@tversity.com</a>> wrote:<br>
> About ID3v2FrameFactory - I agree it is a very small issue (unlike the ASF<br>
> one), however, I thought that when no one holds a reference, why keep the<br>
> object? Our software is server side based and must be extremely tight.<br>
<br>
</div>My only problem is that it's a lot of code for no purpose. The few<br>
extra bytes will not cost you anything and the memory won't grow over<br>
time. I imagine that in a server application you are going to use<br>
TagLib more than once, so re-creating the object over and over again<br>
is eventually going to cost you a little more than the extra memory.<br>
<div class="im"><br>
> Too bad about the aac and m4v. We use it all the time so it will require us<br>
> to always patch your "formal" versions. Most of the AAC we see is in MP4.<br>
<br>
</div>You don't have to use a patched version of TagLib. You can use<br>
FileRef::addFileTypeResolver to add new file type resolvers from your<br>
application.<br>
<div class="im"><br>
> About the debug callback - I'm not showing this to the user. It gets written<br>
> to the application logfile and is very helpful in troubleshooting issues,<br>
> even when compiled with debug (NDEBUG). Keeping it in cout and just in debug<br>
> mode is a waste. Besides - it is optional. If you do not set the callback,<br>
> it never gets called. Please consider adding it.<br>
<br>
</div>I don't have a problem with adding it, but in this case it should be<br>
named differently. Either following the naming convention from the C<br>
API (i.e. taglib_XXX) or it should be a C++ function in the TagLib<br>
namespace.<br>
<div><div></div><div class="h5"><br>
Lukas<br>
_______________________________________________<br>
taglib-devel mailing list<br>
<a href="mailto:taglib-devel@kde.org">taglib-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/taglib-devel" target="_blank">https://mail.kde.org/mailman/listinfo/taglib-devel</a><br>
</div></div></blockquote></div><br></div></div>