Oh, and about the file names, looks like there's no real standard yet:

boud@linux:~/kf5/src/frameworks> find . -name \*Debug\*h
./kactivities/src/service/Debug.h
boud@linux:~/kf5/src/frameworks> find . -name \*debug\*h
./krunner/src/declarative/krunner_debug.h
./krunner/src/krunner_debug.h
./networkmanager-qt/src/nmdebug.h
./threadweaver/src/debuggingaids.h
./kio/src/core/kiocoredebug.h
./kauth/src/kauthdebug.h
./kactivities/src/lib/core/debug_p.h
./kservice/src/sycoca/sycocadebug.h
./kwindowsystem/src/debug_p.h
./kjobwidgets/src/debug.h
./kdelibs4support/autotests/kdebug_unittest.h
./kdelibs4support/src/kdecore/kdebugdbusiface_p.h
./kiconthemes/src/debug.h
./sonnet/src/plugins/hunspell/hunspelldebug.h
./sonnet/src/plugins/voikko/voikkodebug.h
./khtml/src/ecma/debugger/debugdocument.h
./khtml/src/ecma/debugger/debugwindow.h
./khtml/src/ecma/kjs_debugwin.h
./baloo/src/file/baloodebug.h
./modemmanager-qt/src/mmdebug_p.h
./kconfig/autotests/kconfig_compiler/test_qdebugcategory_debug.h
./ktexteditor/src/utils/katepartdebug.h
./bluez-qt/src/debug.h

But most of it seems lower-case.


On Mon, 14 Sep 2015, Boudewijn Rempt wrote:

On Tue, 8 Sep 2015, Jaroslaw Staniek wrote:

On 8 September 2015 at 21:31, Friedrich W. H. Kossebau <kosse...@kde.org>
wrote:
Am Samstag, 5. September 2015, 12:55:28 schrieb Jaroslaw Staniek:
With increasing modularity it's quite important
for our debugging needs to have logging categories.

Agreed.

A scheme could be like
{libnamelowercase}{Debug|Warning|Critical}()

+1 for that scheme. Simply because that namespace-prefixing seems to match
the
usual namespace-prefixing as done with classes/files, don't see another
reason
for/against.

I also propose to use a libname prefix for the header name. This is
what many bits in KF5 do. So we won't have to deal with dozens files
prefixed with "Debug" soon (so libpigment's DebugPigment.h could
become PigmentDebug.h or pigment_debug.h).

That's sort of fine, I don't have a real preference there. I still think that
having one debug category per library is a limitation, and we'll need categories for debugging functional areas, not libraries, and funcational areas over-arch
libaries (whether that's good architecture, I don't know...)


When I looked into debug porting for libs/ last week (still need to
complete
end of next week, so anyone wants to take over meanwhile? no code yet
written
here), I wondered if installed headers better should not include any
debug-
only headers at all.

*debug.h should not be installed. I see that KF5 does not install own
headers.

But with publix/exported templates which need/better have debug statements
that might be unavoidable. Still not really sure though, it somehow seems
dirty to me :)

Is there at least such a case in calligra.git? We can add a template
specialization and keep the implementation in .cpp.

KoResourceLoader, I think.

I'm now porting libs to qCDebug. After thinking about it a bit,
I really, really, really don't like

vectorImageDebug(),

I think that

debugVectorImage()

is clearer and more grammatical as well. That may just be because we've had dbgCategory in Krita for ten years now, but this is the form I would
prefer to use.


Boudewijn
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to