travis仅构建失败:使用已删除的功能‘std :: unique_ptr <_Tp,_Dp> :: unique_ptr(const std :: unique_ptr <_Tp,_Dp>&)

时间:2019-12-02 13:59:50

标签: c++11 unique-ptr

该代码在本地编译时没有问题(ubuntu 19.04,gcc-8.3.0),但是在travis构建过程中失败(ubuntu 18,gcc-8.3.0)。

错误

In file included from /usr/include/c++/7/memory:64:0,
                 from ./src/Game.hpp:5,
                 from ./src/main.cpp:3:
/usr/include/c++/7/bits/stl_construct.h: In instantiation of ‘void std::_Construct(_T1*, _Args&& ...) [with _T1 = std::unique_ptr<Cell>; _Args = {const std::unique_ptr<Cell, std::default_delete<Cell> >&}]’:
/usr/include/c++/7/bits/stl_uninitialized.h:83:18:   required from ‘static _ForwardIterator std::__uninitialized_copy<_TrivialValueTypes>::__uninit_copy(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = __gnu_cxx::__normal_iterator<const std::unique_ptr<Cell>*, std::vector<std::unique_ptr<Cell> > >; _ForwardIterator = std::unique_ptr<Cell>*; bool _TrivialValueTypes = false]’
/usr/include/c++/7/bits/stl_uninitialized.h:134:15:   required from ‘_ForwardIterator std::uninitialized_copy(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = __gnu_cxx::__normal_iterator<const std::unique_ptr<Cell>*, std::vector<std::unique_ptr<Cell> > >; _ForwardIterator = std::unique_ptr<Cell>*]’
/usr/include/c++/7/bits/stl_uninitialized.h:289:37:   required from ‘_ForwardIterator std::__uninitialized_copy_a(_InputIterator, _InputIterator, _ForwardIterator, std::allocator<_Tp>&) [with _InputIterator = __gnu_cxx::__normal_iterator<const std::unique_ptr<Cell>*, std::vector<std::unique_ptr<Cell> > >; _ForwardIterator = std::unique_ptr<Cell>*; _Tp = std::unique_ptr<Cell>]’
/usr/include/c++/7/bits/stl_vector.h:331:31:   required from ‘std::vector<_Tp, _Alloc>::vector(const std::vector<_Tp, _Alloc>&) [with _Tp = std::unique_ptr<Cell>; _Alloc = std::allocator<std::unique_ptr<Cell> >]’
/usr/include/c++/7/bits/stl_construct.h:75:7:   required from ‘void std::_Construct(_T1*, _Args&& ...) [with _T1 = std::vector<std::unique_ptr<Cell> >; _Args = {const std::vector<std::unique_ptr<Cell, std::default_delete<Cell> >, std::allocator<std::unique_ptr<Cell, std::default_delete<Cell> > > >&}]’
/usr/include/c++/7/bits/stl_uninitialized.h:83:18:   required from ‘static _ForwardIterator std::__uninitialized_copy<_TrivialValueTypes>::__uninit_copy(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = __gnu_cxx::__normal_iterator<const std::vector<std::unique_ptr<Cell> >*, std::vector<std::vector<std::unique_ptr<Cell> > > >; _ForwardIterator = std::vector<std::unique_ptr<Cell> >*; bool _TrivialValueTypes = false]’
/usr/include/c++/7/bits/stl_uninitialized.h:134:15:   required from ‘_ForwardIterator std::uninitialized_copy(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = __gnu_cxx::__normal_iterator<const std::vector<std::unique_ptr<Cell> >*, std::vector<std::vector<std::unique_ptr<Cell> > > >; _ForwardIterator = std::vector<std::unique_ptr<Cell> >*]’
/usr/include/c++/7/bits/stl_uninitialized.h:289:37:   required from ‘_ForwardIterator std::__uninitialized_copy_a(_InputIterator, _InputIterator, _ForwardIterator, std::allocator<_Tp>&) [with _InputIterator = __gnu_cxx::__normal_iterator<const std::vector<std::unique_ptr<Cell> >*, std::vector<std::vector<std::unique_ptr<Cell> > > >; _ForwardIterator = std::vector<std::unique_ptr<Cell> >*; _Tp = std::vector<std::unique_ptr<Cell> >]’
/usr/include/c++/7/bits/stl_vector.h:331:31:   required from ‘std::vector<_Tp, _Alloc>::vector(const std::vector<_Tp, _Alloc>&) [with _Tp = std::vector<std::unique_ptr<Cell> >; _Alloc = std::allocator<std::vector<std::unique_ptr<Cell> > >]’
./src/Systems/Physics/Grid.hpp:9:7:   required from here
/usr/include/c++/7/bits/stl_construct.h:75:7: error: use of deleted function ‘std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = Cell; _Dp = std::default_delete<Cell>]’
     { ::new(static_cast<void*>(__p)) _T1(std::forward<_Args>(__args)...); }
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/c++/7/memory:80:0,
                 from ./src/Game.hpp:5,
                 from ./src/main.cpp:3:
