Утечка памяти при динамическом выделении

QSGGeometryNode *NavRect::lineNode(QPoint beginPoint , QPoint endPoint, QColor colorLine, QSGGeometryNode *oldNode)
{
    auto material = new QSGFlatColorMaterial;
    material->setColor(colorLine);

    if(!oldNode)
    {
        auto pNode = new QSGGeometryNode;
        auto pLineGeometry=new QSGGeometry(QSGGeometry::defaultAttributes_Point2D(), 2);
        pLineGeometry->setLineWidth(1);
        pLineGeometry->setDrawingMode(GL_LINES);

        pLineGeometry->vertexDataAsPoint2D()[0].set(beginPoint.x(), beginPoint.y());
        pLineGeometry->vertexDataAsPoint2D()[1].set(endPoint.x(), endPoint.y());

        pNode->setGeometry(pLineGeometry);
        pNode->setFlags(QSGNode::OwnsGeometry | QSGNode::OwnsMaterial);
        pNode->setMaterial(material);

        return pNode;
    }
    else
    {
        oldNode->setMaterial(material);
    }

    return oldNode;
}

Рекомендуем хостинг TIMEWEB

Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Привет.

Здесь не надо ничего такого делать, память здесь будет очищаться за счёт этой строчки

pNode->setFlags(QSGNode::OwnsGeometry | QSGNode::OwnsMaterial);

То есть ты берёшь владение материалом и геометрией на pNode, а сама pNode должна будет уничтожаться дальше в дереве объектов OpenGL, попутно уничтожая материал и геометрию. QML всё это предусматривает.

Если по какой-то причине всё-таки происходит утечка памяти, то значит косяк где-то в логике метода updatePaintNode

Для проверки можно сделать для всех этих классов наследников и переопределить деструктор с выводом qDebug, как сделано в этой статье для QStandardItem . Это позволит убедиться, что удаление всё-таки происходит.

Оки спасибо, сейчас проверю.

В общем все таки утечка есть, по причине логики метода где вызывается данная функция.
Вот так она выглядит

QSGNode *NavRect::updatePaintNode(QSGNode *oldNode, QQuickItem::UpdatePaintNodeData *updatePaintNodeData)
{
    Q_UNUSED(updatePaintNodeData)

    QColor foreground = Qt::white;
    QColor background = Qt::black;

    auto width = boundingRect().width();
    auto height = boundingRect().height();

    if(hasFocus())
    {
        background = Qt::white;
        foreground = Qt::black;
    }

    setProperty("foreground", foreground);
    setProperty("background", background);


    if(!oldNode)
    {
        //! complete creation of objects and attaching them to a node, background color
        m_pBackgroundNode = window()->createRectangleNode();
        m_pBackgroundNode->setRect(boundingRect());
        m_pBackgroundNode->setColor(background);

        //! draw lines
        m_pNodeLineOne = lineNode(QPoint(0, 0), QPoint(width, 0), foreground);
        m_pNodeLineTwo = lineNode(QPoint(width, 0), QPoint(width, height), foreground);
        m_pNodeLineThree = lineNode(QPoint(width, height), QPoint(0, height), foreground);
        m_pNodeLineFour = lineNode(QPoint(0, height), QPoint(0, 0), foreground);

        m_pBackgroundNode->appendChildNode(m_pNodeLineOne);
        m_pBackgroundNode->appendChildNode(m_pNodeLineTwo);
        m_pBackgroundNode->appendChildNode(m_pNodeLineThree);
        m_pBackgroundNode->appendChildNode(m_pNodeLineFour);

        return m_pBackgroundNode;
    }

    //! if a node exists, update the background
    m_pBackgroundNode->setColor(background);

    //! updating lines
    m_pNodeLineOne = lineNode(QPoint(0, 0), QPoint(width, 0), foreground, m_pNodeLineOne);
    m_pNodeLineTwo = lineNode(QPoint(width, 0), QPoint(width, height), foreground, m_pNodeLineTwo);
    m_pNodeLineThree = lineNode(QPoint(width, height), QPoint(0, height), foreground, m_pNodeLineThree);
    m_pNodeLineFour = lineNode(QPoint(0, height), QPoint(0, 0), foreground, m_pNodeLineFour);

    return oldNode;
}

То есть при первом входе создаюется обьект и вызывается lineNode для создания линий, а потом когда уже нода существует то мы снова заходим в метод и опять делаем new.
Так что вот с этим и проблема как раз.

Проблема в том, что если нода уже была создана, то ты заново пересоздаешь ноды, а не перерисовываешь. Посмотри как сделано в этой статье QML - Урок 032. Создаём Custom QuickItem из C++ с использованием средств OpenGL

Я не создаю там новые объекты, если oldNode существует, я перерисовываю ноду в уже существющих объектах. А ты просто создаешь каждый раз новые объекты m_pNodeLineOne, m_pNodeLineTwo, m_pNodeLineThree, m_pNodeLineFour если oldNode . Либо удаляй их, если они существуют и потом создавай новые, либо переопределяй некоторые свойства. Переопределение наверняка будет производительнее, чем создавать новые объекты.

В общем тебе нужен ещё один метод, который будет менять точки, то есть менять геометрию.

хотя, если вниметльно посмотреть на метод lineNode, то скорее всего проблема в том, что ты создаешь материал в начале метода. Старый скорее всего не удаляется, пока не была удалена нода, а просто заменяется, и остаётся висеть в памяти.

В общем я просто сделал вторйо метод который простог обнавляет цвет и материал, вроде сделал правильно.
Но вот на эмбедеде есл ия быстро переключаюсь межуд обьектами они начали как бы мерцать, у меня предположение что я что то еще должен добавить в метод но не пойму что.

QSGGeometryNode *NavRect::nonExistLineNode(QPoint beginPoint , QPoint endPoint, QColor colorLine, QSGGeometryNode *oldNode)
{
    m_pMaterial = new QSGFlatColorMaterial;
    m_pMaterial->setColor(colorLine);

    if(!oldNode)
    {
        m_pGeometryNode = new QSGGeometryNode;
        m_pGeometry = new QSGGeometry(QSGGeometry::defaultAttributes_Point2D(), 2);
        m_pGeometry->setLineWidth(1);
        m_pGeometry->setDrawingMode(GL_LINES);

        m_pGeometry->vertexDataAsPoint2D()[0].set(beginPoint.x(), beginPoint.y());
        m_pGeometry->vertexDataAsPoint2D()[1].set(endPoint.x(), endPoint.y());

        m_pGeometryNode->setGeometry(m_pGeometry);
        m_pGeometryNode->setFlags(QSGNode::OwnsGeometry | QSGNode::OwnsMaterial);
        m_pGeometryNode->setMaterial(m_pMaterial);

        return m_pGeometryNode;
    }
    else
    {
        oldNode->setMaterial(m_pMaterial);
    }

    return oldNode;
}

QSGGeometryNode *NavRect::existLineNode(QPoint beginPoint, QPoint endPoint, QColor colorLine, QSGGeometryNode *oldNode)
{
    m_pMaterial->setColor(colorLine);

    oldNode->setMaterial(m_pMaterial);

    return oldNode;
}

Само собою на компе этого незаметно.

Попробуй не переустанавливать материал, а просто менять цвет