[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: Patch: Colored debug
From: Thiago Macieira <thiago () kde ! org>
Date: 2007-08-31 8:19:00
Message-ID: 200708311019.00952.thiago () kde ! org
[Download RAW message or body]
On Friday 31 August 2007 03:31:11 Andreas Hartmetz wrote:
> On Thursday 30 August 2007 01:07:37 Olivier Goffart wrote:
> > Hi,
> >
> > I'd like to commit the attached patch.
> > It does coloration of the kdebug stream as seen on the screenshot
> > http://bepointbe.be/files/tmp/kdebug1.png
> >
> > I think it make the debug output much more easier to read.
> >
> > Is there any objections ? or do you want more colors ?
>
> I actually think that this is a great idea. While the text terminal is
> still the most powerful tool for some things there is no reason to pretend
> that it's still connected to a teletype (that's where "tty" comes from)
> that hammers black and white letters on paper. We should always use every
> facility available to us on the vast majority of systems, and colorful text
> is one of them. Go for it.
> Could you also make kWarning()'s (or whatever its name is) output red, bold
> or whatever? Warnings are more interesting if used correctly so they should
> stand out. If there's a warning there is a nontrivial chance that things
> are, actually, going really wrong.
> Thanks in advance :)
Mind you: the output is going through QDebug, which means it's handled by
qt_message_output. And the output from that can be redirected by installing a
message handler (qInstallMsgHandler).
All QtTest-based testcases have a message handler installed, so be careful
with what you're outputting. I somehow doubt that we want ANSI colouring in
XML.
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
["signature.asc" (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic