# V839. Function returns a constant value. This may interfere with move semantics.

Это диагностическое правило основано на пункте F.49 CppCoreGuidelines.

Анализатор обнаружил объявление функции, которая возвращает значения константного типа. Несмотря на то, что квалификатор 'const' по задумке автора специально не позволяет модифицировать временные объекты, такое поведение может препятствовать семантике перемещения. В результате возможно замедление из-за дорогостоящего копирования больших объектов.

Рассмотрим синтетический пример:

    class Object { .... };

    const std::vector<Object> GetAllObjects() {...};

    void g(std::vector<Object> &vo)
    {
      vo = GetAllObjects();
    }

Из-за наличия квалификатора 'const' у возвращаемого типа функции 'GetAllObjects' при выборе перегрузки оператора '=' компилятор не будет рассматривать оператор перемещения и выберет оператор копирования. Собственно, будет выполнено копирования вектора объектов, возвращенного функцией.

Чтобы оптимизировать код, достаточно удалить квалификатор 'const':

    class Object { .... };

    std::vector<Object> GetAllObjects() {...};

    void g(std::vector<Object> &vo)
    {
      vo = GetAllObjects();
    }

Теперь при вызове функции 'GetAllObjects' возвращаемый вектор объектов будет перемещен.

Обращаем внимание, что диагностическое правило применяется только для кода, написанного на C++11 или выше. Это поведение связано с семантикой перемещения, которая была добавлена в C++11.

Рассмотрим следующий код (реализация пользовательского типа для длинной арифметики):

    class BigInt
    {
    private:
      ....
    public:
      BigInt& operator++();
      BigInt& operator++(int);
      BigInt& operator--();
      BigInt& operator--(int);

      friend const BigInt operator+(const BigInt &,
                                    const BigInt &) noexcept;
      // other operators
    };

    void foo(const BigInt &lhs, const BigInt &rhs)
    {
      auto obj = ++(lhs + rhs); // compile-time error
                                // can't call operator++()
                                // on const object
    }

Такой паттерн использовался в прошлом, когда семантика перемещения еще не была внедрена в C++, и при этом хотелось запретить вызывать модифицирующие операции на временном объекте.

Если возвращать из перегруженного оператора '+' не константный объект, то можно будет вызывать перегруженный оператор префиксного инкремента на временном объекте. Для встроенных арифметических типов такая семантика запрещена.

Начиная с C++11 код можно переделать при помощи ref-квалификаторов нестатических функций-членов и дать возможность компилятору применять семантику перемещения:

    class BigInt
    {
      ....
    public:
      BigInt& operator++() & noexcept;
      BigInt& operator++(int) & noexcept;
      BigInt& operator--() & noexcept;
      BigInt& operator--(int) & noexcept;

      friend BigInt operator+(const BigInt &,
                              const BigInt &) noexcept;
      // other operators
    };

    void foo(const BigInt &lhs, const BigInt &rhs)
    {
      auto obj = ++(lhs + rhs); // compile-time error
                                // can't call BigInt::operator++()
                                // on prvalue
    }