Opened 5 years ago

Closed 5 years ago

#5707 task closed fixed (fixed)

Remove uses of apply() where covered by tests

Reported by: Vladimir Perić Owned by: Vladimir Perić
Priority: low Milestone: Python-3.x
Component: core Keywords: py3k
Cc: Branch: branches/apply-removal-5707
branch-diff, diff-cov, branch-cov, buildbot
Author: vperic

Description

The apply() built-in is deprecated since 2.3 and has been removed in Python 3. The solution is to just call the function directly. It's used in a handful of places in Twisted, this ticket only covers those which are covered by tests.

Change History (4)

comment:1 Changed 5 years ago by Vladimir Perić

Branch: branches/apply-removal-5707

(In [34528]) Create branch apply-removal-5707

comment:2 Changed 5 years ago by Vladimir Perić

Keywords: review added

Submitting for review, see also Build results (but I don't see how this could cause errors)

comment:3 Changed 5 years ago by Itamar Turner-Trauring

Keywords: review removed
Owner: set to Vladimir Perić

Looks good, merge.

comment:4 Changed 5 years ago by Vladimir Perić

Resolution: fixed
Status: newclosed

(In [34534]) Merge apply-removal-5707: Remove uses of apply()

Author: vperic Reviewer: itamar Fixes: #5707

The apply() built-in has been deprecated since 2.3 and removed in Python 3. Instead, just call the function directly. This covers the uses of apply() covered by tests.

Note: See TracTickets for help on using tickets.