Kill more lingering string exceptions in Lib/plat-mac/.

collinw · collinw · commit 6cd2a2036db6 · 2007-08-30T18:11:48.000Z
diff --git a/Lib/plat-mac/FrameWork.py b/Lib/plat-mac/FrameWork.py
@@ -229,7 +229,7 @@ def dispatch(self, event):
     def asyncevents(self, onoff):
         """asyncevents - Set asynchronous event handling on or off"""
         if MacOS.runtimemodel == 'macho':
-            raise 'Unsupported in MachoPython'
+            raise NotImplementedError('Unsupported in MachoPython')
         old = self._doing_asyncevents
         if old:
             MacOS.SetEventHandler()
@@ -577,7 +577,7 @@ def additem(self, label, shortcut=None, callback=None, kind=None):
 
     def delitem(self, item):
         if item != len(self.items):
-            raise 'Can only delete last item of a menu'
+            raise ValueError('Can only delete last item of a menu')
         self.menu.DeleteMenuItem(item)
         del self.items[item-1]
 
diff --git a/Lib/plat-mac/MiniAEFrame.py b/Lib/plat-mac/MiniAEFrame.py
@@ -141,7 +141,8 @@ def callback_wrapper(self, _request, _reply):
         elif ('****', '****') in self.ae_handlers:
             _function = self.ae_handlers[('****', '****')]
         else:
-            raise 'Cannot happen: AE callback without handler', (_class, _type)
+            raise RuntimeError('AE callback without handler: '
+                               + str((_class, _type)))
 
         # XXXX Do key-to-name mapping here
 
diff --git a/Lib/plat-mac/PixMapWrapper.py b/Lib/plat-mac/PixMapWrapper.py
@@ -93,7 +93,8 @@ def _unstuff(self, element):
 
     def __setattr__(self, attr, val):
         if attr == 'baseAddr':
-            raise 'UseErr', "don't assign to .baseAddr -- assign to .data instead"
+            raise RuntimeError("don't assign to .baseAddr "
+                               "-- assign to .data instead")
         elif attr == 'data':
             self.__dict__['data'] = val
             self._stuff('baseAddr', id(self.data) + MacOS.string_id_to_buffer)
@@ -121,7 +122,7 @@ def __getattr__(self, attr):
             return self._unstuff('rowBytes') & 0x7FFF
         elif attr == 'bounds':
             # return bounds in official Left, Top, Right, Bottom order!
-            return ( \
+            return (
                 self._unstuff('left'),
                 self._unstuff('top'),
                 self._unstuff('right'),
@@ -161,7 +162,7 @@ def fromstring(self,s,width,height,format=imgformat.macrgb):
         # so convert if necessary
         if format != imgformat.macrgb and format != imgformat.macrgb16:
             # (LATER!)
-            raise "NotImplementedError", "conversion to macrgb or macrgb16"
+            raise NotImplementedError("conversion to macrgb or macrgb16")
         self.data = s
         self.bounds = (0,0,width,height)
         self.cmpCount = 3
@@ -182,7 +183,7 @@ def tostring(self, format=imgformat.macrgb):
             return self.data
         # otherwise, convert to the requested format
         # (LATER!)
-            raise "NotImplementedError", "data format conversion"
+            raise NotImplementedError("data format conversion")
 
     def fromImage(self,im):
         """Initialize this PixMap from a PIL Image object."""
