File: redundant-function-ptr-dereference.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 (45 lines) | stat: -rw-r--r-- 1,083 bytes parent folder | download | duplicates (27) 
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
 
// RUN: %check_clang_tidy %s readability-redundant-function-ptr-dereference %t

void f(int i);

void positive() {
  void (*p)(int) = f;

  (**p)(1);
  // CHECK-MESSAGES: :[[@LINE-1]]:4: warning: redundant repeated dereference of function pointer [readability-redundant-function-ptr-dereference]
  // CHECK-FIXES: (*p)(1);
  (*****p)(2);
  // CHECK-MESSAGES: :[[@LINE-1]]:4: warning: redundant repeated
  // CHECK-MESSAGES: :[[@LINE-2]]:5: warning: redundant repeated
  // CHECK-MESSAGES: :[[@LINE-3]]:6: warning: redundant repeated
  // CHECK-MESSAGES: :[[@LINE-4]]:7: warning: redundant repeated
  // CHECK-FIXES: (*p)(2);
}

template<typename T>
void invoke(const T& fn) {
  fn(0); // 1
  (*fn)(0); // 2
  // CHECK-MESSAGES: :[[@LINE-1]]:4: warning: redundant repeated
  // CHECK-FIXES: fn(0); // 1
  // CHECK-FIXES: (fn)(0); // 2
  // FIXME: Remove unnecessary parentheses.
}

void f1(int);
void f2(double);
void f3(char);

void instantiate() {
  invoke(f1);
  invoke(f2);
  invoke(f3);
  invoke([](unsigned) {});
}

void negative() {
  void (*q)(int) = &f;

  q(1);
  (*q)(2);
}