Changeset 1145:4904633ee0ec

Show
Ignore:
Timestamp:
04/29/08 12:02:04 (2 months ago)
Author:
Ronny Pfannschmidt <Ronny.Pfannschmidt@…>
Message:

some code cleanups

Files:
6 modified

Legend:

Unmodified
Added
Removed

  • pida-plugins/man/man.py

    r884 r1145  
    162162 
    163163        # don't make empty search 
    164         if len(pattern) <= 0: 
     164        if not pattern: 
    165165            return 
    166166 

  • pida/core/options.py

    r412 r1145  
    177177        return len(self._options) 
    178178 
    179     def iter_options(self): 
    180         return self._options.values() 
     179    def __iter__(self): 
     180        return self._options.itervalues() 
     181     
     182    def __nonzero__(self): 
     183        return bool(self._options) 
    181184 
    182  

  • pida/core/projects.py

    r1142 r1145  
    102102        default = self.get_default_controller() 
    103103        if default is None: 
    104             if len(self.controllers): 
     104            if self.controllers: 
    105105                self.controllers[0].default = True 
    106106        self.save() 

  • pida/services/optionsmanager/optionsmanager.py

    r1144 r1145  
    7474        self._services = [] 
    7575        for svc in self.svc.boss.get_services(): 
    76             if len(svc.options): 
     76            if svc.options: 
    7777                self._services.append(svc) 
    7878                self._services_display.append( 
    … …  
    109109        labelsizer = gtk.SizeGroup(gtk.SIZE_GROUP_HORIZONTAL) 
    110110        widgetsizer = gtk.SizeGroup(gtk.SIZE_GROUP_HORIZONTAL) 
    111         options = list(svc.options.iter_options()) 
    112         options.sort() 
    113         for opt in options: 
     111 
     112        for opt in sorted(svc.options): 
    114113            vb = gtk.VBox(spacing=2) 
    115114            vb.set_border_width(6) 
    … …  
    192191 
    193192    def plugin_changed(self, plugin): 
    194         if len(plugin.options): 
     193        if plugin.options: 
    195194            self.svc.refresh_view() 
    196195 

  • pida/services/sessions/sessions.py

    r1141 r1145  
    304304        load each file in self.buffers into the buffer manager 
    305305        """ 
    306         if len(files): 
     306        if files: 
    307307            self.boss.cmd('buffer', 'open_file', file_name=files.pop()) 
    308308        else: 

  • tests/core/test_base.py

    r108 r1145  
    1010 
    1111class BaseConfigTest(TestCase): 
    12  
    13     def setUp(self): 
    14         self._svc = Mock({'get_name': 'banana'}) 
    15         self._conf = TestConfig(self._svc) 
    16  
    17     def test_get_name(self): 
    18         self.assertEqual(self._conf.get_service_name(), 'banana') 
    19      
     12    pass