[Development] QLog ( Work on qDebug and friends)

[wolfgang.beck at nokia.com]

Well we have 3 different ways and priorities to set the config file.

One of the possibilities is:
We have a default config file which uses a hardcoded filename but the path to this file is created with the default config file path + organization name + application name.
For detailed information please visit:
http://codereview.qt-project.org/13226

B.R.
 WB




-----Original Message-----
From: development-bounces+wolfgang.beck=nokia.com at qt-project.org [mailto:development-bounces+wolfgang.beck=nokia.com at qt-project.org] On Behalf Of ext Thiago Macieira
Sent: Tuesday, February 21, 2012 3:18 AM
To: development at qt-project.org
Subject: Re: [Development] QLog ( Work on qDebug and friends)

On segunda-feira, 20 de fevereiro de 2012 16.42.00, David Faure wrote:
> > Clearly you haven't read 
> > http://codereview.qt-project.org/#change,13226
> >
> > 
> >
> > > We've removed all traces of
> > > QSettings support so QSettings could be moved away.
> 
> QFile is all that's needed to implement a basic config file, QSettings 
> is not  needed.

No, I haven't looked at the patch yet. (Take a look at my dashboard to see
why)

Let QLog have a function that allows one to externally enable and disable categories. If you really want to, provide a way to parse config data via QFile, like you suggested.

But please don't hardcode any file names inside the library.

--
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
     Intel Sweden AB - Registration Number: 556189-6027
     Knarrarnäsgatan 15, 164 40 Kista, Stockholm, Sweden