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
|
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; fill-column: 100 -*- */
/*
* This file is part of the LibreOffice project.
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
*/
#include <map>
#include <memory>
#include <rtl/ustring.hxx>
namespace test1
{
int foo(int);
void main()
{
int x = 1;
foo(x = 2);
// expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
int y = x = 1; // no warning expected
(void)y;
int z = foo(x = 1);
// expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
(void)z;
switch (x = 1)
{ // expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
}
std::map<int, int> map1;
map1[x = 1] = 1;
// expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
}
}
namespace test2
{
struct MyInt
{
int x;
MyInt(int i)
: x(i)
{
}
MyInt(MyInt const&) = default;
MyInt& operator=(MyInt const&) = default;
MyInt& operator=(int) { return *this; }
bool operator<(MyInt const& other) const { return x < other.x; }
};
MyInt foo(MyInt);
void main()
{
MyInt x = 1;
foo(x = 2);
// expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
MyInt y = x = 1; // no warning expected
(void)y;
MyInt z = foo(x = 1);
// expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
(void)z;
z = x; // no warning expected
std::map<MyInt, int> map1;
map1[x = 1] = 1;
// expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
}
}
namespace test3
{
void main(OUString sUserAutoCorrFile, OUString sExt)
{
OUString sRet;
if (sUserAutoCorrFile == "xxx")
sRet = sUserAutoCorrFile; // no warning expected
if (sUserAutoCorrFile == "yyy")
(sRet = sUserAutoCorrFile)
+= sExt; // expected-error@-1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
}
}
// no warning expected
namespace test4
{
struct Task
{
void exec();
};
std::unique_ptr<Task> pop();
void foo()
{
std::unique_ptr<Task> pTask;
while ((pTask = pop()))
pTask->exec();
}
}
namespace test5
{
void main(OUString sUserAutoCorrFile, int* p2)
{
OUString sRet;
int* p1;
if (sUserAutoCorrFile == "yyy" && (p1 = p2))
sRet = sUserAutoCorrFile;
if (sUserAutoCorrFile == "yyy" && nullptr != (p1 = p2))
sRet = sUserAutoCorrFile;
// expected-error@+1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
if (nullptr != (p1 = p2))
sRet = sUserAutoCorrFile;
// expected-error@+1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
if ((p1 = p2) != nullptr)
sRet = sUserAutoCorrFile;
// expected-error@+1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
if ((p1 = p2))
sRet = sUserAutoCorrFile;
// expected-error@+1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
if ((p1 = p2) && sUserAutoCorrFile == "yyy")
sRet = sUserAutoCorrFile;
// expected-error@+1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
if ((p1 = p2) || sUserAutoCorrFile == "yyy")
sRet = sUserAutoCorrFile;
// expected-error@+1 {{buried assignment, rather put on own line [loplugin:buriedassign]}}
if ((p1 = p2) && sUserAutoCorrFile == "yyy")
sRet = sUserAutoCorrFile;
(void)sRet;
}
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */
|
