diff --git a/sol/proxy.hpp b/sol/proxy.hpp index aa0d6129..09097660 100644 --- a/sol/proxy.hpp +++ b/sol/proxy.hpp @@ -27,103 +27,103 @@ namespace sol { template struct proxy { private: - Table& tbl; - Key& key; + Table& tbl; + Key& key; public: - template - proxy( Table& table, T&& key ) : tbl( table ), key( std::forward( key ) ) { + template + proxy(Table& table, T&& key) : tbl(table), key(std::forward(key)) { - } + } - template - T get( ) const { - return tbl.get( key ); - } + template + T get() const { + return tbl.get(key); + } - template - proxy& set( T&& item ) { - tbl.set( key, std::forward( item ) ); - return *this; - } + template + proxy& set(T&& item) { + tbl.set(key, std::forward(item)); + return *this; + } - template - proxy& set_function( Args&&... args ) { - tbl.set_function( key, std::forward( args )... ); - return *this; - } + template + proxy& set_function(Args&&... args) { + tbl.set_function(key, std::forward(args)...); + return *this; + } - template - EnableIf>> operator=( U&& other ) { - tbl.set_function( key, std::forward( other ) ); - } + template + EnableIf>> operator=(U&& other) { + tbl.set_function(key, std::forward(other)); + } - template - DisableIf>> operator=( U&& other ) { - tbl.set( key, std::forward( other ) ); - } + template + DisableIf>> operator=(U&& other) { + tbl.set(key, std::forward(other)); + } - operator nil_t ( ) const { - return get( ); - } + operator nil_t () const { + return get(); + } - operator object( ) const { - return get( ); - } + operator object() const { + return get(); + } - operator function( ) const { - return get( ); - } + operator function() const { + return get(); + } - operator std::string( ) const { - return get( ); - } + operator std::string() const { + return get(); + } - template - operator bool( ) const { - return get( ); - } + template + operator bool() const { + return get(); + } - template - operator double( ) const { - return get( ); - } + template + operator double() const { + return get(); + } - template - operator float( ) const { - return get( ); - } + template + operator float() const { + return get(); + } - template - operator int( ) const { - return get( ); - } + template + operator int() const { + return get(); + } - template - typename multi_return::type call( Args&&... args ) { - return tbl.get(key)(types(), std::forward( args )...); - } + template + typename multi_return::type call(Args&&... args) { + return tbl.get(key)(types(), std::forward(args)...); + } }; template -inline bool operator== ( T&& left, const proxy& right ) { - return left == right.template get>( ); +inline bool operator== (T&& left, const proxy& right) { + return left == right.template get>(); } template -inline bool operator== ( const proxy& right, T&& left ) { - return right.template get>( ) == left; +inline bool operator== (const proxy& right, T&& left) { + return right.template get>() == left; } template -inline bool operator!= ( T&& left, const proxy& right ) { - return right.template get>( ) != left; +inline bool operator!= (T&& left, const proxy& right) { + return right.template get>() != left; } template -inline bool operator!= ( const proxy& right, T&& left ) { - return right.template get>( ) != left; +inline bool operator!= (const proxy& right, T&& left) { + return right.template get>() != left; } } // sol \ No newline at end of file diff --git a/sol/stack.hpp b/sol/stack.hpp index 6bb1608f..50ca2540 100644 --- a/sol/stack.hpp +++ b/sol/stack.hpp @@ -208,7 +208,7 @@ inline int push_user(lua_State* L, T& item) { namespace detail { template inline void push_tuple(lua_State* L, indices, T&& tuplen) { - using swallow = char[ 1 + sizeof...(I) ]; + using swallow = char[1 + sizeof...(I)]; swallow {'\0', (sol::stack::push(L, std::get(tuplen)), '\0')... }; } diff --git a/sol/state.hpp b/sol/state.hpp index 714a0f43..1c933903 100644 --- a/sol/state.hpp +++ b/sol/state.hpp @@ -175,13 +175,13 @@ public: } template - proxy operator[]( T&& key ) { - return global[ std::forward(key) ]; + proxy operator[](T&& key) { + return global[std::forward(key)]; } template - proxy operator[]( T&& key ) const { - return global[ std::forward( key ) ]; + proxy operator[](T&& key) const { + return global[std::forward(key)]; } }; } // sol diff --git a/sol/table.hpp b/sol/table.hpp index 7f51609a..39d18914 100644 --- a/sol/table.hpp +++ b/sol/table.hpp @@ -120,13 +120,13 @@ public: } template - proxy operator[]( T&& key ) { - return proxy( *this, std::forward( key ) ); + proxy operator[](T&& key) { + return proxy(*this, std::forward(key)); } template - proxy operator[]( T&& key ) const { - return proxy( *this, std::forward( key ) ); + proxy operator[](T&& key) const { + return proxy(*this, std::forward(key)); } private: @@ -182,7 +182,7 @@ private: void* userobjdata = static_cast(detail::get_ptr(obj)); lua_CFunction freefunc = &static_object_lua_func, TFx>::call; const char* freefuncname = fkey.c_str(); - const luaL_Reg funcreg[ 2 ] = { + const luaL_Reg funcreg[2] = { { freefuncname, freefunc }, { nullptr, nullptr } }; @@ -204,7 +204,7 @@ private: Decay target(std::forward(fx)); lua_CFunction freefunc = &static_lua_func::call; const char* freefuncname = fkey.c_str(); - const luaL_Reg funcreg[ 2 ] = { + const luaL_Reg funcreg[2] = { { freefuncname, freefunc }, { nullptr, nullptr } }; @@ -229,7 +229,7 @@ private: lua_CFunction freefunc = &lua_func::call; const char* freefuncname = fkey.c_str(); const char* metatablename = metakey.c_str(); - const luaL_Reg funcreg[ 2 ] = { + const luaL_Reg funcreg[2] = { { freefuncname, freefunc }, { nullptr, nullptr } }; diff --git a/sol/traits.hpp b/sol/traits.hpp index 57575cc2..cb082ca7 100644 --- a/sol/traits.hpp +++ b/sol/traits.hpp @@ -62,20 +62,20 @@ struct is_function_impl : std::is_function::type template struct is_function_impl { - using yes = char; - using no = struct { char s[ 2 ]; }; + using yes = char; + using no = struct { char s[2]; }; - struct F { void operator()( ); }; - struct Derived : T, F { }; - template struct Check; + struct F { void operator()(); }; + struct Derived : T, F { }; + template struct Check; - template - static no test( Check* ); + template + static no test(Check*); - template - static yes test( ... ); + template + static yes test(...); - static const bool value = sizeof( test( 0 ) ) == sizeof( yes ); + static const bool value = sizeof(test(0)) == sizeof(yes); }; } // detail diff --git a/tests.cpp b/tests.cpp index b0072c0a..8ec1a907 100644 --- a/tests.cpp +++ b/tests.cpp @@ -105,7 +105,7 @@ TEST_CASE("simple/call_lambda", "A C++ lambda is exposed to lua and called") { int x = 0; - lua.set_function("foo", [ &x ] { x = 1; }); + lua.set_function("foo", [&x] { x = 1; }); lua.script("foo()"); @@ -117,33 +117,33 @@ TEST_CASE("advanced/get_and_call", "Checks for lambdas returning values after a const static std::tuple heh_tuple = std::make_tuple(1, 6.28f, 3.14, std::string("heh")); sol::state lua; - REQUIRE_NOTHROW(lua.set_function("a", [ ] { return 42; })); + REQUIRE_NOTHROW(lua.set_function("a", [] { return 42; })); REQUIRE(lua.get("a").call() == 42); - REQUIRE_NOTHROW(lua.set_function("b", [ ] { return 42u; })); + REQUIRE_NOTHROW(lua.set_function("b", [] { return 42u; })); REQUIRE(lua.get("b").call() == 42u); - REQUIRE_NOTHROW(lua.set_function("c", [ ] { return 3.14; })); + REQUIRE_NOTHROW(lua.set_function("c", [] { return 3.14; })); REQUIRE(lua.get("c").call() == 3.14); - REQUIRE_NOTHROW(lua.set_function("d", [ ] { return 6.28f; })); + REQUIRE_NOTHROW(lua.set_function("d", [] { return 6.28f; })); REQUIRE(lua.get("d").call() == 6.28f); - REQUIRE_NOTHROW(lua.set_function("e", [ ] { return "lol"; })); + REQUIRE_NOTHROW(lua.set_function("e", [] { return "lol"; })); REQUIRE(lua.get("e").call() == lol); - REQUIRE_NOTHROW(lua.set_function("f", [ ] { return true; })); + REQUIRE_NOTHROW(lua.set_function("f", [] { return true; })); REQUIRE(lua.get("f").call()); - REQUIRE_NOTHROW(lua.set_function("g", [ ] { return std::string("str"); })); + REQUIRE_NOTHROW(lua.set_function("g", [] { return std::string("str"); })); REQUIRE(lua.get("g").call() == str); - REQUIRE_NOTHROW(lua.set_function("h", [ ] { })); + REQUIRE_NOTHROW(lua.set_function("h", [] { })); REQUIRE_NOTHROW(lua.get("h").call()); - REQUIRE_NOTHROW(lua.set_function("i", [ ] { return sol::nil; })); + REQUIRE_NOTHROW(lua.set_function("i", [] { return sol::nil; })); REQUIRE(lua.get("i").call() == sol::nil); - REQUIRE_NOTHROW(lua.set_function("j", [ ] { return std::make_tuple(1, 6.28f, 3.14, std::string("heh")); })); + REQUIRE_NOTHROW(lua.set_function("j", [] { return std::make_tuple(1, 6.28f, 3.14, std::string("heh")); })); REQUIRE((lua.get("j").call() == heh_tuple)); } @@ -152,33 +152,33 @@ TEST_CASE("advanced/operator[]_calls", "Checks for lambdas returning values usin const static std::tuple heh_tuple = std::make_tuple(1, 6.28f, 3.14, std::string("heh")); sol::state lua; - REQUIRE_NOTHROW(lua.set_function("a", [ ] { return 42; })); + REQUIRE_NOTHROW(lua.set_function("a", [] { return 42; })); REQUIRE(lua["a"].call() == 42); - REQUIRE_NOTHROW(lua.set_function("b", [ ] { return 42u; })); + REQUIRE_NOTHROW(lua.set_function("b", [] { return 42u; })); REQUIRE(lua["b"].call() == 42u); - REQUIRE_NOTHROW(lua.set_function("c", [ ] { return 3.14; })); + REQUIRE_NOTHROW(lua.set_function("c", [] { return 3.14; })); REQUIRE(lua["c"].call() == 3.14); - REQUIRE_NOTHROW(lua.set_function("d", [ ] { return 6.28f; })); + REQUIRE_NOTHROW(lua.set_function("d", [] { return 6.28f; })); REQUIRE(lua["d"].call() == 6.28f); - REQUIRE_NOTHROW(lua.set_function("e", [ ] { return "lol"; })); + REQUIRE_NOTHROW(lua.set_function("e", [] { return "lol"; })); REQUIRE(lua["e"].call() == lol); - REQUIRE_NOTHROW(lua.set_function("f", [ ] { return true; })); + REQUIRE_NOTHROW(lua.set_function("f", [] { return true; })); REQUIRE(lua["f"].call()); - REQUIRE_NOTHROW(lua.set_function("g", [ ] { return std::string("str"); })); + REQUIRE_NOTHROW(lua.set_function("g", [] { return std::string("str"); })); REQUIRE(lua["g"].call() == str); - REQUIRE_NOTHROW(lua.set_function("h", [ ] { })); + REQUIRE_NOTHROW(lua.set_function("h", [] { })); REQUIRE_NOTHROW(lua["h"].call()); - REQUIRE_NOTHROW(lua.set_function("i", [ ] { return sol::nil; })); + REQUIRE_NOTHROW(lua.set_function("i", [] { return sol::nil; })); REQUIRE(lua["i"].call() == sol::nil); - REQUIRE_NOTHROW(lua.set_function("j", [ ] { return std::make_tuple(1, 6.28f, 3.14, std::string("heh")); })); + REQUIRE_NOTHROW(lua.set_function("j", [] { return std::make_tuple(1, 6.28f, 3.14, std::string("heh")); })); REQUIRE((lua["j"].call() == heh_tuple)); } @@ -186,7 +186,7 @@ TEST_CASE("advanced/call_lambdas", "A C++ lambda is exposed to lua and called") sol::state lua; int x = 0; - lua.set_function("set_x", [ &] (int new_x) { + lua.set_function("set_x", [&] (int new_x) { x = new_x; return 0; }); @@ -216,16 +216,16 @@ TEST_CASE("tables/variables", "Check if tables and variables work as intended") TEST_CASE("tables/functions_variables", "Check if tables and function calls work as intended") { sol::state lua; lua.open_libraries(sol::lib::base, sol::lib::os); - auto run_script = [ ] (sol::state& lua) -> void { + auto run_script = [] (sol::state& lua) -> void { lua.script("assert(os.fun() == \"test\")"); }; lua.get("os").set_function("fun", - [ ] () { + [] () { std::cout << "stateless lambda()" << std::endl; return "test"; } - ); + ); REQUIRE_NOTHROW(run_script(lua)); lua.get("os").set_function("fun", &free_function); @@ -239,11 +239,11 @@ TEST_CASE("tables/functions_variables", "Check if tables and function calls work // stateful lambda: non-convertible, unoptomizable int breakit = 50; lua.get("os").set_function("fun", - [ &breakit ] () { + [&breakit] () { std::cout << "stateless lambda()" << std::endl; return "test"; } - ); + ); REQUIRE_NOTHROW(run_script(lua)); // r-value, cannot optomize @@ -256,12 +256,12 @@ TEST_CASE("tables/functions_variables", "Check if tables and function calls work REQUIRE_NOTHROW(run_script(lua)); } -TEST_CASE("functions/return_order_and_multi_get", "Check if return order is in the same reading order specified in Lua" ) { +TEST_CASE("functions/return_order_and_multi_get", "Check if return order is in the same reading order specified in Lua") { const static std::tuple triple = std::make_tuple(10, 11, 12); sol::state lua; - lua.set_function( "f", [ ] { - return std::make_tuple( 10, 11, 12 ); - } ); + lua.set_function("f", [] { + return std::make_tuple(10, 11, 12); + }); lua.script("function g() return 10, 11, 12 end\nx,y,z = g()"); auto tcpp = lua.get("f").call(); auto tlua = lua.get("g").call(); @@ -301,7 +301,7 @@ TEST_CASE("tables/operator[]", "Check if operator[] retrieval and setting works REQUIRE_NOTHROW(lua.script("assert(test(10, 11, \"hello\") == 11)")); // function retrieval - sol::function test = lua[ "test" ]; + sol::function test = lua["test"]; REQUIRE(test.call(10, 11, "hello") == 11); // setting a lambda @@ -312,13 +312,13 @@ TEST_CASE("tables/operator[]", "Check if operator[] retrieval and setting works REQUIRE_NOTHROW(lua.script("assert(lamb(220) == 440)")); // function retrieval of a lambda - sol::function lamb = lua[ "lamb" ]; + sol::function lamb = lua["lamb"]; REQUIRE(lamb.call(220) == 440); // test const table retrieval auto assert1 = [](const sol::table& t) { std::string a = t["foo"]; - int b = t["bar"]; + int b = t["bar"]; std::cout << a << ',' << b << '\n'; };