Messages

1

System / Re: leak ram

« on: October 04, 2015, 01:01:58 am »

The solution to fix this is to simply reuse the same thread.

Bad solution. If i use a lot of draw function and other function, work with lists, in main thread, all this very loaded and becomes visible small hang. Solution was to divide all of different thread.

2

System / Re: leak ram

« on: October 02, 2015, 11:25:52 pm »

on one forume i got this answer:
Problem is that when creating in thread, creates a hidden context opengl and added to "global basked"(std::set <GlContext*> internalContext) and at destruction thread isn`t released.

3

System / Re: leak ram

« on: October 02, 2015, 11:18:37 pm »

new code without pointers
main.h

(click to show/hide)

class Main
{
private:
        sf::CircleShape shape1, shape2;
        sf::ContextSettings settings;
        sf::Clock clock;
        sf::RenderWindow window;
        sf::Event evt;
        sf::Mutex mut;
        float time;

public:
        Main();
        void PlayMain();
        void THREAD_Draw();
};

main.cpp

(click to show/hide)

#include "main.h"

Main::Main()
{
        window.create(sf::VideoMode(800, 600), "SFML works!", sf::Style::Close, settings);

        shape1.setRadius(100);
        shape2.setRadius(100);
        shape1.setFillColor(sf::Color::Green);
        shape2.setFillColor(sf::Color::Red);
        shape1.setPosition(100, 0);
        shape2.setPosition(300, 0);

        time = 0;
        PlayMain();
}

void Main::PlayMain()
{
        sf::Thread drawThread(&Main::THREAD_Draw, this);
        while (window.isOpen())
        {
                time = (float)clock.getElapsedTime().asMicroseconds() / 800;
                clock.restart();

                while (window.pollEvent(evt))
                {
                        if (evt.type == sf::Event::Closed)
                                window.close();
                }

                window.clear();
                window.setActive(false);

                drawThread.launch();
                drawThread.wait();

                window.display();
        }
}

void Main::THREAD_Draw()
{
        //mut->lock();
        window.draw(shape1);
        window.draw(shape2);
        window.setActive(false);
        //mut->unlock();
}

old problem remains

4

System / Re: leak ram

« on: October 02, 2015, 11:14:37 pm »

this code is not my, if use shared_ptr problem remains.

The idea is not to use shared pointers. Use pure objects wherever possible, not pointers.

I agree with you, but problem not in it

You talk about memory leaks and show a code that blatantly provokes them... So this is hard to believe.

If you're sure that the problem persists, then show the new code. If you write it correctly, the code won't contain a single new keyword.

By the way, to detect memory leaks, you should use a tool designed for that task, not the Windows task manager.

tool, for example?

5

System / Re: leak ram

« on: September 29, 2015, 06:47:57 pm »

And don't use threads. Whatever you think you'd gain, you think wrong.

Are you coming from a Java or C# background, because as Jesper Juhl said, you should use RAII.

then i used thread in training purposes

6

System / Re: leak ram

« on: September 29, 2015, 06:36:49 pm »

Don't use raw owning pointers. Use std::unique_ptr (or std::shared_ptr) for owning pointers. Only use raw pointers for observing non-owning pointers.
Also look up RAII.

this code is not my, if use shared_ptr problem remains. I agree with you, but problem not in it

7

System / leak ram

« on: September 29, 2015, 11:19:00 am »

sorry for my endlish...
At first it was so

(click to show/hide)

#include "SFML\Graphics.hpp"
#ifndef MAIN_H
#define MAIN_H
 
using namespace std;
 
class Main
{
private:
    sf::CircleShape* shape1, *shape2;
    sf::ContextSettings* settings;
    sf::Clock* clock;
    sf::RenderWindow* window;
    sf::Event* evt;
    sf::Mutex* mut;
    sf::Thread* drawThread;
    float time;
 
public:
    Main();
    void PlayMain();
    void THREAD_Draw();
};
#endif

(click to show/hide)

#include "main.h"
 
Main::Main()
{
    clock = new sf::Clock;
    settings = new sf::ContextSettings(16, 24, 2, 0, 8);
    window = new sf::RenderWindow(sf::VideoMode(800, 600), "SFML works!", sf::Style::Close, *settings);
    mut = new sf::Mutex;
    drawThread = new sf::Thread(&Main::THREAD_Draw, this);
    evt = new sf::Event;
 
    shape1 = new sf::CircleShape(100);
    shape2 = new sf::CircleShape(100);
 
    shape1->setFillColor(sf::Color::Green);
    shape2->setFillColor(sf::Color::Red);
    shape1->setPosition(100, 0);
    shape2->setPosition(300, 0);
 
    time = 0;
    PlayMain();
}
 
void Main::PlayMain()
{
 
    while (window->isOpen())
    {
        time = (float)clock->getElapsedTime().asMicroseconds() / 800;
        clock->restart();
 
        while (window->pollEvent(*evt))
        {
            if (evt->type == sf::Event::Closed)
                window->close();
        }
 
        window->clear();
        window->setActive(false);
 
        drawThread->launch(); drawThread->wait();
 
        window->display();
    }
}
 
void Main::THREAD_Draw()
{
    //mut->lock();
    window->draw(*shape1);
    window->draw(*shape2);
    //mut->unlock();
}

screen 1: "Failed to activate the window's context"

after correction:

void Main::THREAD_Draw()
void Main::THREAD_Draw()
{
        window->draw(*shape1);
        window->draw(*shape2);
        window->setActive(false);
}

it was like on screen 2: leak RAM