D9016: remove global statics
Sven Brauch
noreply at phabricator.kde.org
Mon Nov 27 12:35:28 UTC 2017
brauch created this revision.
brauch added a reviewer: flherne.
Restricted Application added a subscriber: kdevelop-devel.
REVISION SUMMARY
Having looked at https://bugs.kde.org/show_bug.cgi?id=385107 for a while, it seems to me like compilers do really strange thingsā¢ when having multiple nontrivially constructed global static variables in a threaded environment. Probably the best solution is to simply remove all the global statics, and use free functions instead. While we're at it, I turned the Helper class into a namespace, it only has static member functions anyways.
All this code needs lots of refactoring and cleanup other than this change, but we have to start somewhere ...
TEST PLAN
Tests pass. Bug 385107 is untestable because it's super rare.
REPOSITORY
R53 KDevelop: Python Support
REVISION DETAIL
https://phabricator.kde.org/D9016
AFFECTED FILES
duchain/helpers.cpp
duchain/helpers.h
projectconfig/projectconfigpage.cpp
pythonparsejob.cpp
To: brauch, flherne
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171127/b0d319f6/attachment.html>
More information about the KDevelop-devel
mailing list