LuaGlue icon indicating copy to clipboard operation
LuaGlue copied to clipboard

Published 20 hours ago • verified publisher icon Tomasu

luaglue namespace

Open JMLX42 opened this issue 11 years ago • 4 comments

Hello,

there is no luaglue namespace; it would be definitely better than prefixing all classes. The biggest problem is some global definitions that might collide with other libs.

Are you ok to add a luaglue namespace?

Regards,

JMLX42 avatar Jan 13 '14 13:01 JMLX42

Yeah, see issue #6 . I'll do it soonish.

Tomasu avatar Jan 14 '14 02:01 Tomasu

Nothing in common with #6. I'm talking about namespacing the LuaGlue* classes. Ex: luaglue::Class instead of LuaGlueClass.

JMLX42 avatar Jan 14 '14 02:01 JMLX42

I'm not aware of any globals that can conflict? If you mean macros, they can't be namespaced.

namespacing might be in the cards.

Tomasu avatar Jan 14 '14 02:01 Tomasu

I had to add remove_const custom meta functions to make it work with VS (incoming pull request btw). I named them luaglue_remove_const which is quite ugly don't you think?

JMLX42 avatar Jan 14 '14 02:01 JMLX42