File: unique-ptr-array-mismatch.rst

package info (click to toggle)
swiftlang 6.0.3-2
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid, trixie
  • size: 2,519,992 kB
  • sloc: cpp: 9,107,863; ansic: 2,040,022; asm: 1,135,751; python: 296,500; objc: 82,456; f90: 60,502; lisp: 34,951; pascal: 19,946; sh: 18,133; perl: 7,482; ml: 4,937; javascript: 4,117; makefile: 3,840; awk: 3,535; xml: 914; fortran: 619; cs: 573; ruby: 573
file content (39 lines) | stat: -rw-r--r-- 1,751 bytes parent folder | download | duplicates (12)
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
.. title:: clang-tidy - bugprone-unique-ptr-array-mismatch

bugprone-unique-ptr-array-mismatch
==================================

Finds initializations of C++ unique pointers to non-array type that are
initialized with an array.

If a pointer ``std::unique_ptr<T>`` is initialized with a new-expression
``new T[]`` the memory is not deallocated correctly. A plain ``delete`` is used
in this case to deallocate the target memory. Instead a ``delete[]`` call is
needed. A ``std::unique_ptr<T[]>`` uses the correct delete operator. The check
does not emit warning if an ``unique_ptr`` with user-specified deleter type is
used.

The check offers replacement of ``unique_ptr<T>`` to ``unique_ptr<T[]>`` if it
is used at a single variable declaration (one variable in one statement).

Example:

.. code-block:: c++

  std::unique_ptr<Foo> x(new Foo[10]); // -> std::unique_ptr<Foo[]> x(new Foo[10]);
  //                     ^ warning: unique pointer to non-array is initialized with array
  std::unique_ptr<Foo> x1(new Foo), x2(new Foo[10]); // no replacement
  //                                   ^ warning: unique pointer to non-array is initialized with array

  D d;
  std::unique_ptr<Foo, D> x3(new Foo[10], d); // no warning (custom deleter used)

  struct S {
    std::unique_ptr<Foo> x(new Foo[10]); // no replacement in this case
    //                     ^ warning: unique pointer to non-array is initialized with array
  };

This check partially covers the CERT C++ Coding Standard rule
`MEM51-CPP. Properly deallocate dynamically allocated resources
<https://wiki.sei.cmu.edu/confluence/display/cplusplus/MEM51-CPP.+Properly+deallocate+dynamically+allocated+resources>`_
However, only the ``std::unique_ptr`` case is detected by this check.