John, Thanks. I have deleted it.
Eric John Hunter wrote: >>>>>> "Eric" == Eric Firing <[EMAIL PROTECTED]> writes: > > Eric> Despite the comment, I don't understand the purpose of the > Eric> last line in the following excerpt from the Axes.plot() > Eric> method: > > Eric> lines = [] for line in self._get_lines(*args, > Eric> **kwargs): self.add_line(line) lines.append(line) lines = > Eric> [line for line in lines] # consume the generator > > That looks like detritus. If I recall correctly, it used to be > something like > > lines = self._get_lines(*args, **kwargs): > lines = [line for line in lines] # consume the generator > > _get_lines is a generator and there was some part of the code that was > having trouble with this so I just converted it to a list and that > fixed the bug. Later it appears the code was rewritten to what you > posted and lines is already a list, so yes, the list comprehension is > redundant and should be removed. > > JDH ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
