mpl: refactor to avoid duplicated code

Signed-off-by: Arthur Koucher <[email protected]>

Author: AcKoucher

src/mpl/src/SACoreSoftMacro.cpp

@@ -109,8 +109,7 @@ bool SACoreSoftMacro::isValid() const
     return false;
   }
 
-  return (width_ <= std::ceil(outline_.getWidth()))
-         && (height_ <= std::ceil(outline_.getHeight()));
+  return resultFitsInOutline();
 }
 
 // acessors functions

src/mpl/src/SimulatedAnnealingCore.cpp

@@ -153,8 +153,7 @@ void SimulatedAnnealingCore<T>::setInitialSequencePair(
 template <class T>
 bool SimulatedAnnealingCore<T>::isValid() const
 {
-  return (width_ <= std::ceil(outline_.getWidth()))
-         && (height_ <= std::ceil(outline_.getHeight()));
+  return resultFitsInOutline();
 }
 
 template <class T>
@@ -769,6 +768,13 @@ void SimulatedAnnealingCore<T>::fastSA()
   }
 }
 
+template <class T>
+bool SimulatedAnnealingCore<T>::resultFitsInOutline() const
+{
+  return (width_ <= std::ceil(outline_.getWidth()))
+          && (height_ <= std::ceil(outline_.getHeight()));
+}
+
 template <class T>
 void SimulatedAnnealingCore<T>::updateBestResult(const float cost)
 {

src/mpl/src/SimulatedAnnealingCore.h

@@ -102,6 +102,7 @@ class SimulatedAnnealingCore
   };
 
   void fastSA();
+  bool resultFitsInOutline() const;
 
   void setAvailableRegionsForUnconstrainedPins(
       const BoundaryRegionList& regions);