Compiling micropython module with g++ errors

[danimasa]

Hello

I'm trying to compile some code with a g++ compiler because I need to integrate micropython kernel with some cpp code. But I'm getting the following error in MP_DEFINE_CONST_FUN_OBJ_3 macro and QSTR generation:

/work/modules/micropython/py/obj.h:375:32: error: expected primary-expression before '.' token
     {{&mp_type_fun_builtin_3}, .fun._3 = fun_name}
                                ^
/work/ports/esp32/MPyDictionary.cpp:22:12: note: in expansion of macro 'MP_DEFINE_CONST_FUN_OBJ_3'
 extern "C" MP_DEFINE_CONST_FUN_OBJ_3(agros_dict_set_obj, agros_dict_set);
            ^~~~~~~~~~~~~~~~~~~~~~~~~
/work/modules/micropython/py/obj.h:372:32: error: expected primary-expression before '.' token
     {{&mp_type_fun_builtin_2}, .fun._2 = fun_name}
                                ^
/work/ports/esp32/MPyDictionary.cpp:31:12: note: in expansion of macro 'MP_DEFINE_CONST_FUN_OBJ_2'
 extern "C" MP_DEFINE_CONST_FUN_OBJ_2(agros_dict_get_obj, agros_dict_get);
            ^~~~~~~~~~~~~~~~~~~~~~~~~
/work/ports/esp32/MPyDictionary.cpp:34:17: error: 'MP_QSTR_dict_set' was not declared in this scope
   { MP_ROM_QSTR(MP_QSTR_dict_set), MP_ROM_PTR(&agros_dict_set_obj) },
                 ^~~~~~~~~~~~~~~~
/work/modules/micropython/py/obj.h:96:56: note: in definition of macro 'MP_OBJ_NEW_QSTR'
 #define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 3) | 2))
                                                        ^~~
/work/ports/esp32/MPyDictionary.cpp:34:5: note: in expansion of macro 'MP_ROM_QSTR'
   { MP_ROM_QSTR(MP_QSTR_dict_set), MP_ROM_PTR(&agros_dict_set_obj) },
     ^~~~~~~~~~~
/work/ports/esp32/MPyDictionary.cpp:34:17: note: suggested alternative: 'MP_QSTR_dict_view'
   { MP_ROM_QSTR(MP_QSTR_dict_set), MP_ROM_PTR(&agros_dict_set_obj) },
                 ^~~~~~~~~~~~~~~~
/work/modules/micropython/py/obj.h:96:56: note: in definition of macro 'MP_OBJ_NEW_QSTR'
 #define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 3) | 2))
                                                        ^~~
/work/ports/esp32/MPyDictionary.cpp:34:5: note: in expansion of macro 'MP_ROM_QSTR'
   { MP_ROM_QSTR(MP_QSTR_dict_set), MP_ROM_PTR(&agros_dict_set_obj) },
     ^~~~~~~~~~~
/work/ports/esp32/MPyDictionary.cpp:35:17: error: 'MP_QSTR_dict_get' was not declared in this scope
   { MP_ROM_QSTR(MP_QSTR_dict_get), MP_ROM_PTR(&agros_dict_get_obj) },
                 ^~~~~~~~~~~~~~~~
/work/modules/micropython/py/obj.h:96:56: note: in definition of macro 'MP_OBJ_NEW_QSTR'
 #define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 3) | 2))
                                                        ^~~
/work/ports/esp32/MPyDictionary.cpp:35:5: note: in expansion of macro 'MP_ROM_QSTR'
   { MP_ROM_QSTR(MP_QSTR_dict_get), MP_ROM_PTR(&agros_dict_get_obj) },
     ^~~~~~~~~~~
/work/ports/esp32/MPyDictionary.cpp:35:17: note: suggested alternative: 'MP_QSTR_dict_view'
   { MP_ROM_QSTR(MP_QSTR_dict_get), MP_ROM_PTR(&agros_dict_get_obj) },
                 ^~~~~~~~~~~~~~~~
/work/modules/micropython/py/obj.h:96:56: note: in definition of macro 'MP_OBJ_NEW_QSTR'
 #define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 3) | 2))
                                                        ^~~
/work/ports/esp32/MPyDictionary.cpp:35:5: note: in expansion of macro 'MP_ROM_QSTR'
   { MP_ROM_QSTR(MP_QSTR_dict_get), MP_ROM_PTR(&agros_dict_get_obj) },
     ^~~~~~~~~~~
/work/ports/esp32/MPyDictionary.cpp:42:11: error: 'MP_QSTR_agrosdict' was not declared in this scope
   .name = MP_QSTR_agrosdict,
           ^~~~~~~~~~~~~~~~~
/work/ports/esp32/MPyDictionary.cpp:42:11: note: suggested alternative: 'MP_QSTR_randint'
   .name = MP_QSTR_agrosdict,
           ^~~~~~~~~~~~~~~~~
           MP_QSTR_randint

I tried to compile the code with an extern "C" flag but didn't help.

[danimasa]

I resolved the problem adapting the syntax to cpp:

// from
MP_DEFINE_CONST_FUN_OBJ_3(agros_dict_set_obj, agros_dict_set);

// to
const mp_obj_fun_builtin_fixed_t agros_dict_set_obj =
    {{&mp_type_fun_builtin_3}, { ._3 = agros_dict_set }};

[mo-FEAR]

I just created a PR for this. It should allow pure cpp files as user c modules without having to split the MP definitions into c files