File: inefficient-string-concatenation.cpp

package info (click to toggle)
llvm-toolchain-17 1%3A17.0.6-22
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid, trixie
  • size: 1,799,624 kB
  • sloc: cpp: 6,428,607; ansic: 1,383,196; asm: 793,408; python: 223,504; objc: 75,364; f90: 60,502; lisp: 33,869; pascal: 15,282; sh: 9,684; perl: 7,453; ml: 4,937; awk: 3,523; makefile: 2,889; javascript: 2,149; xml: 888; fortran: 619; cs: 573
file content (48 lines) | stat: -rw-r--r-- 1,634 bytes parent folder | download | duplicates (24)
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
// RUN: %check_clang_tidy %s performance-inefficient-string-concatenation %t

namespace std {
template <typename T>
class basic_string {
public:
  basic_string() {}
  ~basic_string() {}
  basic_string<T> *operator+=(const basic_string<T> &) {}
  friend basic_string<T> operator+(const basic_string<T> &, const basic_string<T> &) {}
};
typedef basic_string<char> string;
typedef basic_string<wchar_t> wstring;
}

void f(std::string) {}
std::string g(std::string) {}

int main() {
  std::string mystr1, mystr2;
  std::wstring mywstr1, mywstr2;
  auto myautostr1 = mystr1;
  auto myautostr2 = mystr2;

  for (int i = 0; i < 10; ++i) {
    f(mystr1 + mystr2 + mystr1);
    // CHECK-MESSAGES: :[[@LINE-1]]:23: warning: string concatenation results in allocation of unnecessary temporary strings; consider using 'operator+=' or 'string::append()' instead
    mystr1 = mystr1 + mystr2;
    // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: string concatenation
    mystr1 = mystr2 + mystr2 + mystr2;
    // CHECK-MESSAGES: :[[@LINE-1]]:30: warning: string concatenation
    mystr1 = mystr2 + mystr1;
    // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: string concatenation
    mywstr1 = mywstr2 + mywstr1;
    // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: string concatenation
    mywstr1 = mywstr2 + mywstr2 + mywstr2;
    // CHECK-MESSAGES: :[[@LINE-1]]:33: warning: string concatenation
    myautostr1 = myautostr1 + myautostr2;
    // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: string concatenation

    mywstr1 = mywstr2 + mywstr2;
    mystr1 = mystr2 + mystr2;
    mystr1 += mystr2;
    f(mystr2 + mystr1);
    mystr1 = g(mystr1);
  }
  return 0;
}