Skip to content

Commit

Permalink
Delete copy ctor and copy-assignment for log::Batch
Browse files Browse the repository at this point in the history
Fixes rule-of-five violation and reduces flakiness of playback tests.

Signed-off-by: Michael Beardsworth <beardsworth@intrinsic.ai>
  • Loading branch information
mbeards committed May 14, 2024
1 parent f043ddb commit 8bb598a
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions log/include/gz/transport/log/Batch.hh
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,12 @@ namespace gz
/// \return The updated Batch instance.
public: Batch& operator=(Batch &&_other); // NOLINT

/// \brief No copy constructor.
public: Batch(const Batch& other) = delete;

/// \brief No copy-assignment operator.
public: Batch& operator=(const Batch& other) = delete;

/// \brief destructor
public: ~Batch();

Expand Down

0 comments on commit 8bb598a

Please sign in to comment.