wallet2_api: use std::optional instead of handmade optional

Cleaner, adheres to POLA, and allows for better compiler optimization. Declaration should be backwards compatible. Requires C++17.

src/wallet/api/wallet2_api.h

@@ -31,14 +31,15 @@
 #pragma once
 
 
-#include <string>
+#include <cstdint>
-#include <vector>
-#include <list>
-#include <set>
 #include <ctime>
 #include <iostream>
+#include <list>
+#include <optional>
+#include <set>
 #include <stdexcept>
-#include <cstdint>
+#include <string>
+#include <vector>
 
 //  Public interface for libwallet library
 namespace Monero {
@@ -54,18 +55,9 @@ enum NetworkType : uint8_t {
         void onStartup();
     }
 
+    // backwards compatible shim for old declaration of handmade optional<> struct
     template<typename T>
-    class optional {
+    using optional = std::optional<T>;
-      public:
-        optional(): set(false) {}
-        optional(const T &t): t(t), set(true) {}
-        const T &operator*() const { return t; }
-        T &operator*() { return t; }
-        operator bool() const { return set; }
-      private:
-        T t;
-        bool set;
-    };
 
 /**
  * @brief Transaction-like interface for sending money