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

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

Stabiles Hosting des sozialen Netzwerks EVILEG. Wir empfehlen VDS-Hosting für Django-Projekte.

Привет.

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

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;
}

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

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