On Saturday, November 26, 2016 at 8:04:20 AM UTC-6, Edward K. Ream wrote:
 
>
> In short, we are getting to matters of style.  Imo, this:
>
>     return [self.lstrip_line(z) for z in lines]
>
> is slightly preferable to:
>
>     return ['\n' if z.isspace() else z.lstrip() for z in lines]
>
>
Furthermore, i.clean_blank_lines() uses i.lstrip_line():

def clean_blank_lines(self, lines):
    '''Remove all blanks and tabs in all blank lines.'''
    return [self.lstrip_line(z) if z.isspace() else z for z in lines]

Yes, we could replace the call to self.lstrip_line, but that would create 
nested ternary operators, which I avoid on stylistic grounds.

EKR

-- 
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to leo-editor+unsubscr...@googlegroups.com.
To post to this group, send email to leo-editor@googlegroups.com.
Visit this group at https://groups.google.com/group/leo-editor.
For more options, visit https://groups.google.com/d/optout.

Reply via email to