Remove unused variables:

/smtp.phpmailer.inc
Unused variable $key.
Unused variable $tmp.

/smtp.mail.inc
Unused variable $id.

CommentFileSizeAuthor
#7 smtp-n2548133-7.patch2.02 KBDamienMcKenna
#2 Removed-unused-variables-2548133-1.patch1.67 KBrrfegade
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rrfegade created an issue. See original summary.

rrfegade’s picture

rrfegade CreditAttribution: rrfegade as a volunteer commented
FileSize
Removed-unused-variables-2548133-1.patch1.67 KB

pfa the patch

rrfegade’s picture

rrfegade CreditAttribution: rrfegade as a volunteer commented
Status: Active » Needs review
wundo’s picture

wundo CreditAttribution: wundo at Chuva Inc. for Chuva Inc. commented

What did you use to generate those patchs?

rrfegade’s picture

rrfegade CreditAttribution: rrfegade as a volunteer commented

patch is created manually

wundo queued 2: Removed-unused-variables-2548133-1.patch for re-testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna as a volunteer commented
FileSize
smtp-n2548133-7.patch2.02 KB

Rerolled.

Status: Needs review » Needs work

The last submitted patch, 7: smtp-n2548133-7.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna as a volunteer commented
Status: Needs work » Needs review
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna as a volunteer commented
Assigned: rrfegade » Unassigned
wundo’s picture

wundo CreditAttribution: wundo at Chuva Inc. for Chuva Inc. commented
wundo’s picture

wundo CreditAttribution: wundo at Chuva Inc. for Chuva Inc. commented
Status: Needs review » Fixed

  • wundo committed 2a72142 on 7.x-1.x authored by DamienMcKenna 
    Issue #2548133 by rrfegade, DamienMcKenna, wundo: Remove unused varibles...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.