Edgewall Software
Modify

Opened 13 years ago

Closed 13 years ago

#748 closed defect (wontfix)

`.readlines()` missing in pylint?

Reported by: debacle@… Owned by:
Priority: major Milestone:
Component: Recipe commands Version: 0.6
Keywords: Cc:
Operating System: Linux

Description

I believe, in trunk/bitten/build/pythontools.py is missing a .readlines():

  • trunk/bitten/build/pythontools.py

    old new  
    156156    try:
    157157        fd = open(ctxt.resolve(file_), 'r')
    158158        try:
    159             for line in fd:
     159            for line in fd.readlines():
    160160                match = msg_re.search(line)
    161161                if match:
    162162                    msg_type = match.group('type')

Attachments (0)

Change History (2)

comment:1 follow-up: Changed 13 years ago by debacle@…

Sorry, my test was wrong. Iterating over the file is as good as using .readlines. Please close.

comment:2 in reply to: ↑ 1 Changed 13 years ago by osimons

  • Milestone 0.6.2 deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Replying to debacle@…:

Please close.

Ok.

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The ticket will remain with no owner.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.