Skip to content

Commit ea8f66b

Browse files
committed
Merge #89: header: Delete default Minisketch constructor
5cc7d20 header: Delete default Minisketch constructor (TheCharlatan) Pull request description: I might be missing a reason for having the default constructor, but it is not used anywhere and trying to use one of its methods will cause a segfault. ACKs for top commit: sipa: ACK 5cc7d20 Tree-SHA512: 5d395ce919b0105b644e83cf3d9867c4ee8dd92d3561c56aa72794704f26167465a24d541d6cf266bf49a9fec48ad492edf4ad1083c0ae799335865a23ac462c
2 parents fc24958 + 5cc7d20 commit ea8f66b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

include/minisketch.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -248,8 +248,8 @@ class Minisketch
248248
/** Check whether this Minisketch object is valid. */
249249
explicit operator bool() const noexcept { return bool{m_minisketch}; }
250250

251-
/** Construct an (invalid) Minisketch object. */
252-
Minisketch() noexcept = default;
251+
/** Default constructor is deleted. */
252+
Minisketch() noexcept = delete;
253253

254254
/** Move constructor. */
255255
Minisketch(Minisketch&&) noexcept = default;

0 commit comments

Comments
 (0)