FS#7426 - Remove debug output from taglib.
Attached to Project:
Arch Linux
Opened by Sander Jansen (GogglesGuy) - Tuesday, 12 June 2007, 03:51 GMT
Last edited by Jan de Groot (JGC) - Thursday, 14 June 2007, 06:08 GMT
Opened by Sander Jansen (GogglesGuy) - Tuesday, 12 June 2007, 03:51 GMT
Last edited by Jan de Groot (JGC) - Thursday, 14 June 2007, 06:08 GMT
|
Details
By default taglib is compiled to print out debug information
to std:err/out. When reading files it may print out warnings
like this:
TagLib: MPEG::Header::parse() -- Invalid sample rate. TagLib: String::prepare() - Invalid UTF16 string. IMHO libraries should never print out to the terminal, neither GUI apps which may use these libraries. Please use --disable-debug on the ./configure command line to disable this. |
This task depends upon
Only when compiled for debug, they should print out asserts or other warnings that may be useful for developers. As a end user you're not interested in those warnings. Warnings or errors the user should see, should be passed to the application. Let the application decide whether to display it on the terminal or in some other way.
Now in case of tag lib the debug information printed out is not very useful for the end user anyway, since you have no idea which file it is referring too. Of course if you only open one file with tag lib, you may know, but if the application is scanning your whole harddrive you don't know because scanning a couple of 100 files goes very quickly. Also when you read in a lot of files, printing stuff out to a slow terminal will slow things down as well.
Anyway lots of reasons to disable debug output. (and might I add that even taglib considers this debug output).