[Review][IcedTea-Web]: Add tracing to itw

Omair Majid omajid at redhat.com
Fri Feb 18 11:58:08 PST 2011


Hi,

I talked to Andrew Su offline about this (a while ago actually), so this
is mostly for the list.

On 01/21/2011 11:56 AM, Andrew Su wrote:
> Here is a patch to add some tracing to indicate which part of the
> plugin is being executed at the moment. There are still some more
> places to add the traces to, so it's only a preview to see if there
> are any objections to how it's being done.
>

I don't think this is the right approach at all. As far as I know, the
only place where tracing is (supposed to be?) visible to a user (using
the proprietary plugin) is through the Java Console. This patch makes it
so that tracing and our Java Console don't mix.

A much simpler solution, IMHO is to simply fix the existing debug
statements and add new ones where appropriate. The Java Console shows
the debug statements just fine; they will simply double as trace
statements too.

Thanks,
Omair



More information about the distro-pkg-dev mailing list