Avoid using lower case l as a variable name.

peterjc · peterjc · commit 7deaebeed4d1 · 2017-10-24T16:09:40.000+01:00
This fixes new error in flake v3.5.0 from pccodestyle 2.1.0, $ flake8 Scripts/ Scripts/xbbtools/nextorf.py:102:9: E741 ambiguous variable name 'l' Scripts/xbbtools/xbb_widget.py:214:9: E741 ambiguous variable name 'l' Rational is to avoid the characters 'l', 'O', or 'I' as variable names because in some fonts they are indistinguishable from the numerals one and zero. Closes biopython#1422
diff --git a/Scripts/xbbtools/nextorf.py b/Scripts/xbbtools/nextorf.py
@@ -99,12 +99,12 @@ def Gc(self, seq):
         return round(gc * 100.0 / (d['A'] + d['T'] + gc), 1)
 
     def Gc2(self, seq):
-        l = len(seq)
+        length = len(seq)
         d = {}
         for nt in ['A', 'T', 'G', 'C']:
             d[nt] = [0, 0, 0]
 
-        for i in range(0, l, 3):
+        for i in range(0, length, 3):
             codon = seq[i:i + 3]
             if len(codon) < 3:
                 codon += '  '
diff --git a/Scripts/xbbtools/xbb_widget.py b/Scripts/xbbtools/xbb_widget.py
@@ -210,15 +210,15 @@ def create_buttons(self, parent):
             b_id.pack(side='top', pady=5, padx=10)
             self.buttons[text] = b_id
 
-        f = ttk.Frame(self.button_frame)
-        l = ttk.Label(f, text='Goto:')
-        l.pack(side='left')
-        l.bind('<Button-1>', self.goto)
+        frame = ttk.Frame(self.button_frame)
+        label = ttk.Label(frame, text='Goto:')
+        label.pack(side='left')
+        label.bind('<Button-1>', self.goto)
 
-        self.goto_entry = ttk.Entry(f, width=5)
+        self.goto_entry = ttk.Entry(frame, width=5)
         self.goto_entry.pack(side='right', pady=5, padx=4)
         self.goto_entry.bind('<Return>', self.goto)
-        f.pack(side='bottom')
+        frame.pack(side='bottom')
 
     def create_seqfield(self, parent):
         self.sequence_id = tk.Text(parent, wrap='char', width=self.seqwidth)
