From b795e8d7c2dbb04ed9bde4500a46511339b8df31 Mon Sep 17 00:00:00 2001 From: Kai Koehne Date: Fri, 11 Jul 2014 11:11:46 +0200 Subject: Remove normalization of "default" category Commit d61e774307819 got rid of the special handling of the 'default' category in the qloggingregistry, so we don't need the private header file anymore, nor does it gain us anything to 'normalize' all default categories to one string. Change-Id: I027b4b82ff915428287432f39f76b4e719cf7874 Reviewed-by: Oswald Buddenhagen Reviewed-by: Thiago Macieira Reviewed-by: Alex Blasche --- src/corelib/io/io.pri | 1 - src/corelib/io/qloggingcategory.cpp | 12 ++----- src/corelib/io/qloggingcategory_p.h | 66 ------------------------------------- src/corelib/io/qloggingregistry.cpp | 1 - 4 files changed, 3 insertions(+), 77 deletions(-) delete mode 100644 src/corelib/io/qloggingcategory_p.h (limited to 'src') diff --git a/src/corelib/io/io.pri b/src/corelib/io/io.pri index f651860e08..7afff7ff81 100644 --- a/src/corelib/io/io.pri +++ b/src/corelib/io/io.pri @@ -51,7 +51,6 @@ HEADERS += \ io/qfileselector.h \ io/qfileselector_p.h \ io/qloggingcategory.h \ - io/qloggingcategory_p.h \ io/qloggingregistry_p.h SOURCES += \ diff --git a/src/corelib/io/qloggingcategory.cpp b/src/corelib/io/qloggingcategory.cpp index 63241f6698..a368e92932 100644 --- a/src/corelib/io/qloggingcategory.cpp +++ b/src/corelib/io/qloggingcategory.cpp @@ -40,7 +40,6 @@ ****************************************************************************/ #include "qloggingcategory.h" -#include "qloggingcategory_p.h" #include "qloggingregistry_p.h" QT_BEGIN_NAMESPACE @@ -221,15 +220,10 @@ void QLoggingCategory::init(const char *category, QtMsgType severityLevel) { enabled.store(0x01010101); // enabledDebug = enabledWarning = enabledCritical = true; - const bool isDefaultCategory - = (category == 0) || (strcmp(category, qtDefaultCategoryName) == 0); - - // normalize "default" category name, so that we can just do - // pointer comparison in QLoggingRegistry::updateCategory - if (isDefaultCategory) - name = qtDefaultCategoryName; - else + if (category) name = category; + else + name = qtDefaultCategoryName; if (QLoggingRegistry *reg = QLoggingRegistry::instance()) reg->registerCategory(this, severityLevel); diff --git a/src/corelib/io/qloggingcategory_p.h b/src/corelib/io/qloggingcategory_p.h deleted file mode 100644 index 7802f017fd..0000000000 --- a/src/corelib/io/qloggingcategory_p.h +++ /dev/null @@ -1,66 +0,0 @@ -/**************************************************************************** -** -** Copyright (C) 2013 Digia Plc and/or its subsidiary(-ies). -** Contact: http://www.qt-project.org/legal -** -** This file is part of the QtCore module of the Qt Toolkit. -** -** $QT_BEGIN_LICENSE:LGPL$ -** Commercial License Usage -** Licensees holding valid commercial Qt licenses may use this file in -** accordance with the commercial license agreement provided with the -** Software or, alternatively, in accordance with the terms contained in -** a written agreement between you and Digia. For licensing terms and -** conditions see http://qt.digia.com/licensing. For further information -** use the contact form at http://qt.digia.com/contact-us. -** -** GNU Lesser General Public License Usage -** Alternatively, this file may be used under the terms of the GNU Lesser -** General Public License version 2.1 as published by the Free Software -** Foundation and appearing in the file LICENSE.LGPL included in the -** packaging of this file. Please review the following information to -** ensure the GNU Lesser General Public License version 2.1 requirements -** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. -** -** In addition, as a special exception, Digia gives you certain additional -** rights. These rights are described in the Digia Qt LGPL Exception -** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. -** -** GNU General Public License Usage -** Alternatively, this file may be used under the terms of the GNU -** General Public License version 3.0 as published by the Free Software -** Foundation and appearing in the file LICENSE.GPL included in the -** packaging of this file. Please review the following information to -** ensure the GNU General Public License version 3.0 requirements will be -** met: http://www.gnu.org/copyleft/gpl.html. -** -** -** $QT_END_LICENSE$ -** -****************************************************************************/ - -#ifndef QLOGGINGCATEGORY_P_H -#define QLOGGINGCATEGORY_P_H - -// -// W A R N I N G -// ------------- -// -// This file is not part of the Qt API. It exists for the convenience -// of a number of Qt sources files. This header file may change from -// version to version without notice, or even be removed. -// -// We mean it. -// - -#include - -QT_BEGIN_NAMESPACE - -// unique pointer to default category -// (allows to compare for pointers instead of strings) -extern const char qtDefaultCategoryName[]; - -QT_END_NAMESPACE - -#endif // QLOGGINGCATEGORY_P_H diff --git a/src/corelib/io/qloggingregistry.cpp b/src/corelib/io/qloggingregistry.cpp index dd1612c6b4..3ce0a571b4 100644 --- a/src/corelib/io/qloggingregistry.cpp +++ b/src/corelib/io/qloggingregistry.cpp @@ -40,7 +40,6 @@ ****************************************************************************/ #include "qloggingregistry_p.h" -#include "qloggingcategory_p.h" #include #include -- cgit v1.2.3