Bug 1597237 - Remove useless return; declaration, r=sylvestre
authordanwalsh99 <walshd9@uni.coventry.ac.uk>
Sun, 24 Nov 2019 08:28:39 +0000
changeset 503594 cf4e718812707016ddc4536adc287e44d30916dc
parent 503593 46acccbd937dacc4251e11915ede38b9bbb5e202
child 503598 21be90ebc129905536e6134b9d1849032df72ed0
push id101407
push usersledru@mozilla.com
push dateSun, 24 Nov 2019 08:29:13 +0000
treeherderautoland@cf4e71881270 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssylvestre
bugs1597237
milestone72.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1597237 - Remove useless return; declaration, r=sylvestre Differential Revision: https://phabricator.services.mozilla.com/D54414
dom/events/DataTransfer.cpp
--- a/dom/events/DataTransfer.cpp
+++ b/dom/events/DataTransfer.cpp
@@ -730,17 +730,16 @@ void DataTransfer::GetExternalTransferab
                                   kUnicodeMime,     kPNGImageMime};
 
   for (const char* format : formats) {
     auto index = flavors.IndexOf(nsCString(format));
     if (index != flavors.NoIndex) {
       aResult->AppendElement(nsCString(format));
     }
   }
-  return;
 }
 
 nsresult DataTransfer::SetDataAtInternal(const nsAString& aFormat,
                                          nsIVariant* aData, uint32_t aIndex,
                                          nsIPrincipal* aSubjectPrincipal) {
   if (aFormat.IsEmpty()) {
     return NS_OK;
   }