/usr/include/c++/7/bits/unique_ptr.h:388:7: note: declared here
       unique_ptr(const unique_ptr&) = delete;

构造unique_ptr<Cell>的唯一地方是这里

Grid::Grid(float gridLength, float halfWidth) : \
            gridLength(gridLength),
            cellsInRow(ceilf(gridLength / (2 * halfWidth))), 
            halfWidth(halfWidth),
            collisionDetector()
{
    this->cells.resize(this->cellsInRow);
    // cell insertion
    for (int row = 0; row < this->cells.size() ; row++)
    {
        this->cells[row].reserve(this->cellsInRow);
        for (int col = 0; col < this->cellsInRow; col++)
        {
            float x = col * 2 * halfWidth + halfWidth;
            float z = row * 2 * halfWidth + halfWidth;
            this->cells[row].push_back(std::make_unique<Cell>(glm::vec3(x, 0.f, z), halfWidth, row, col));
        }
    }
}

这段代码中的某处似乎调用了unique_ptr的副本构造函数,但我不知道在哪里。

编辑:

Grid.hpp

#pragma once

#include <memory>
#include <vector>
#include "Cell.hpp"
#include "Collider.hpp"
#include "CollisionDetector.hpp"

class Grid
{
    public:

        /**
        Divides the 3d space into a 2d grid
         */
        Grid(float gridLength, float halfWidth);
        /** 
        Insert a Collider collider
         */
        void Insert(std::shared_ptr<Collider>   object);

        /** 
        Deletes an object from the grid.
         */
        void Remove(std::shared_ptr<Collider>   object);

        /**
        Performs a collision check on all the cells and generates contact data.
         */
        std::vector<std::shared_ptr<Collision>> CheckCollisions();

        std::vector<std::shared_ptr<Collision>> CheckCells(int rowA, int colA, int rowB, int colB);

        // ===============
        // Utility methods
        // ===============

        /** 
        Returns a vector of pairs denoting the cells to be checked for collision detection
        */
        std::vector<std::pair<int, int>> GetEligibleCells(int cellRow, int cellCol);

        /** 
        GetInsertRow returns the row of the cell that the object needs to get inserted into
         */
        int  GetInsertRow(glm::vec3 point);

        /** 
        GetInsertCol returns the column of the cell where the object should be inserted.
         */
        int  GetInsertCol(glm::vec3 point);

        const std::vector< std::vector< std::unique_ptr<Cell>>>& GetCells();

    private:

        int     cellsInRow;
        float   halfWidth;
        float   gridLength;
        CollisionDetector collisionDetector;
        std::vector< std::vector< std::unique_ptr<Cell> > > cells;
};

网格仅由PhysicsSystem

使用
PhysicsSystem::PhysicsSystem(float gridLength, float cellHalfWidth) : grid(gridLength, cellHalfWidth)
{
    this->primaryBitset = ComponentType::Physics | ComponentType::Transform;
}

物理系统仅在此处初始化

Game::Game(int width, int height) :
    width(width), height(height) , 
    physicsSystem(70.f, 5.f)
{
   // stuff
}

0 个答案:

没有答案