Remove move semantics from addManifest

There is no reason to use move for a shared smart pointer. Let
the pointer control block deal with the references.

Author: jamescowens

src/gridcoin/scraper/scraper.cpp

@@ -4271,7 +4271,7 @@ bool ScraperSendFileManifestContents(CBitcoinAddress& Address, CKey& Key) EXCLUS
 
     LOCK2(CScraperManifest::cs_mapManifest, CSplitBlob::cs_mapParts);
 
-    bool bAddManifestSuccessful = CScraperManifest::addManifest(std::move(manifest), Key);
+    bool bAddManifestSuccessful = CScraperManifest::addManifest(manifest, Key);
 
     if (bAddManifestSuccessful)
         _log(logattribute::INFO, "ScraperSendFileManifestContents", "addManifest (send) from this scraper (address "

src/gridcoin/scraper/scraper_net.cpp

@@ -756,7 +756,7 @@ bool CScraperManifest::RecvManifest(CNode* pfrom, CDataStream& vRecv)
     return true;
 }
 
-bool CScraperManifest::addManifest(std::shared_ptr<CScraperManifest>&& m, CKey& keySign)
+bool CScraperManifest::addManifest(std::shared_ptr<CScraperManifest> m, CKey& keySign)
 EXCLUSIVE_LOCKS_REQUIRED(CScraperManifest::cs_mapManifest, cs_mapParts)
 {
     uint256 hash;

src/gridcoin/scraper/scraper_net.h

@@ -125,7 +125,7 @@ class CScraperManifest
      static bool SendManifestTo(CNode* pfrom, const uint256& hash);
 
     /** Add new manifest object into list of known manifests */
-    static bool addManifest(std::shared_ptr<CScraperManifest>&& m, CKey& keySign);
+    static bool addManifest(std::shared_ptr<CScraperManifest> m, CKey& keySign);
 
     /** Validate whether received manifest is authorized */
     static bool IsManifestAuthorized(int64_t& nTime, CPubKey& PubKey, unsigned int& banscore_out);