Minor refactoring based on static analysis on the code of *ROIAlign: (#2892)

- Remove unnecessary return.

Minor refactoring based on static analysis on the code of *ROIAlign: (#2892)
- Remove unnecessary return.
d8ccbe7e · Vasilis Vryniotis · GitHub · 2e23e477 · d8ccbe7e · d8ccbe7e
Unverified Commit d8ccbe7e authored Oct 26, 2020 by Vasilis Vryniotis Committed by GitHub Oct 26, 2020
4 changed files
--- a/torchvision/csrc/cpu/PSROIAlign_cpu.cpp
+++ b/torchvision/csrc/cpu/PSROIAlign_cpu.cpp
@@ -194,8 +194,6 @@ void bilinear_interpolate_gradient(
  // T val = (w1 * v1 + w2 * v2 + w3 * v3 + w4 * v4);

  w1 = hy * hx, w2 = hy * lx, w3 = ly * hx, w4 = ly * lx;
-
-  return;
 }

 template <class T>

--- a/torchvision/csrc/cpu/ROIAlign_cpu.cpp
+++ b/torchvision/csrc/cpu/ROIAlign_cpu.cpp
@@ -269,8 +269,6 @@ void bilinear_interpolate_gradient(
  // T val = (w1 * v1 + w2 * v2 + w3 * v3 + w4 * v4);

  w1 = hy * hx, w2 = hy * lx, w3 = ly * hx, w4 = ly * lx;
-
-  return;
 }

 template <class T>

--- a/torchvision/csrc/cuda/PSROIAlign_cuda.cu
+++ b/torchvision/csrc/cuda/PSROIAlign_cuda.cu
@@ -192,8 +192,6 @@ __device__ void bilinear_interpolate_gradient(
  // T val = (w1 * v1 + w2 * v2 + w3 * v3 + w4 * v4);

  w1 = hy * hx, w2 = hy * lx, w3 = ly * hx, w4 = ly * lx;
-
-  return;
 }

 template <typename T>

--- a/torchvision/csrc/cuda/ROIAlign_cuda.cu
+++ b/torchvision/csrc/cuda/ROIAlign_cuda.cu
@@ -194,8 +194,6 @@ __device__ void bilinear_interpolate_gradient(
  // T val = (w1 * v1 + w2 * v2 + w3 * v3 + w4 * v4);

  w1 = hy * hx, w2 = hy * lx, w3 = ly * hx, w4 = ly * lx;
-
-  return;
 }

 template <typename T>