From b68a06799c6b3ab753c2a5c40d60de37c3dad7db Mon Sep 17 00:00:00 2001 From: hjk <qtc-committer@nokia.com> Date: Thu, 30 Apr 2009 12:16:48 +0200 Subject: [PATCH] debugger: code cosmetics --- src/plugins/debugger/cdb/cdbcom.h | 1 + src/plugins/debugger/cdb/cdbdumperhelper.h | 4 ++-- src/plugins/debugger/cdb/cdboptions.cpp | 1 + src/plugins/debugger/cdb/cdbstacktracecontext.h | 2 +- src/plugins/debugger/cdb/cdbsymbolgroupcontext.cpp | 6 +++++- src/plugins/debugger/cdb/cdbsymbolgroupcontext.h | 3 ++- src/plugins/debugger/win/dbgwinutils.cpp | 5 +++-- src/plugins/debugger/win/dbgwinutils.h | 11 ++++++----- src/plugins/debugger/win/peutils.cpp | 6 +++--- src/plugins/debugger/win/peutils.h | 10 +++++----- 10 files changed, 29 insertions(+), 20 deletions(-) diff --git a/src/plugins/debugger/cdb/cdbcom.h b/src/plugins/debugger/cdb/cdbcom.h index 8a631e3f1fc..19208710a9f 100644 --- a/src/plugins/debugger/cdb/cdbcom.h +++ b/src/plugins/debugger/cdb/cdbcom.h @@ -43,4 +43,5 @@ typedef IDebugDataSpaces4 CIDebugDataSpaces; typedef IDebugSymbolGroup2 CIDebugSymbolGroup; typedef IDebugBreakpoint2 CIDebugBreakpoint; + #endif // CDBCOM_H diff --git a/src/plugins/debugger/cdb/cdbdumperhelper.h b/src/plugins/debugger/cdb/cdbdumperhelper.h index b9ec2bccccb..9500d4d4e01 100644 --- a/src/plugins/debugger/cdb/cdbdumperhelper.h +++ b/src/plugins/debugger/cdb/cdbdumperhelper.h @@ -32,6 +32,7 @@ #include "watchutils.h" #include "cdbcom.h" + #include <QtCore/QStringList> #include <QtCore/QMap> @@ -42,8 +43,7 @@ struct CdbComInterfaces; class IDebuggerManagerAccessForEngines; class DebuggerManager; -/* For code clarity, all the stuff related to custom dumpers - * goes here. +/* For code clarity, all the stuff related to custom dumpers goes here. * "Custom dumper" is a library compiled against the current * Qt containing functions to evaluate values of Qt classes * (such as QString, taking pointers to their addresses). diff --git a/src/plugins/debugger/cdb/cdboptions.cpp b/src/plugins/debugger/cdb/cdboptions.cpp index b26053beee8..1740c3a6d7b 100644 --- a/src/plugins/debugger/cdb/cdboptions.cpp +++ b/src/plugins/debugger/cdb/cdboptions.cpp @@ -28,6 +28,7 @@ **************************************************************************/ #include "cdboptions.h" + #include <QtCore/QSettings> #include <QtCore/QDir> #include <QtCore/QFileInfo> diff --git a/src/plugins/debugger/cdb/cdbstacktracecontext.h b/src/plugins/debugger/cdb/cdbstacktracecontext.h index 98e509ad982..b5e041d2c2f 100644 --- a/src/plugins/debugger/cdb/cdbstacktracecontext.h +++ b/src/plugins/debugger/cdb/cdbstacktracecontext.h @@ -73,7 +73,7 @@ public: // Top-Level instruction offset for disassembler ULONG64 instructionOffset() const { return m_instructionOffset; } - CdbStackFrameContext*frameContextAt(int index, QString *errorMessage); + CdbStackFrameContext *frameContextAt(int index, QString *errorMessage); // Format for logging void format(QTextStream &str) const; diff --git a/src/plugins/debugger/cdb/cdbsymbolgroupcontext.cpp b/src/plugins/debugger/cdb/cdbsymbolgroupcontext.cpp index 7daba56a587..47035c5ec4d 100644 --- a/src/plugins/debugger/cdb/cdbsymbolgroupcontext.cpp +++ b/src/plugins/debugger/cdb/cdbsymbolgroupcontext.cpp @@ -31,6 +31,7 @@ #include "cdbdebugengine_p.h" #include "watchhandler.h" #include "watchutils.h" + #include <QtCore/QTextStream> enum { debug = 0 }; @@ -40,7 +41,10 @@ static inline QString msgSymbolNotFound(const QString &s) return QString::fromLatin1("The symbol '%1' could not be found.").arg(s); } -static inline bool isTopLevelSymbol(const DEBUG_SYMBOL_PARAMETERS &p) { return p.ParentSymbol == DEBUG_ANY_ID; } +static inline bool isTopLevelSymbol(const DEBUG_SYMBOL_PARAMETERS &p) +{ + return p.ParentSymbol == DEBUG_ANY_ID; +} static inline void debugSymbolFlags(unsigned long f, QTextStream &str) { diff --git a/src/plugins/debugger/cdb/cdbsymbolgroupcontext.h b/src/plugins/debugger/cdb/cdbsymbolgroupcontext.h index 033252e387c..ea992e546cf 100644 --- a/src/plugins/debugger/cdb/cdbsymbolgroupcontext.h +++ b/src/plugins/debugger/cdb/cdbsymbolgroupcontext.h @@ -130,7 +130,8 @@ private: }; // Helper to a sequence of WatchData into a list. -class WatchDataBackInserter { +class WatchDataBackInserter +{ public: explicit WatchDataBackInserter(QList<WatchData> &wh) : m_wh(wh) {} diff --git a/src/plugins/debugger/win/dbgwinutils.cpp b/src/plugins/debugger/win/dbgwinutils.cpp index 2145f001ad2..9ef66aa1ad9 100644 --- a/src/plugins/debugger/win/dbgwinutils.cpp +++ b/src/plugins/debugger/win/dbgwinutils.cpp @@ -34,6 +34,7 @@ #ifdef USE_PSAPI # include <psapi.h> #endif + namespace Debugger { namespace Internal { @@ -75,5 +76,5 @@ QList<ProcData> winProcessList() return rc; } -} -} +} // namespace Internal +} // namespace Debugger diff --git a/src/plugins/debugger/win/dbgwinutils.h b/src/plugins/debugger/win/dbgwinutils.h index 1045e822b42..af16913a8ed 100644 --- a/src/plugins/debugger/win/dbgwinutils.h +++ b/src/plugins/debugger/win/dbgwinutils.h @@ -27,8 +27,8 @@ ** **************************************************************************/ -#ifndef _DBG_WINDUTILS_H -#define _DBG_WINDUTILS_H +#ifndef DEBUGGER_DBG_WINUTILS_H +#define DEBUGGER_DBG_WINUTILS_H #include <QtCore/QList> @@ -39,6 +39,7 @@ struct ProcData; // debuggerdialogs, used by the process listing dialogs QList<ProcData> winProcessList(); -} -} -#endif +} // namespace Internal +} // namespace Debugger + +#endif // DEBUGGER_DBG_WINUTILS_H diff --git a/src/plugins/debugger/win/peutils.cpp b/src/plugins/debugger/win/peutils.cpp index 23cabb8bc3b..084e511f468 100644 --- a/src/plugins/debugger/win/peutils.cpp +++ b/src/plugins/debugger/win/peutils.cpp @@ -40,7 +40,7 @@ using Core::Utils::winErrorMessage; // a memory mapped file template <class Ptr> - inline Ptr *makePtr(void *base, ptrdiff_t offset) +inline Ptr *makePtr(void *base, ptrdiff_t offset) { return reinterpret_cast<Ptr*>(static_cast<char*>(base) + offset); } @@ -275,5 +275,5 @@ bool getPDBFiles(const QString &peExecutableFileName, QStringList *rc, QString * return success; } -} -} +} // namespace Internal +} // namespace Debugger diff --git a/src/plugins/debugger/win/peutils.h b/src/plugins/debugger/win/peutils.h index c9885991be9..7b4b0061fc9 100644 --- a/src/plugins/debugger/win/peutils.h +++ b/src/plugins/debugger/win/peutils.h @@ -27,8 +27,8 @@ ** **************************************************************************/ -#ifndef PEUTILS_H -#define PEUTILS_H +#ifndef DEBUGGER_PEUTILS_H +#define DEBUGGER_PEUTILS_H #include <QtCore/qnamespace.h> @@ -45,7 +45,7 @@ namespace Internal { // Return a list of Program-Database (*.pdb) files a PE executable refers to. */ bool getPDBFiles(const QString &peExecutableFileName, QStringList *rc, QString *errorMessage); -} -} +} // namespace Internal +} // namespace Debugger -#endif // PEUTILS_H +#endif // DEBUGGER_PEUTILS_H -- GitLab