Closed Bug 1412769 Opened 2 years ago Closed 2 years ago

nsSmtpProtocol::SendRecipientResponse(): Identical code for different branches

Categories

(MailNews Core :: Networking: SMTP, enhancement)

enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 58.0

People

(Reporter: Sylvestre, Assigned: Sylvestre)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, Whiteboard: [CID 1420380])

Attachments

(1 file)

No description provided.
Attached patch aze.diffSplinter Review
Attachment #8923298 - Flags: review?(jorgk)
Comment on attachment 8923298 [details] [diff] [review]
aze.diff

Nice find ;-) How did you find this? Some linting or clang trick?

Please let me land this for you!! I'm coordinating our landings with M-C merges.
Attachment #8923298 - Flags: review?(jorgk) → review+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/55b85200c486
nsSmtpProtocol::SendRecipientResponse(): Remove identical code for if/else branches. r=jorgk
Status: NEW → RESOLVED
Closed: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 58.0
You need to log in before you can comment on or make changes to this bug.