call by reference, using a pointer or a non-const reference, does not work

[shierei]

Declaring a metaclass function with a pointer or a non-const reference to any basic type as an argument does not compile. The pointer to a basic type can be made compilable if you register this basic type. But if you register this basic type, and property or function returning value of this type becomes ponder::ValueKind::User. This prevents you from converting the property value or the function return value back to the basic type. The conversion would throw an exception. It basically makes this basic value type used as a property value or a function returning value not visible to the client.

[billyquith]

This is an awkward one to fix. However, I would like to fix it. It is tricky because Values handle basic types, but they do not support references to basic types. Hence you have to reference them as user types, which do support references (both ref and values). It seems a bit overkill to support basic types like this, but perhaps that should be a default solution until I can think of a better one.

[billyquith]

Note to self: Value holder policies. external/internal.

[shierei]

I have a fix to support pointer typed argument nicely. That is all you need if you want a metaclass's function to return something through a function argument with a pointer to a basic type.

[billyquith]

Sure, yes, I’d be interested in seeing it.

[shierei]

I like to show you in a tested code but it looks like currently the ponder branch is not buildable.

[billyquith]

What seems to be broken? All of the tests appear to be passes in develop and master branches on all platforms.

[shierei]

I was not able to build on a RedHat platform. On Mac, it built however. Below is the build error.

[  1%] Building CXX object CMakeFiles/ponder.dir/src/class.cpp.o
In file included from ./ponder/include/ponder/detail/idtraits.hpp:57:0,
                 from ./ponder/include/ponder/config.hpp:79,
                 from ./ponder/include/ponder/classget.hpp:36,
                 from ./ponder/include/ponder/class.hpp:34,
                 from ./ponder/src/class.cpp:30:
./ponder/include/ponder/detail/string_view.hpp: In instantiation of  constexpr int ponder::detail::basic_string_view<CharT, Traits>::compare(ponder::detail::basic_string_view<CharT, Traits>) const [with CharT = char; Traits = std::char_traits<char>] :
./ponder/include/ponder/detail/string_view.hpp:301:85:   required from  constexpr bool ponder::detail::basic_string_view<CharT, Traits>::operator<(const ponder::detail::basic_string_view<CharT, Traits>&) const [with CharT = char; Traits = std::char_traits<char>] 
./ponder/include/ponder/detail/dictionary.hpp:46:49:   required from  bool ponder::detail::DictKeyCmp<T>::operator()(T, T) const [with T = ponder::detail::basic_string_view<char>] 
./ponder/include/ponder/detail/dictionary.hpp:102:59:   required from  ponder::detail::Dictionary<KEY, KEY_REF, VALUE, CMP>::const_iterator ponder::detail::Dictionary<KEY, KEY_REF, VALUE, CMP>::findKey(KEY_REF) const [with KEY = std::basic_string<char>; KEY_REF = ponder::detail::basic_string_view<char>; VALUE = std::shared_ptr<ponder::Function>; CMP = ponder::detail::DictKeyCmp<ponder::detail::basic_string_view<char> >; ponder::detail::Dictionary<KEY, KEY_REF, VALUE, CMP>::const_iterator = __gnu_cxx::__normal_iterator<const ponder::detail::Dictionary<std::basic_string<char>, ponder::detail::basic_string_view<char>, std::shared_ptr<ponder::Function> >::pair_t*, std::vector<ponder::detail::Dictionary<std::basic_string<char>, ponder::detail::basic_string_view<char>, std::shared_ptr<ponder::Function> >::pair_t, std::allocator<ponder::detail::Dictionary<std::basic_string<char>, ponder::detail::basic_string_view<char>, std::shared_ptr<ponder::Function> >::pair_t> > >] 
./ponder/include/ponder/detail/dictionary.hpp:119:40:   required from  bool ponder::detail::Dictionary<KEY, KEY_REF, VALUE, CMP>::tryFind(KEY_REF, ponder::detail::Dictionary<KEY, KEY_REF, VALUE, CMP>::const_iterator&) const [with KEY = std::basic_string<char>; KEY_REF = ponder::detail::basic_string_view<char>; VALUE = std::shared_ptr<ponder::Function>; CMP = ponder::detail::DictKeyCmp<ponder::detail::basic_string_view<char> >; ponder::detail::Dictionary<KEY, KEY_REF, VALUE, CMP>::const_iterator = __gnu_cxx::__normal_iterator<const ponder::detail::Dictionary<std::basic_string<char>, ponder::detail::basic_string_view<char>, std::shared_ptr<ponder::Function> >::pair_t*, std::vector<ponder::detail::Dictionary<std::basic_string<char>, ponder::detail::basic_string_view<char>, std::shared_ptr<ponder::Function> >::pair_t, std::allocator<ponder::detail::Dictionary<std::basic_string<char>, ponder::detail::basic_string_view<char>, std::shared_ptr<ponder::Function> >::pair_t> > >] 
./ponder/include/ponder/class.inl:78:37:   required from here
./ponder/include/ponder/detail/string_view.hpp:201:5: error: body of constexpr function  constexpr int ponder::detail::basic_string_view<CharT, Traits>::compare(ponder::detail::basic_string_view<CharT, Traits>) const [with CharT = char; Traits = std::char_traits<char>]  not a return-statement
     }
     ^
make[2]: *** [CMakeFiles/ponder.dir/src/class.cpp.o] Error 1
make[1]: *** [CMakeFiles/ponder.dir/all] Error 2
make: *** [all] Error 2

[billyquith]

Please note compiler version notes. C++14 required.

[shierei]

Cmake automatically configured it to use std=gnu++1y on the RedHat platform.

[shierei]

This must be new since I was able to build just a few days ago.

[billyquith]

Yes. It’s mentioned in the change log and release notes.

[billyquith]

This must be new since I was able to build just a few days ago.

I put a more explicit note about C++14 support in the notes. Sorry if it wasn't clear.