Statement executeMany not compiling with GCC 9 C++17

[SRombauts]

See https://travis-ci.org/SRombauts/SQLiteCpp/jobs/549915202

In file included from /usr/include/c++/9/pstl/glue_algorithm_defs.h:13,
                 from /usr/include/c++/9/algorithm:71,
                 from /home/travis/build/SRombauts/SQLiteCpp/googletest/googletest/include/gtest/internal/gtest-port.h:274,
                 from /home/travis/build/SRombauts/SQLiteCpp/googletest/googletest/include/gtest/internal/gtest-internal.h:40,
                 from /home/travis/build/SRombauts/SQLiteCpp/googletest/googletest/include/gtest/gtest.h:58,
                 from /home/travis/build/SRombauts/SQLiteCpp/tests/ExecuteMany_test.cpp:17:
/usr/include/c++/9/functional: In instantiation of ‘std::_Bind<_Functor(_Bound_args ...)>::_Bind(const _Functor&, _Args&& ...) [with _Args = {std::tuple<int>}; _Functor = SQLite::Statement; _Bound_args = {std::tuple<int>}]’:
/usr/include/c++/9/functional:811:38:   required from ‘typename std::_Bind_helper<std::__is_socketlike<_Func>::value, _Func, _BoundArgs ...>::type std::bind(_Func&&, _BoundArgs&& ...) [with _Func = SQLite::Statement&; _BoundArgs = {std::tuple<int>}; typename std::_Bind_helper<std::__is_socketlike<_Func>::value, _Func, _BoundArgs ...>::type = std::_Bind<SQLite::Statement(std::tuple<int>)>]’
/home/travis/build/SRombauts/SQLiteCpp/include/SQLiteCpp/ExecuteMany.h:84:9:   required from ‘void SQLite::bind_exec(SQLite::Statement&, std::tuple<_Tps ...>&&) [with Types = {int}]’
/home/travis/build/SRombauts/SQLiteCpp/include/SQLiteCpp/ExecuteMany.h:50:14:   required from ‘void SQLite::execute_many(SQLite::Database&, const char*, Arg&&, Types&& ...) [with Arg = std::tuple<int>; Types = {std::tuple<int, const char*>, std::tuple<int, const char*>}]’
/home/travis/build/SRombauts/SQLiteCpp/tests/ExecuteMany_test.cpp:35:9:   required from here
/usr/include/c++/9/functional:462:59: error: ‘SQLite::Statement::Statement(const SQLite::Statement&)’ is private within this context
  462 |  : _M_f(__f), _M_bound_args(std::forward<_Args>(__args)...)
      |                                                           ^
In file included from /home/travis/build/SRombauts/SQLiteCpp/include/SQLiteCpp/Column.h:13,
                 from /home/travis/build/SRombauts/SQLiteCpp/include/SQLiteCpp/Database.h:13,
                 from /home/travis/build/SRombauts/SQLiteCpp/tests/ExecuteMany_test.cpp:13:
/home/travis/build/SRombauts/SQLiteCpp/include/SQLiteCpp/Statement.h:696:5: note: declared private here
  696 |     Statement(const Statement&);
      |     ^~~~~~~~~

[SRombauts]

Added by #197

[SRombauts]

Perhaps @maxbachmann can have a look at this?

[maxbachmann]

@SRombauts I will look into this
Edit: after looking on it I am quite confused why this error is caused. I mean for it works with c++17 in gcc8 so it makes not much sence that is does not work in gcc9. I mean it´s still the same c++ standard and has nothing to do with the flags. Do you have any idea what could cause this?

[maxbachmann]

@SRombauts this kind of bugged me so I tested it locally on my computer where I use gcc 9.1.1 and it worked with c++11 c++14 and c++2a without any issues ;) (which is kind of strange because travis uses gcc 9.1.0 and it makes no sence that it behaves differently)

[maxbachmann]

It appears even though namespace std is not used the ADL (argument dependent lookup) still picks std::bind instead of SQLite::bind. So the issue can be fixed by using the scope SQLite in this place aswell and it might be a good idea to avoid using a name from the standard library in the future