Better hide methods which are not part of the public API.

From C++ perspective these methods are already not accessible. But code-completion in MSVC does not correctly recognize this.

Author: rsomla1

include/mysqlx/devapi/detail/result.h

@@ -74,6 +74,10 @@ class PUBLIC_API Result_detail
   Result_detail(const Result_detail&) = delete;
   Result_detail& operator=(const Result_detail&) = delete;
 
+public:
+
+  struct INTERNAL Impl;
+
 protected:
 
   Result_detail(common::Result_init&);
@@ -87,13 +91,9 @@ class PUBLIC_API Result_detail
 
   Result_detail& operator=(Result_detail&&);
 
-public:
-
   Result_detail() = default;
   virtual ~Result_detail();
 
-  struct INTERNAL Impl;
-
   Impl& get_impl();
 
   const Impl& get_impl() const
@@ -115,7 +115,7 @@ class PUBLIC_API Result_detail
   bool has_data() const;
   bool next_result();
 
-private:
+protected:
 
   Impl  *m_impl = nullptr;
   bool m_owns_impl = false;
@@ -149,6 +149,8 @@ class PUBLIC_API Result_detail
 
   using WarningList = internal::List_initializer<Array_source<Warning_src>>;
 
+protected:
+
   unsigned    get_warning_count() const;
   Warning     get_warning(size_t pos);
 
@@ -181,7 +183,7 @@ class PUBLIC_API Result_detail
 class PUBLIC_API Column_detail
   : virtual common::Printable
 {
-public:
+protected:
 
   using Impl = common::Column_info<string>;
 

include/mysqlx/devapi/detail/session.h

@@ -209,6 +209,8 @@ class PUBLIC_API Schema_detail
   using TableList      = List_initializer<List_source<Table_src>>;
   using StringList     = List_initializer<List_source<Name_src>>;
 
+protected:
+
   void  create_collection(const string &name, bool reuse);
   void  drop_collection(const string &name);