News:

When registered with our forums, feel free to send a "here I am" post here to differ human beings from SPAM bots.

Main Menu

Dependencies / ProjectManager::CloseProject

Started by Bat, March 27, 2018, 09:02:51 PM

Previous topic - Next topic

Bat

I saw from long time a crash on C::B windows, when lots of files and projects are modified outsided C::B (typically git checkout on another branch without C::B closed). I tracked it.

I land here :
bool ProjectManager::CloseProject(cbProject* project, bool dontsave, bool refresh)
We have :
    RemoveProjectFromAllDependencies(project);
But I think we need also this call, to clean the global DepsMap array that will track pointer on closed project :
    ClearProjectDependencies(project);

Note :
1)this "leak" can lead to crash only after the second reloading of a project (or any other DepsMap pointers usage), not the first one
2)in my specific case, it solved the crash

Am I missing something ?
If not did you want a ticket and a patch ?

oBFusCATed

(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

oBFusCATed

Unfortunately I've just got a crash when switching branches in git and the whole workspace had to be reloaded.
The backtrace seems similar like the fixed crash, so the problem is not 100% fixed. :(
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]