<div dir="ltr">Sorry about the formatting. I use slightly different formatting and couldn't adjust my IDE just for taglib. My apologies. <div><br><div>About ID3v2FrameFactory - I agree it is a very small issue (unlike the ASF one), however, I thought that when no one holds a reference, why keep the object? Our software is server side based and must be extremely tight.</div>
<div><br></div><div>Too bad about the aac and m4v. We use it all the time so it will require us to always patch your "formal" versions. Most of the AAC we see is in MP4.</div><div><br></div><div>About the debug callback - I'm not showing this to the user. It gets written to the application logfile and is very helpful in troubleshooting issues, even when compiled with debug (NDEBUG). Keeping it in cout and just in debug mode is a waste. Besides - it is optional. If you do not set the callback, it never gets called. Please consider adding it.</div>
<div><br></div><div>Thanks you for this great library!</div><div><br></div><div>Tal</div><div><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 1:45 PM, Tal Sliwowicz <<a href="mailto:tal@tversity.com">tal@tversity.com</a>> wrote:<br>

><br>
> I've created several patches based on taglib 1.7. They can easily be merged<br>
> to the the master as well.<br>
> 1. Fixed memory leaks.<br>
<br>
</div>I disagree with "fixing" the ID3v2FrameFactory issue. It's a singleton<br>
that never gets more than one instance. I'll have a look at the ASF<br>
changes. Thank you.<br>
<br>
In general, the patches don't follow the TagLib coding style (no tabs,<br>
2 spaces, no space after if/for), so have to be reformatted.<br>
<div class="im"><br>
> 2. Added m4v and aac files to the list of files supported after testing it<br>
> works<br>
<br>
</div>I'll not merge this one, see <a href="https://github.com/taglib/taglib/pull/12" target="_blank">https://github.com/taglib/taglib/pull/12</a><br>
for the reason.<br>
<div class="im"><br>
> 3. Added a debug callback function. I use this to print messages to the log<br>
> file instead of cout<br>
<br>
</div>I'm not sure about this, the debug messages are meant to be debug<br>
messages, not general-purpose error messages displayed to the user.<br>
<br>
Lukas<br>
<div><div></div><div class="h5">_______________________________________________<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></div>