Fix Enums (#114)

* handle `std::byte` as a special case

* fix BuildScopeProxyDict to handle enum constants decl as static

Author: Vipul-Cariappa

src/Converters.cxx

@@ -3488,7 +3488,8 @@ CPyCppyy::Converter* CPyCppyy::CreateConverter(Cppyy::TCppType_t type, cdims_t d
 // converters for known C++ classes and default (void*)
     Converter* result = nullptr;
     Cppyy::TCppScope_t klass = Cppyy::GetScopeFromType(realType);
-    if (klass || (klass = Cppyy::GetFullScope(realTypeStr))) {
+    // std::byte is a special enum class used to access raw memory
+    if ((realTypeStr != "std::byte") && (klass || (klass = Cppyy::GetFullScope(realTypeStr)))) {
         Cppyy::TCppType_t raw{0};
         if (Cppyy::GetSmartPtrInfo(realTypeStr, &raw, nullptr)) {
             if (cpd == "") {

src/ProxyWrappers.cxx

@@ -99,7 +99,7 @@ void AddPropertyToClass(PyObject* pyclass,
     PyType_Type.tp_setattro(pyclass, pname, (PyObject*)property);
 
 // allow access at the class level (always add after setting instance level)
-    if (Cppyy::IsStaticDatamember(data))
+    if (Cppyy::IsStaticDatamember(data) || Cppyy::IsEnumConstant(data))
         PyType_Type.tp_setattro((PyObject*)Py_TYPE(pyclass), pname, (PyObject*)property);
 
 // cleanup
@@ -378,11 +378,7 @@ static int BuildScopeProxyDict(Cppyy::TCppScope_t scope, PyObject* pyclass, cons
             continue;
 
     // enum datamembers (this in conjunction with previously collected enums above)
-        if (Cppyy::IsEnumType(Cppyy::GetDatamemberType(datamember)) && Cppyy::IsStaticDatamember(datamember)) {
-        // some implementation-specific data members have no address: ignore them
-            if (!Cppyy::GetDatamemberOffset(datamember))
-                continue;
-
+        if (Cppyy::IsEnumType(Cppyy::GetDatamemberType(datamember)) && Cppyy::IsEnumConstant(datamember)) {
         // two options: this is a static variable, or it is the enum value, the latter
         // already exists, so check for it and move on if set
             PyObject* eset = PyObject_GetAttrString(pyclass,

src/Utility.cxx

@@ -1244,13 +1244,7 @@ std::string CPyCppyy::Utility::MapOperatorName(const std::string& name, bool bTa
         if (gOpRemove.find(op) != gOpRemove.end())
             return "";
 
-    // check first if none, to prevent spurious deserializing downstream
         TC2POperatorMapping_t::iterator pop = gC2POperatorMapping.find(op);
-        if (pop == gC2POperatorMapping.end() && gOpSkip.find(op) == gOpSkip.end()) {
-            op = Cppyy::ResolveName(op);
-            pop = gC2POperatorMapping.find(op);
-        }
-
     // map C++ operator to python equivalent, or made up name if no equivalent exists
         if (pop != gC2POperatorMapping.end()) {
             return pop->second;