Pass code check.

examples/multiping.py

@@ -30,10 +30,9 @@ def startpings( host, targetips ):
     # BL: Not sure why loopback intf isn't up!
     host.cmd( 'ifconfig lo up' )
 
-    
     # Simple ping loop
     cmd = ( 'while true; do '
-            ' for ip in %s; do ' % targetips  +
+            ' for ip in %s; do ' % targetips +
             '  echo -n %s "->" $ip ' % host.IP() +
             '   `ping -c1 -w 1 $ip | grep packets` ;'
             '  sleep 1;'
@@ -85,10 +84,3 @@ def multiping( netsize, chunksize, seconds):
 if __name__ == '__main__':
     setLogLevel( 'info' )
     multiping( netsize=20, chunksize=4, seconds=10 )
-
-    
-    
-
-    
-    
-    

examples/multipoll.py

@@ -8,13 +8,13 @@ monitoring them
 from mininet.topo import SingleSwitchTopo
 from mininet.net import Mininet
 from mininet.log import setLogLevel
-from mininet.util import quietRun
 
-from time import time, sleep
+from time import time
 from select import poll, POLLIN
 from subprocess import Popen, PIPE
 
 def monitorFiles( outfiles, seconds, timeoutms ):
+    "Monitor set of files and return [(host, line)...]"
     devnull = open( '/dev/null', 'w' )
     tails, fdToFile, fdToHost = {}, {}, {}
     for h, outfile in outfiles.iteritems():
@@ -79,5 +79,3 @@ def monitorTest( N=3, seconds=3 ):
 if __name__ == '__main__':
     setLogLevel('info')
     monitorTest()
-
-    

examples/simpleperf.py

@@ -12,34 +12,33 @@ from mininet.util import dumpNodeConnections
 from mininet.log import setLogLevel
 
 class SingleSwitchTopo(Topo):
-   "Single switch connected to n hosts."
-   def __init__(self, n=2, **opts):
-       Topo.__init__(self, **opts)
-       switch = self.add_switch('s1')
-       for h in range(n):
-        # Each host gets 50%/n of system CPU
-           host = self.add_host('h%s' % (h + 1),
-            cpu=.5/n)
-        # 10 Mbps, 5ms delay, 10% loss
-           self.add_link(host, switch,
-            bw=10, delay='5ms', loss=10, use_htb=True)
+    "Single switch connected to n hosts."
+    def __init__(self, n=2, **opts):
+        Topo.__init__(self, **opts)
+        switch = self.add_switch('s1')
+        for h in range(n):
+            # Each host gets 50%/n of system CPU
+            host = self.add_host('h%s' % (h + 1),
+                                 cpu=.5 / n)
+            # 10 Mbps, 5ms delay, 10% loss
+            self.add_link(host, switch,
+                          bw=10, delay='5ms', loss=10, use_htb=True)
 
 def perfTest():
-   "Create network and run simple performance test"
-   topo = SingleSwitchTopo(n=4)
-   net = Mininet(topo=topo,
-                 host=CPULimitedHost, link=TCLink)
-   net.start()
-   print "Dumping host connections"
-   dumpNodeConnections(net.hosts)
-   print "Testing network connectivity"
-   net.pingAll()
-   print "Testing bandwidth between h1 and h4"
-   h1 = net.getNodeByName('h1')
-   h4 = net.getNodeByName('h4')
-   net.iperf((h1, h4))
-   net.stop()
+    "Create network and run simple performance test"
+    topo = SingleSwitchTopo(n=4)
+    net = Mininet(topo=topo,
+                  host=CPULimitedHost, link=TCLink)
+    net.start()
+    print "Dumping host connections"
+    dumpNodeConnections(net.hosts)
+    print "Testing network connectivity"
+    net.pingAll()
+    print "Testing bandwidth between h1 and h4"
+    h1, h4 = net.getNodeByName('h1', 'h4')
+    net.iperf((h1, h4))
+    net.stop()
 
 if __name__ == '__main__':
-   setLogLevel('info')
-   perfTest()
+    setLogLevel('info')
+    perfTest()