Replace PYWIN_ATTR_CONVERT with PyUnicode_AsUTF8

Avasam · Avasam · commit 5bcbf7f550d0 · 2025-04-30T19:10:01.000-04:00
diff --git a/Pythonwin/win32cmdui.cpp b/Pythonwin/win32cmdui.cpp
@@ -282,7 +282,7 @@ static struct PyMethodDef PyCCmdUI_methods[] = {
 
 PyObject *PyCCmdUI::getattro(PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (strcmp(name, "m_nIndex") == 0) {  // @prop int|m_nIndex|
         CCmdUI *pCU = PyCCmdUI::GetCCmdUIPtr(this);
         if (!pCU)
diff --git a/Pythonwin/win32dlg.cpp b/Pythonwin/win32dlg.cpp
@@ -171,7 +171,7 @@ PyCDialog::~PyCDialog()
 
 PyObject *PyCDialog::getattro(PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (strcmp(name, "data") == 0) {
         Py_INCREF(dddict);
         return dddict;
diff --git a/Pythonwin/win32toolbar.cpp b/Pythonwin/win32toolbar.cpp
@@ -164,7 +164,7 @@ struct MyMemberList dcmembers[] = {
 
 PyObject *PyCDockContext::getattro(PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     CDockContext *pC = GetDockContext(this);
@@ -200,7 +200,7 @@ PyObject *PyCDockContext::getattro(PyObject *obname)
 
 int PyCDockContext::setattro(PyObject *obname, PyObject *value)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
     CDockContext *pC = GetDockContext(this);
@@ -446,7 +446,7 @@ static struct PyMethodDef PyCControlBar_methods[] = {
 
 PyObject *PyCControlBar::getattro(PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     CControlBar *pCtlBar = PyCControlBar::GetControlBar(this);
@@ -477,7 +477,7 @@ PyObject *PyCControlBar::getattro(PyObject *obname)
 
 int PyCControlBar::setattro(PyObject *obname, PyObject *v)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
 
diff --git a/Pythonwin/win32util.cpp b/Pythonwin/win32util.cpp
@@ -92,7 +92,7 @@ static PySequenceMethods PyCRect_Sequence = {
 
 PyObject *PyCRect::getattro(PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     if (strcmp(name, "left") == 0)
@@ -140,7 +140,7 @@ CString PyCRect::repr()
 
 int PyCRect::setattro(PyObject *obname, PyObject *v)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
     int intval = PyLong_AsLong(v);
diff --git a/com/win32com/src/PyIBase.cpp b/com/win32com/src/PyIBase.cpp
@@ -32,14 +32,14 @@ PyObject *PyIBase::getattr(char *name) { return PyObject_GetAttrString(this, nam
 
 /*static*/ int PyIBase::setattro(PyObject *op, PyObject *obname, PyObject *v)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
     PyIBase *bc = (PyIBase *)op;
     return bc->setattr(name, v);
 }
 
-int PyIBase::setattr(char *name, PyObject *v)
+int PyIBase::setattr(const char *name, PyObject *v)
 {
     char buf[128];
     sprintf(buf, "%s has read-only attributes", ob_type->tp_name);
diff --git a/com/win32com/src/PyRecord.cpp b/com/win32com/src/PyRecord.cpp
@@ -590,7 +590,7 @@ PyObject *PyRecord::getattro(PyObject *self, PyObject *obname)
     PyRecord *pyrec = (PyRecord *)self;
     GUID structguid;
     OLECHAR *guidString;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     if (strcmp(name, "__record_type_guid__") == 0) {
diff --git a/com/win32com/src/extensions/PySTGMEDIUM.cpp b/com/win32com/src/extensions/PySTGMEDIUM.cpp
@@ -230,7 +230,7 @@ void PySTGMEDIUM::Close()
 
 PyObject *PySTGMEDIUM::getattro(PyObject *self, PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
 
diff --git a/com/win32com/src/include/PythonCOM.h b/com/win32com/src/include/PythonCOM.h
@@ -168,7 +168,7 @@ class PYCOM_EXPORT PyIBase : public PyObject {
    public:
     // virtuals for Python support
     virtual PyObject *getattr(char *name);
-    virtual int setattr(char *name, PyObject *v);
+    virtual int setattr(const char *name, PyObject *v);
     virtual PyObject *repr();
     virtual int compare(PyObject *other)
     {
diff --git a/com/win32comext/adsi/src/PyADSIUtil.cpp b/com/win32comext/adsi/src/PyADSIUtil.cpp
@@ -449,7 +449,7 @@ class PyADS_ATTR_INFO : public PyObject {
 
     static PyObject *getattro(PyObject *self, PyObject *obname)
     {
-        char *name = PYWIN_ATTR_CONVERT(obname);
+        const char *name = PyUnicode_AsUTF8(obname);
         if (name == NULL)
             return NULL;
         if (strcmp(name, "__members__") == 0) {
diff --git a/com/win32comext/adsi/src/PyIADs.cpp b/com/win32comext/adsi/src/PyIADs.cpp
@@ -247,7 +247,7 @@ static struct PyMethodDef PyIADs_methods[] = {
 
 PyObject *PyIADs_getattro(PyObject *ob, PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (!name)
         return NULL;
 
diff --git a/com/win32comext/directsound/src/PyDSBUFFERDESC.cpp b/com/win32comext/directsound/src/PyDSBUFFERDESC.cpp
@@ -103,8 +103,8 @@ PyTypeObject PyDSBUFFERDESCType = {
     // IDirectSound::CreateSoundBuffer call. With this flag set, an application using DirectSound can continue to play
     // its sticky focus buffers if the user switches to another application not using DirectSound. In this situation,
     // the application's normal buffers are muted, but the sticky focus buffers are still audible. This is useful for
-    // nongame applications, such as movie playback (DirectShow�), when the user wants to hear the soundtrack while
-    // typing in Microsoft Word or Microsoft� Excel, for example. However, if the user switches to another DirectSound
+    // nongame applications, such as movie playback (DirectShow™), when the user wants to hear the soundtrack while
+    // typing in Microsoft Word or Microsoft® Excel, for example. However, if the user switches to another DirectSound
     // application, all sound buffers, both normal and sticky focus, in the previous application are muted.
     // @flag DSBCAPS_GLOBALFOCUS|The buffer is a global sound buffer. With this flag set, an application using
     // DirectSound can continue to play its buffers if the user switches focus to another application, even if the new
@@ -167,7 +167,7 @@ PyDSBUFFERDESC::~PyDSBUFFERDESC() { Py_XDECREF(m_obWFX); }
 int PyDSBUFFERDESC::setattro(PyObject *self, PyObject *obname, PyObject *obvalue)
 {
     PyDSBUFFERDESC *obself = (PyDSBUFFERDESC *)self;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
 
diff --git a/com/win32comext/directsound/src/PyDSCBUFFERDESC.cpp b/com/win32comext/directsound/src/PyDSCBUFFERDESC.cpp
@@ -135,7 +135,7 @@ PyDSCBUFFERDESC::~PyDSCBUFFERDESC() { Py_XDECREF(m_obWFX); }
 int PyDSCBUFFERDESC::setattro(PyObject *self, PyObject *obname, PyObject *obvalue)
 {
     PyDSCBUFFERDESC *obself = (PyDSCBUFFERDESC *)self;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
 
     if (name == NULL)
         return -1;
diff --git a/win32/src/PyOVERLAPPED.cpp b/win32/src/PyOVERLAPPED.cpp
@@ -110,7 +110,7 @@ PYWINTYPES_EXPORT PyTypeObject PyOVERLAPPEDType = {
     {"Offset", T_ULONG,
      OFF(m_overlapped.Offset)},  // @prop integer|Offset|Specifies a file position at which to start the transfer. The
                                  // file position is a byte offset from the start of the file. The calling process sets
-                                 // this member before calling the ReadFile�or WriteFile function. This member is
+                                 // this member before calling the ReadFile or WriteFile function. This member is
                                  // ignored when reading from or writing to named pipes and communications devices.
     {"OffsetHigh", T_ULONG, OFF(m_overlapped.OffsetHigh)},  // @prop integer|OffsetHigh|Specifies the high word of the
                                                             // byte offset at which to start the transfer.
@@ -123,7 +123,7 @@ PYWINTYPES_EXPORT PyTypeObject PyOVERLAPPEDType = {
     {"hEvent", T_OBJECT, OFF(obDummy)},  // @prop <o PyHANDLE>|hEvent|Identifies an event set to the signaled state when
                                          // the transfer has been completed. The calling process sets this member before
                                          // calling the <om win32file.ReadFile>, <om win32file.WriteFile>, <om
-                                         // win32pipe.ConnectNamedPipe>, or <om win32pipe.TransactNamedPipe>�function.
+                                         // win32pipe.ConnectNamedPipe>, or <om win32pipe.TransactNamedPipe> function.
     {"Internal", T_OBJECT,
      OFF(obDummy)},  // @prop integer|Internal|Reserved for operating system use. (pointer-sized value)
     {"InternalHigh", T_OBJECT,
@@ -182,7 +182,7 @@ PyObject *PyOVERLAPPED::richcompareFunc(PyObject *ob, PyObject *other, int op)
 
 PyObject *PyOVERLAPPED::getattro(PyObject *self, PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     if (strcmp("hEvent", name) == 0) {
@@ -210,7 +210,7 @@ int PyOVERLAPPED::setattro(PyObject *self, PyObject *obname, PyObject *v)
         PyErr_SetString(PyExc_AttributeError, "can't delete OVERLAPPED attributes");
         return -1;
     }
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     if (strcmp("hEvent", name) == 0) {
diff --git a/win32/src/PySECURITY_ATTRIBUTES.cpp b/win32/src/PySECURITY_ATTRIBUTES.cpp
@@ -157,7 +157,7 @@ int PySECURITY_ATTRIBUTES::setattro(PyObject *self, PyObject *obname, PyObject *
         PyErr_SetString(PyExc_AttributeError, "can't delete SECURITY_ATTRIBUTES attributes");
         return -1;
     }
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
     if (strcmp(name, "SECURITY_DESCRIPTOR") == 0) {
diff --git a/win32/src/PyWinTypes.h b/win32/src/PyWinTypes.h
@@ -52,9 +52,6 @@
 // Macro to handle PyObject layout changes in Py3k
 #define PYWIN_OBJECT_HEAD PyVarObject_HEAD_INIT(NULL, 0)
 
-// This macro is kept for legacy reason as it's casting from `const char *` to `char *`
-#define PYWIN_ATTR_CONVERT (char *)PyUnicode_AsUTF8
-
 typedef Py_ssize_t Py_hash_t;
 
 // This only enables runtime checks in debug builds - so we use
diff --git a/win32/src/win32api_display.cpp b/win32/src/win32api_display.cpp
@@ -125,7 +125,7 @@ PyObject *PyDISPLAY_DEVICE::Clear(PyObject *self, PyObject *args)
 PyObject *PyDISPLAY_DEVICE::getattro(PyObject *self, PyObject *obname)
 {
     PDISPLAY_DEVICE pdisplay_device = &((PyDISPLAY_DEVICE *)self)->display_device;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
 
@@ -162,7 +162,7 @@ PyObject *PyDISPLAY_DEVICE::getattro(PyObject *self, PyObject *obname)
 
 int PyDISPLAY_DEVICE::setattro(PyObject *self, PyObject *obname, PyObject *obvalue)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
 
diff --git a/win32/src/win32consolemodule.cpp b/win32/src/win32consolemodule.cpp
@@ -466,7 +466,7 @@ struct PyMemberDef PyINPUT_RECORD::members[] = {
 PyObject *PyINPUT_RECORD::tp_getattro(PyObject *self, PyObject *obname)
 {
     INPUT_RECORD *pir = &((PyINPUT_RECORD *)self)->input_record;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     if (strcmp(name, "ControlKeyState") == 0) {
@@ -514,8 +514,7 @@ PyObject *PyINPUT_RECORD::tp_getattro(PyObject *self, PyObject *obname)
 int PyINPUT_RECORD::tp_setattro(PyObject *self, PyObject *obname, PyObject *obvalue)
 {
     INPUT_RECORD *pir = &((PyINPUT_RECORD *)self)->input_record;
-    char *name;
-    name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return -1;
     if (obvalue == NULL) {
diff --git a/win32/src/win32crypt/PyCERT_CONTEXT.cpp b/win32/src/win32crypt/PyCERT_CONTEXT.cpp
@@ -141,7 +141,7 @@ PyObject *PyWinObject_FromCERT_EXTENSIONArray(PCERT_EXTENSION pce, DWORD ext_cnt
 
 PyObject *PyCERT_CONTEXT::getattro(PyObject *self, PyObject *obname)
 {
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name == NULL)
         return NULL;
     PCCERT_CONTEXT pcc = ((PyCERT_CONTEXT *)self)->GetPCCERT_CONTEXT();
diff --git a/win32/src/win32crypt/PyCRYPTKEY.cpp b/win32/src/win32crypt/PyCRYPTKEY.cpp
@@ -62,7 +62,7 @@ int PyCRYPTKEY::setattro(PyObject *self, PyObject *obname, PyObject *v)
 PyObject *PyCRYPTKEY::getattro(PyObject *self, PyObject *obname)
 {
     /*
-    char *name=PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name==NULL)
         return NULL;
     if (strcmp(name,"HCRYPTKEY")==0){
diff --git a/win32/src/win32crypt/PyCRYPTMSG.cpp b/win32/src/win32crypt/PyCRYPTMSG.cpp
@@ -50,7 +50,7 @@ int PyCRYPTMSG::setattro(PyObject *self, PyObject *obname, PyObject *v)
 PyObject *PyCRYPTMSG::getattro(PyObject *self, PyObject *obname)
 {
     /*
-    char *name=PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (name==NULL)
         return NULL;
     if (strcmp(name,"HCRYPTMSG")==0){
diff --git a/win32/src/win32file_comm.cpp b/win32/src/win32file_comm.cpp
@@ -171,7 +171,7 @@ PyDCB::~PyDCB(void) {}
 PyObject *PyDCB::getattro(PyObject *self, PyObject *obname)
 {
     PyDCB *pydcb = (PyDCB *)self;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (!name)
         return NULL;
 
@@ -212,7 +212,7 @@ int PyDCB::setattro(PyObject *self, PyObject *obname, PyObject *v)
         PyErr_SetString(PyExc_AttributeError, "can't delete DCB attributes");
         return -1;
     }
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (!name)
         return -1;
     SET_BITFIELD_ENTRY(fBinary)
@@ -373,7 +373,7 @@ PyCOMSTAT::~PyCOMSTAT(void) {}
 PyObject *PyCOMSTAT::getattro(PyObject *self, PyObject *obname)
 {
     PyCOMSTAT *pyCOMSTAT = (PyCOMSTAT *)self;
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (!name)
         return NULL;
     if (0)  // boot up our macro magic (the macro starts with an 'else')
@@ -408,7 +408,7 @@ int PyCOMSTAT::setattro(PyObject *self, PyObject *obname, PyObject *v)
         PyErr_SetString(PyExc_AttributeError, "can't delete COMSTAT attributes");
         return -1;
     }
-    char *name = PYWIN_ATTR_CONVERT(obname);
+    const char *name = PyUnicode_AsUTF8(obname);
     if (!name)
         return -1;
     SET_BITFIELD_ENTRY(fCtsHold)
diff --git a/win32/src/win32gui.i b/win32/src/win32gui.i
@@ -921,7 +921,7 @@ PyWNDCLASS::~PyWNDCLASS(void)
 
 PyObject *PyWNDCLASS::getattro(PyObject *self, PyObject *obname)
 {
-	char *name=PYWIN_ATTR_CONVERT(obname);
+	const char *name = PyUnicode_AsUTF8(obname);
 	if (name==NULL)
 		return NULL;
 	PyWNDCLASS *pW = (PyWNDCLASS *)self;
@@ -976,7 +976,7 @@ int PyWNDCLASS::setattro(PyObject *self, PyObject *obname, PyObject *v)
 		PyErr_SetString(PyExc_AttributeError, "can't delete WNDCLASS attributes");
 		return -1;
 	}
-	char *name=PYWIN_ATTR_CONVERT(obname);
+	const char *name = PyUnicode_AsUTF8(obname);
 	if (name==NULL)
 		return -1;
 	PyWNDCLASS *pW = (PyWNDCLASS *)self;
@@ -1128,7 +1128,7 @@ PyBITMAP::~PyBITMAP(void)
 
 PyObject *PyBITMAP::getattro(PyObject *self, PyObject *obname)
 {
-	char *name=PYWIN_ATTR_CONVERT(obname);
+	const char *name = PyUnicode_AsUTF8(obname);
 	if (name==NULL)
 		return NULL;
 	PyBITMAP *pB = (PyBITMAP *)self;
@@ -1144,7 +1144,7 @@ int PyBITMAP::setattro(PyObject *self, PyObject *obname, PyObject *v)
 		PyErr_SetString(PyExc_AttributeError, "can't delete BITMAP attributes");
 		return -1;
 	}
-	char *name=PYWIN_ATTR_CONVERT(obname);
+	const char *name = PyUnicode_AsUTF8(obname);
 	if (name==NULL)
 		return -1;
 	if (strcmp("bmBits", name)==0) {
@@ -1268,7 +1268,7 @@ PyLOGFONT::~PyLOGFONT(void)
 
 PyObject *PyLOGFONT::getattro(PyObject *self, PyObject *obname)
 {
-	char *name=PYWIN_ATTR_CONVERT(obname);
+	const char *name = PyUnicode_AsUTF8(obname);
 	if (name==NULL)
 		return NULL;
 	PyLOGFONT *pL = (PyLOGFONT *)self;
@@ -1284,7 +1284,7 @@ int PyLOGFONT::setattro(PyObject *self, PyObject *obname, PyObject *v)
 		PyErr_SetString(PyExc_AttributeError, "can't delete LOGFONT attributes");
 		return -1;
 	}
-	char *name=PYWIN_ATTR_CONVERT(obname);
+	const char *name = PyUnicode_AsUTF8(obname);
 	if (name==NULL)
 		return -1;
 	if (strcmp("lfFaceName", name)==0) {
diff --git a/win32/src/win32helpmodule.cpp b/win32/src/win32helpmodule.cpp
diff --git a/win32/src/win32process.i b/win32/src/win32process.i
diff --git a/win32/src/win32rasmodule.cpp b/win32/src/win32rasmodule.cpp
diff --git a/win32/src/win32security_sspi.cpp b/win32/src/win32security_sspi.cpp
diff --git a/win32/src/win32wnet/PyNCB.cpp b/win32/src/win32wnet/PyNCB.cpp
diff --git a/win32/src/win32wnet/PyNetresource.cpp b/win32/src/win32wnet/PyNetresource.cpp

Original file line number	Diff line number	Diff line change
`@@ -282,7 +282,7 @@ static struct PyMethodDef PyCCmdUI_methods[] = {`
`282`	`282`
`283`	`283`	`PyObject PyCCmdUI::getattro(PyObject obname)`
`284`	`284`	`{`
`285`		`- char *name = PYWIN_ATTR_CONVERT(obname);`
	`285`	`+ const char *name = PyUnicode_AsUTF8(obname);`
`286`	`286`	`if (strcmp(name, "m_nIndex") == 0) { // @prop int\|m_nIndex\|`
`287`	`287`	`CCmdUI *pCU = PyCCmdUI::GetCCmdUIPtr(this);`
`288`	`288`	`if (!pCU)`
Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,7 @@ PyCDialog::~PyCDialog()`
`171`	`171`
`172`	`172`	`PyObject PyCDialog::getattro(PyObject obname)`
`173`	`173`	`{`
`174`		`- char *name = PYWIN_ATTR_CONVERT(obname);`
	`174`	`+ const char *name = PyUnicode_AsUTF8(obname);`
`175`	`175`	`if (strcmp(name, "data") == 0) {`
`176`	`176`	`Py_INCREF(dddict);`
`177`	`177`	`return dddict;`
Original file line number	Diff line number	Diff line change
`@@ -32,14 +32,14 @@ PyObject PyIBase::getattr(char name) { return PyObject_GetAttrString(this, nam`
`32`	`32`
`33`	`33`	`/static/ int PyIBase::setattro(PyObject op, PyObject obname, PyObject *v)`
`34`	`34`	`{`
`35`		`- char *name = PYWIN_ATTR_CONVERT(obname);`
	`35`	`+ const char *name = PyUnicode_AsUTF8(obname);`
`36`	`36`	`if (name == NULL)`
`37`	`37`	`return -1;`
`38`	`38`	`PyIBase bc = (PyIBase )op;`
`39`	`39`	`return bc->setattr(name, v);`
`40`	`40`	`}`
`41`	`41`
`42`		`-int PyIBase::setattr(char name, PyObject v)`
	`42`	`+int PyIBase::setattr(const char name, PyObject v)`
`43`	`43`	`{`
`44`	`44`	`char buf[128];`
`45`	`45`	`sprintf(buf, "%s has read-only attributes", ob_type->tp_name);`
Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,7 @@ void PySTGMEDIUM::Close()`
`230`	`230`
`231`	`231`	`PyObject PySTGMEDIUM::getattro(PyObject self, PyObject *obname)`
`232`	`232`	`{`
`233`		`- char *name = PYWIN_ATTR_CONVERT(obname);`
	`233`	`+ const char *name = PyUnicode_AsUTF8(obname);`
`234`	`234`	`if (name == NULL)`
`235`	`235`	`return NULL;`
`236`	`236`
Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ class PYCOM_EXPORT PyIBase : public PyObject {`
`168`	`168`	`public:`
`169`	`169`	`// virtuals for Python support`
`170`	`170`	`virtual PyObject getattr(char name);`
`171`		`- virtual int setattr(char name, PyObject v);`
	`171`	`+ virtual int setattr(const char name, PyObject v);`
`172`	`172`	`virtual PyObject *repr();`
`173`	`173`	`virtual int compare(PyObject *other)`
`174`	`174`	`{`
Original file line number	Diff line number	Diff line change
`@@ -449,7 +449,7 @@ class PyADS_ATTR_INFO : public PyObject {`
`449`	`449`
`450`	`450`	`static PyObject getattro(PyObject self, PyObject *obname)`
`451`	`451`	`{`
`452`		`- char *name = PYWIN_ATTR_CONVERT(obname);`
	`452`	`+ const char *name = PyUnicode_AsUTF8(obname);`
`453`	`453`	`if (name == NULL)`
`454`	`454`	`return NULL;`
`455`	`455`	`if (strcmp(name, "__members__") == 0) {`
Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,7 @@ static struct PyMethodDef PyIADs_methods[] = {`
`247`	`247`
`248`	`248`	`PyObject PyIADs_getattro(PyObject ob, PyObject *obname)`
`249`	`249`	`{`
`250`		`- char *name = PYWIN_ATTR_CONVERT(obname);`
	`250`	`+ const char *name = PyUnicode_AsUTF8(obname);`
`251`	`251`	`if (!name)`
`252`	`252`	`return NULL;`
`253`	`253`