Opened 4 years ago

Closed 4 years ago

#16472 closed Cleanup/optimization (fixed)

cleaning django.db.models.query.iterator code

Reported by: fva@… Owned by: nobody
Component: Database layer (models, ORM) Version: 1.3
Severity: Normal Keywords: query iterator
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

removed unused variables pk_val and pk_idx

version 1.4 pre-alpha SVN-16544

Attachments (1)

clean.diff (1.3 KB) - added by fva@… 4 years ago.
trunk diff

Download all attachments as: .zip

Change History (3)

Changed 4 years ago by fva@…

trunk diff

comment:1 Changed 4 years ago by aaugustin

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

The patch applies cleanly, a review of the source code indicates that these variables are indeed unused, and the entire test suite passes. Good to go.

comment:2 Changed 4 years ago by jezdez

  • Resolution set to fixed
  • Status changed from new to closed

In [16562]:

Fixed #16472 -- Removed stale code in the query code. Thanks, fva@….

Note: See TracTickets for help on using tickets.
Back to Top