1597237 - Remove useless return; declaration

Status: RESOLVED FIXED

(Core :: General, task, P3)

Description

[Sylvestre Ledru [:Sylvestre]]

+++ This bug was initially created as a clone of Bug #1597214 +++

+++ This bug was initially created as a clone of Bug #1592270 +++

Filling as a good first bug to learn workflows.

As return; is at the end of a void function, the declaration is useless
https://searchfox.org/mozilla-central/source/dom/events/DataTransfer.cpp#738

As the change is trivial, it is just to learn how to contribute to Firefox.

Comment 1

[Elliot Matthew]

Hi, getting familiar with the contribution process looking for a first bug fix. I'd like to take on the bug.

Comment 2

[Sylvestre Ledru [:Sylvestre]]

walshd9 is working on it (but I will unassigned it if no progress in a few days)
Send me an email if you want me to open a new bug for you

Comment 3

[Dan Walsh]

Hi, sorry about the delay, I've had a lot of emails coming in from mozilla and have had to slowly filter through them. I'll get on this over the weekend. From reading above, all that is needed is the remove of the one return statement. Is this correct? Cheers

Comment 4

[Dan Walsh]

Attachment: Bug 1597237 - Remove useless return; declaration, r=Sylvestre

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cf4e71881270
Remove useless return; declaration, r=sylvestre

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/cf4e71881270