Filterin the debug output

Thiago Macieira thiago at kde.org
Tue Oct 23 12:37:45 BST 2007


Em Tuesday 23 October 2007 13:22:34 Boudewijn Rempt escreveu:
> On Tue, 23 Oct 2007, Boudewijn Rempt wrote:
> > Indeed. And making non-existing debug areas give no output by default
> > would make sure people know when they are using them.
>
> Something like
>
> Index: kdebug.cpp
> ===================================================================
> --- kdebug.cpp	(revision 728385)
> +++ kdebug.cpp	(working copy)
> @@ -314,7 +314,7 @@
>          }
>
>          KConfigGroup cg(config, QString::number(area));
> -        int mode = cg.readEntry(key, 2);
> +        int mode = cg.readEntry(key, 4);
>          return OutputMode(mode);
>      }
>
>
> Should work, I guess.

I almost sent that patch, but that doesn't solve the problem. It'll simply 
make the default for ALL areas be NoOutput. That's not what we want.

The attached patch does it (untested, but compiles).

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdebug-no-output-for-unknown-areas.patch
Type: text/x-diff
Size: 1566 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071023/ac0c3a0b/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071023/ac0c3a0b/attachment.sig>


More information about the kde-core-devel mailing list