Mir

Merge lp:~albaguirre/mir/fix-1599867 into lp:mir

Proposed by Alberto Aguirre
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: no longer in the source branch.
Merged at revision: 3581
Proposed branch: lp:~albaguirre/mir/fix-1599867
Merge into: lp:mir
Diff against target: 23 lines (+1/-11)
1 file modified
src/client/error_connections.cpp (+1/-11)
To merge this branch: bzr merge lp:~albaguirre/mir/fix-1599867
Reviewer Review Type Date Requested Status
Mir CI Bot continuous-integration Approve
Cemil Azizoglu (community) Approve
Kevin DuBois (community) Approve
Review via email: mp+299426@code.launchpad.net

Commit message

Avoid using std::call_once to work around possible TLS collisions in some android devices.

To post a comment you must log in.
Revision history for this message
Kevin DuBois (kdub) wrote :

lgtm

review: Approve
Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

ok

review: Approve
Revision history for this message
Mir CI Bot (mir-ci-bot) wrote :

PASSED: Continuous integration, rev:3581
https://mir-jenkins.ubuntu.com/job/mir-ci/1244/
Executed test runs:
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-mir/1448
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1500
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1491
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/1491
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1462
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1462/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1462
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1462/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1462
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1462/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1462
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1462/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1462
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1462/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://mir-jenkins.ubuntu.com/job/mir-ci/1244/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/client/error_connections.cpp'
2--- src/client/error_connections.cpp 2014-03-31 14:36:08 +0000
3+++ src/client/error_connections.cpp 2016-07-07 13:51:44 +0000
4@@ -20,19 +20,9 @@
5
6 namespace mcl = mir::client;
7
8-namespace
9-{
10-std::once_flag error_connections_init;
11-}
12-
13 mcl::ErrorConnections& mcl::ErrorConnections::instance()
14 {
15- static ErrorConnections* error_connections = nullptr;
16-
17- std::call_once(
18- error_connections_init,
19- [&] { error_connections = new ErrorConnections();});
20-
21+ static ErrorConnections* error_connections = new ErrorConnections();
22 return *error_connections;
23 }
24

Subscribers

People subscribed via source and target branches