[rfc][icedtea-web] refactored logging
Omair Majid
omajid at redhat.com
Tue Sep 24 07:31:07 PDT 2013
Hi Jiri,
On 09/24/2013 02:43 AM, Jiri Vanek wrote:
> One possible improvement (related to this change set) :
> There is 138 occurrences of log(Level level, Throwable s)
> There is 67 occurrences of log(Throwable s)
> The log(Throwable s) is call to log(Level.ERROR_DEBUG, (Object) s);
> and those log(Level level, Throwable s) are mostly
> log(Level.ERROR_ALL, throwable) calls
>
> I was thinking about changing it to
> There is 68 occurrences of log(Level level, Throwable s)
> There is 138 occurrences of log(Throwable s)
> where The log(Throwable s) will be call to log(Level.ERROR_ALL,
> (Object) s)
>
> What do you think?
I think this is one of those decisions where having some data would
allow for a better evaluation. Is the code calling log(Throwable)
expecting it to be user-facing or merely debugging information ?
Thanks,
Omair
--
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95 0056 F286 F14F 6648 4681
More information about the distro-pkg-dev
mailing list