[DynInst_API:] [dyninst/dyninst] e5409c: Make CondVar non-copyable, non-moveable (#1426)


Date: Tue, 09 May 2023 10:09:33 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] e5409c: Make CondVar non-copyable, non-moveable (#1426)
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: e5409c471292cb8b1c87e9b5ae1e94fa9b6eb573
      https://github.com/dyninst/dyninst/commit/e5409c471292cb8b1c87e9b5ae1e94fa9b6eb573
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-09 (Tue, 09 May 2023)

  Changed paths:
    M common/src/dthread.h

  Log Message:
  -----------
  Make CondVar non-copyable, non-moveable (#1426)

This makes it consistent with std::condition_variable.

Found using cppcheck:

common/src/dthread.h:114:4: warning: Class 'CondVar' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor]
   mutex = new mutex_t;
   ^
common/src/dthread.h:114:4: warning: Class 'CondVar' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq]
   mutex = new mutex_t;


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] e5409c: Make CondVar non-copyable, non-moveable (#1426), Tim Haines <=