dlr-gtlab / gt-logging

Basic C++14 logging library based on QsLog
1 stars 1 forks source link

Resolve "Missing mutex lockers when adding or removing a destination may result in UB" - [merged] #106

Closed rainman110 closed 11 months ago

rainman110 commented 1 year ago

In GitLab by @mariusalexander on May 5, 2023, 15:12

Merges 44-missing-mutex-lockers-when-adding-or-removing-a-destination-may-result-in-ub -> master

Closes #44

I have added missing mutex lockers to the removeDestination and destination method of the ´gt::log::Logger` class

rainman110 commented 1 year ago

In GitLab by @mariusalexander on May 5, 2023, 15:15

added 2 commits

Compare with previous version

rainman110 commented 1 year ago

In GitLab by @mariusalexander on May 5, 2023, 15:15

marked this merge request as ready

rainman110 commented 1 year ago

In GitLab by @mariusalexander on May 5, 2023, 15:15

requested review from @rainman110

rainman110 commented 1 year ago

In GitLab by @mariusalexander on May 5, 2023, 15:21

mentioned in merge request gtlab-coredev/gtlab!412

rainman110 commented 1 year ago

In GitLab by @rainman110 on May 8, 2023, 06:45

approved this merge request

rainman110 commented 1 year ago

In GitLab by @rainman110 on May 8, 2023, 06:45

Looks good to me.

rainman110 commented 1 year ago

In GitLab by @mariusalexander on May 8, 2023, 07:12

mentioned in commit d0bc2b24429eef2e8994f34fdae71875a17b5940