Romain Beauxis wrote:
Le Wednesday 12 December 2007 10:57:46 Samuel Mimram, vous avez écrit :
Romain Beauxis wrote:
I've greped for Printf.printf, and found some use that are legal, such as
in lang.ml, main.ml and dynliq.ml, and some that do not seem..
I've written quick and dirty patch to convert them to log#f, but perhaps
you'd like to have it another way ?
Yes, I would like you to leave it the way it is.
All the printf you changed are debug messages which are printed only if
a debug parameter is set to true. The only exception is vumeter which
should definitely use printf. For debugging purposes it's more
convenient to have debugging messages not mixed with other logs and
that's why printfs are nice.
Ok, I'll correct only printf that are not binded by a debug parameter then...
Thanks! Do you know if there are any of these?
Cheers,
Sam.