2007-02-07: Frank Zschockelt dixit: > * Bob Copeland <[EMAIL PROTECTED]> [070207 16:19]: > > > > I think nobody wants to have whitespace tags... > > > This is why, failing removal in taglib, I propose to remove > > > it in set_tag_props() of rio_rw.c (just a few lines patch). > > > If taglib eventually changes its behaviour, we will then > > > remove the patch. > Sounds good. > > > I could go either way. I don't think trimming in libkarma > > proper could hurt anything. If we want to ask taglib to do > > it, then we could implement it in lk for now and remove it > > when taglib implements it or if someone complains.
O.K. I'll send the patch tomorrow. > Sure, i just thought that it might be useful for other > applications, which are using taglib as well. I've been inspecting more id3v1-tagged tunes. Not all are padded with trailing whitespace; some are padded with NULLs. Recent versions of lame (>=3.96.1) pad with NULLs, but old lame versions, as well as recent mp3info (0.8.4) yield blank-padding. In any case, by default, I think it is worth to get rid of any trailing whitspace... I'll report this at [EMAIL PROTECTED] Best, EV. ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier. Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ linux-karma-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-karma-devel
