Simplify the code by removing the } else { after a break/return/continue

RESOLVED FIXED in 4.16

Status

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: sylvestre, Assigned: sylvestre)

Tracking

(Blocks 1 bug)

other
4.16
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Comment 1

2 years ago
Posted patch foo.diffSplinter Review
Attachment #8834509 - Flags: review?(kaie)
Assignee

Updated

2 years ago
Attachment #8834509 - Flags: review?(kaie) → review?(ted)
Comment on attachment 8834509 [details] [diff] [review]
foo.diff

I have resigned as NSPR module owner. Sorry for the inconvenience.
Attachment #8834509 - Flags: review?(ted)
Assignee

Comment 3

2 years ago
Comment on attachment 8834509 [details] [diff] [review]
foo.diff

np, trying kaie
(I will probably have to rebase the patch)
Attachment #8834509 - Flags: review?(kaie)
Comment on attachment 8834509 [details] [diff] [review]
foo.diff

Patch still applies cleanly.

r=kaie

I will check it in in a couple of days, we're currently frozen for the NSPR 4.15 release.
Attachment #8834509 - Flags: review?(kaie) → review+

Updated

2 years ago
Flags: needinfo?(kaie)
Keywords: checkin-needed
https://hg.mozilla.org/projects/nspr/rev/85977b5eba80

Let's set the target milestone to NSPR version 4.16, once bug 1370873 is done.
Status: NEW → RESOLVED
Closed: 2 years ago
Depends on: 1370873
Flags: needinfo?(kaie)
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

2 years ago
Target Milestone: --- → 4.16
You need to log in before you can comment on or make changes to this bug.