1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
|
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
/*
* This file is part of the LibreOffice project.
*
* Based on LLVM/Clang.
*
* This file is distributed under the University of Illinois Open Source
* License. See LICENSE.TXT for details.
*
*/
#ifndef LO_CLANG_SHARED_PLUGINS
#include "plugin.hxx"
#include "check.hxx"
#include "config_clang.h"
#include <fstream>
#include <unordered_set>
namespace loplugin
{
/*
Check that we are using exceptionToString when printing exceptions inside SAL_WARN, so that we
get nicely formatted exception details in our logs.
*/
class LogExceptionNicely : public loplugin::FilteringPlugin<LogExceptionNicely>
{
std::unordered_set<SourceLocation> m_visited;
public:
LogExceptionNicely(const InstantiationData& data)
: FilteringPlugin(data)
{
}
bool preRun()
{
std::string fn(handler.getMainFileName());
loplugin::normalizeDotDotInFilePath(fn);
// these are below tools in the module hierarchy, so we can't use the pretty printing
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/cppuhelper/"))
return false;
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/ucbhelper/"))
return false;
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/binaryurp/"))
return false;
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/comphelper/"))
return false;
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/io/"))
return false;
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/javaunohelper/"))
return false;
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/stoc/"))
return false;
// can't do that here, don't have an Any
if (loplugin::hasPathnamePrefix(fn, SRCDIR
"/connectivity/source/drivers/hsqldb/HStorageMap.cxx"))
return false;
return true;
}
void run()
{
if (preRun())
TraverseDecl(compiler.getASTContext().getTranslationUnitDecl());
}
static bool BaseCheckNotExceptionSubclass(const CXXRecordDecl* BaseDefinition)
{
if (!BaseDefinition)
return true;
auto tc = loplugin::TypeCheck(BaseDefinition);
if (tc.Class("Exception")
.Namespace("uno")
.Namespace("star")
.Namespace("sun")
.Namespace("com")
.GlobalNamespace())
return false;
return true;
}
bool isDerivedFromException(const CXXRecordDecl* decl)
{
if (!decl || !decl->hasDefinition())
return false;
auto tc = loplugin::TypeCheck(decl);
if (tc.Class("Exception")
.Namespace("uno")
.Namespace("star")
.Namespace("sun")
.Namespace("com")
.GlobalNamespace())
return true;
if ( // not sure what hasAnyDependentBases() does,
// but it avoids classes we don't want, e.g. WeakAggComponentImplHelper1
!decl->hasAnyDependentBases() && !decl->forallBases(BaseCheckNotExceptionSubclass))
{
return true;
}
return false;
}
bool VisitCXXOperatorCallExpr(const CXXOperatorCallExpr* operatorCallExpr)
{
if (ignoreLocation(operatorCallExpr))
return true;
StringRef fn = getFilenameOfLocation(
compiler.getSourceManager().getExpansionLoc(operatorCallExpr->getBeginLoc()));
// these are below tools in the module hierarchy, so we can't use the pretty printing
if (loplugin::hasPathnamePrefix(fn, SRCDIR "/include/comphelper/"))
return true;
if (operatorCallExpr->getOperator() != OO_LessLess)
return true;
auto expr = operatorCallExpr->getArg(1)->IgnoreImplicit();
if (auto declRefExpr = dyn_cast<DeclRefExpr>(expr))
if (auto varDecl = dyn_cast<VarDecl>(declRefExpr->getDecl()))
{
const clang::Type* type = varDecl->getType()->getUnqualifiedDesugaredType();
const CXXRecordDecl* cxxRecordDecl = type->getAsCXXRecordDecl();
if (!cxxRecordDecl)
cxxRecordDecl = type->getPointeeCXXRecordDecl();
if (!cxxRecordDecl)
return true;
if (!isDerivedFromException(cxxRecordDecl))
return true;
auto loc = operatorCallExpr->getBeginLoc();
// for some reason, I'm warning multiple times? so just check if I've warned already
if (!m_visited.insert(compiler.getSourceManager().getExpansionLoc(loc)).second)
return true;
report(DiagnosticsEngine::Warning,
"use TOOLS_WARN_EXCEPTION/TOOLS_INFO_EXCEPTION/exceptionToString to print "
"exception nicely",
loc)
<< operatorCallExpr->getSourceRange();
return true;
}
return true;
}
};
static Plugin::Registration<LogExceptionNicely> logexceptionnicely("logexceptionnicely");
} // namespace
#endif // LO_CLANG_SHARED_PLUGINS
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
|
