Closed Bug 1158547 Opened 6 years ago Closed 6 years ago

Useless declaration in Parser.cpp


(Core :: JavaScript Engine, defect)

Not set



Tracking Status
firefox40 --- fixed


(Reporter: Sylvestre, Assigned: rnath, Mentored)


(Blocks 1 open bug)


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


(2 files)

forStmt.type = STMT_FOR_OF_LOOP;
is useless as override the line after.
Ryan, here it is if you want a good first bug (super easy)
Flags: needinfo?(nathmatics)
This patch removes the useless declaration.
Flags: needinfo?(nathmatics)
Attachment #8597668 - Flags: review?(sledru)
Ryan, I am not the owner of this code. You should find out who is maintaining it (an easy way is to look at the history of file).
Assignee: nobody → nathmatics
Attachment #8597668 - Flags: review?(sledru)
Attachment #8597668 - Flags: review?(jorendorff)
Comment on attachment 8597668 [details] [diff] [review]
rev1 - removal of useless declaration

Review of attachment 8597668 [details] [diff] [review]:


Attachment #8597668 - Flags: review?(jorendorff) → review+
You probably want to add "checkin-needed" to get someone to check this patch in.

But first, you need to add a commit message to your patch.  See for how to do that, and see for tips on a good commit message.
Flags: needinfo?(nathmatics)
(In reply to Daniel Holbert [:dholbert] from comment #5)
> You probably want to add "checkin-needed" to get someone to check this patch
> in.

(Specifically: once you've reposted the patch with a good commit message, you'll want to add "checkin-needed" to the "Keywords" field at the top of this bug.)

(Normally, the Tree sheriffs require that patches receive a TryServer* test before using "checkin-needed"; however, in this case, I'll assert that this is unnecessary, because the patch clearly has no effect on behavior.)

This is a final patch for this bug. I need someone to checkin.
Flags: needinfo?(nathmatics)
Attachment #8598842 - Flags: review?(jorendorff)
Keywords: coveritycheckin-needed

For future reference:
 - when adding keywords, please don't remove existing keywords. (Just add your new ones, after the existing ones).
 - You should include "r=[reviewer]"  (r=jorendorff in this case) at the end of your commit message. (though whoever lands the patch can add that in this case.)
 - No need to re-request review if you've already received r+ (and you're just reposting with a trivial tweak like a commit message addition).
Keywords: coverity
Attachment #8598842 - Attachment description: Final patch for removal of useless declaration in Parser.cpp → patch for checkin (now with commit message)
Attachment #8598842 - Flags: review?(jorendorff)
Daniel, thanks for those suggestions. I will keep them in mind for future patches. Many thanks!
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.