File: cppcoreguidelines-pro-bounds-array-to-pointer-decay.cpp

package info (click to toggle)
llvm-toolchain-11 1%3A11.0.1-2
  • links: PTS, VCS
  • area: main
  • in suites: bullseye
  • size: 995,808 kB
  • sloc: cpp: 4,767,656; ansic: 760,916; asm: 477,436; python: 170,940; objc: 69,804; lisp: 29,914; sh: 23,855; f90: 18,173; pascal: 7,551; perl: 7,471; ml: 5,603; awk: 3,489; makefile: 2,573; xml: 915; cs: 573; fortran: 503; javascript: 452
file content (51 lines) | stat: -rw-r--r-- 1,404 bytes parent folder | download | duplicates (7)
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
// RUN: %check_clang_tidy %s cppcoreguidelines-pro-bounds-array-to-pointer-decay %t
#include <stddef.h>

namespace gsl {
template <class T>
class array_view {
public:
  template <class U, size_t N>
  array_view(U (&arr)[N]);
};
}

void pointerfun(int *p);
void arrayfun(int p[]);
void arrayviewfun(gsl::array_view<int> &p);
size_t s();

void f() {
  int a[5];
  pointerfun(a);
  // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay]
  pointerfun((int *)a); // OK, explicit cast
  arrayfun(a);
  // CHECK-MESSAGES: :[[@LINE-1]]:12: warning: do not implicitly decay an array into a pointer

  pointerfun(a + s() - 10); // Convert to &a[g() - 10];
  // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: do not implicitly decay an array into a pointer

  gsl::array_view<int> av(a);
  arrayviewfun(av); // OK

  int i = a[0];      // OK
  int j = a[(1 + 2)];// OK
  pointerfun(&a[0]); // OK

  for (auto &e : a) // OK, iteration internally decays array to pointer
    e = 1;
}

const char *g() {
  return "clang"; // OK, decay string literal to pointer
}
const char *g2() {
    return ("clang"); // OK, ParenExpr hides the literal-pointer decay
}

void f2(void *const *);
void bug25362() {
  void *a[2];
  f2(static_cast<void *const*>(a)); // OK, explicit cast
}