Bug 1597237 - Remove useless return; declaration, r=sylvestre
authordanwalsh99 <walshd9@uni.coventry.ac.uk>
Sun, 24 Nov 2019 08:28:39 +0000
changeset 503580 cf4e718812707016ddc4536adc287e44d30916dc
parent 503579 46acccbd937dacc4251e11915ede38b9bbb5e202
child 503581 21be90ebc129905536e6134b9d1849032df72ed0
push id36838
push useraciure@mozilla.com
push dateSun, 24 Nov 2019 21:27:24 +0000
treeherdermozilla-central@e19228b7fd46 [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;
   }