Merge #87: Avoid copy in self-assign

8ea298b Avoid copy in self-assign (Cory Fields)

Pull request description:

  I believe this was harmless, but it avoids a copy and quiets the clang-tidy check.

ACKs for top commit:
  sipa:
    utACK 8ea298b

Tree-SHA512: 21b3c81a32379acc538c462be0f6685a786722772bc09589356b503195e1dd5da00df2d6c1b6d6d6ce77edbd46a06df64e5c256764ce044257c8f070487b57e5

Author: sipa

include/minisketch.h

@@ -239,7 +239,7 @@ class Minisketch
     /** Make this Minisketch a clone of the specified one. */
     Minisketch& operator=(const Minisketch& sketch) noexcept
     {
-        if (sketch.m_minisketch) {
+        if (this != &sketch && sketch.m_minisketch) {
             m_minisketch = std::unique_ptr<minisketch, Deleter>(minisketch_clone(sketch.m_minisketch.get()));
         }
         return *this;