Fix #25: Fix ambiguities with overloaded std::forward definitions.

src/fsm.h

@@ -136,7 +136,10 @@ namespace fsmlite {
 
         template<class F, class Arg1, class Arg2>
         invoke_as_binary_fn_result_t<F, Arg1, Arg2>invoke_as_binary_fn(F&& f, Arg1&& a, Arg2&& b) {
-            return binary_fn_helper<F, Arg1, Arg2>::invoke(forward<F>(f), forward<Arg1>(a), forward<Arg2>(b));
+            return binary_fn_helper<F, Arg1, Arg2>::invoke(
+                detail::forward<F>(f),
+                detail::forward<Arg1>(a),
+                detail::forward<Arg2>(b));
         }
 
         // basic template metaprogramming stuff; note that we could

tests/.gitignore

@@ -7,4 +7,5 @@ test_player
 test_recursive
 test_row
 test_scoped
+test_shared
 test_traits

tests/Makefile.am

@@ -10,6 +10,7 @@ check_PROGRAMS = \
 	test_recursive \
 	test_row \
 	test_scoped \
+	test_shared \
 	test_traits
 
 TESTS = $(check_PROGRAMS)

tests/test_shared.cpp

@@ -0,0 +1,33 @@
+#include <cassert>
+#include <memory>
+
+#include "fsm.h"
+
+class state_machine: public fsmlite::fsm<state_machine> {
+    friend class fsm;  // base class needs access to transition_table
+public:
+    enum states { Init, Exit };
+
+    using event = std::shared_ptr<int>;
+
+    void action(const event&) { }
+
+private:
+    using m = state_machine;
+
+    using transition_table = table<
+//  Row-Type    Start Event  Target Action
+//  -----------+-----+------+------+----------+-
+    mem_fn_row< Init, event, Exit,  &m::action >
+//  -----------+-----+------+------+----------+-
+    >;
+};
+
+int main()
+{
+    state_machine m;
+    assert(m.current_state() == state_machine::Init);
+    m.process_event(state_machine::event());
+    assert(m.current_state() == state_machine::Exit);
+    return 0;
+}