__index and __newindex can not work in kaguya.

[aster2013]

e.g.

static const Variant& VariantMapGetVariant(const VariantMap& variantMap, const String& key)
{
    VariantMap::ConstIterator i = variantMap.Find(StringHash(key));
    if (i == variantMap.End())
        return Variant::EMPTY;

    return i->second_;
}

static void VariantMapSetVariant(VariantMap& variantMap, const String& key, const Variant& value)
{
    variantMap[StringHash(key)] = value;
}

static void RegisterVariantMap(kaguya::State& lua)
{
    using namespace kaguya;

    lua["KVariantMap"].setClass(UserdataMetatable<VariantMap>()
        .setConstructors<VariantMap()>()

        // Can not work in kaguya
        // .addStaticFunction("__index", &VariantMapGetVariant)
        // .addStaticFunction("__newindex", &VariantMapSetVariant)

        .addStaticFunction("Get", &VariantMapGetVariant)
        .addStaticFunction("Set", &VariantMapSetVariant)
        );
}

[aster2013]

Current in kaguya, __index meta method is using for property. This is a good idea.

[satoren]

Property using newindex. All member's using index

[aster2013]

And please add another function:

template<typename ... Funcs>
UserdataMetatable<T>::addStaticOverloadedFunctions(const char* name, Funcs... f)
{
}

[satoren]

Why?

[aster2013]

For example:

class Viewport
{
};

Viewport* CreateViewport1();
Viewport* CreateViewport2(Scene*);
Viewport* CreateViewport3(Scene*, Camera*);

 lua["Viewport"].setClass(UserdataMetatable<Viewport>()
      .addStaticFunction("new", CreateViewport1, &CreateViewport2, &CreateViewport3)
// ...

[satoren]

Currently.

        template<typename... Funcs>
        UserdataMetatable& addOverloadedFunctions(const char* name, Funcs... f)

And the future that does not change.

[aster2013]

Bad performance.

LuaFunction indexfun = kaguya::LuaFunction::loadstring(state, "local arg = {...};local metatable = arg[1];"
                        "return function(table, index)"
                        //                      " if type(table) == 'userdata' then "
                        " local propfun = metatable['_prop_'..index];"
                        " if propfun then return propfun(table) end "
                        //                      " end "
                        " return metatable[index]"
                        " end")(metatable);

the function may call muiti-times, with index like this: prop'name' _propprop_'name' _propprop_prop'name'

you can write print(index) to view it.

[satoren]

index and newindex rewrite on C.

[satoren]

If allow to add index and newindex, do you need what kind of work? Propery and function will not work if it is to overwrite.

[aster2013]

the index and newindex method is using for HashMap in urho3d. like std::map<std::string, variant>.

[satoren]

It can not be convert to lua table?

[satoren]

Maybe better performance lua-table than userdata, If the HashMap modify a lot.

[aster2013]

Current the VariantMap is defines as HashMap<StringHash, Variant>, the key is a string hash not a string. and the VariantMap is just for event data. it will not modify a lot.

[satoren]

OK. I understand now.

[aster2013]

Hi, the function with default argument value is not working now.

[satoren]

default argument value can not detect in C++ code.#17

[satoren]

Added support for default argument value.