build: unreachable code after CV_Error()

Author: alalek

modules/datasets/src/or_pascal.cpp

@@ -143,7 +143,6 @@ Ptr<Object> OR_pascalImp::parseAnnotation(const string &path, const string &id)
         case XML_ERROR_FILE_NOT_FOUND:
             error_message = "XML file not found! " + error_message;
             CV_Error(Error::StsParseError, error_message);
-            return annotation;
         default:
             CV_Error(Error::StsParseError, error_message);
             break;

modules/face/src/lbph_faces.cpp

@@ -307,10 +307,8 @@ static Mat histc(InputArray _src, int minVal, int maxVal, bool normed)
         case CV_32FC1:
             return histc_(src, minVal, maxVal, normed);
             break;
-        default:
-            CV_Error(Error::StsUnmatchedFormats, "This type is not implemented yet."); break;
     }
-    return Mat();
+    CV_Error(Error::StsUnmatchedFormats, "This type is not implemented yet.");
 }
 
 

modules/line_descriptor/src/binary_descriptor.cpp

@@ -337,13 +337,8 @@ int BinaryDescriptor::descriptorSize() const
 /* power function with error management */
 static inline int get2Pow( int i )
 {
-  if( i >= 0 && i <= 7 )
-    return 1 << i;
-  else
-  {
-    CV_Error( Error::StsBadArg, "Invalid power argument" );
-    return -1;
-  }
+  CV_DbgAssert(i >= 0 && i <= 7);
+  return 1 << i;
 }
 
 /* compute Gaussian pyramids */

modules/tracking/src/tracker.cpp

@@ -73,7 +73,6 @@ bool Tracker::init( InputArray image, const Rect2d& boundingBox )
   if( model == 0 )
   {
     CV_Error( -1, "The model is not initialized" );
-    return false;
   }
 
   if( initTracker )

modules/tracking/src/trackerBoostingModel.cpp

@@ -89,7 +89,6 @@ void TrackerBoostingModel::responseToConfidenceMap( const std::vector<Mat>& resp
   if( currentSample.empty() )
   {
     CV_Error( -1, "The samples in Model estimation are empty" );
-    return;
   }
 
   for ( size_t i = 0; i < currentSample.size(); i++ )

modules/tracking/src/trackerFeature.cpp

@@ -90,7 +90,6 @@ Ptr<TrackerFeature> TrackerFeature::create( const String& trackerFeatureType )
   }
 
   CV_Error( -1, "Tracker feature type not supported" );
-  return Ptr<TrackerFeature>();
 }
 
 String TrackerFeature::getClassName() const

modules/tracking/src/trackerMILModel.cpp

@@ -67,7 +67,6 @@ void TrackerMILModel::responseToConfidenceMap( const std::vector<Mat>& responses
   if( currentSample.empty() )
   {
     CV_Error( -1, "The samples in Model estimation are empty" );
-    return;
   }
 
   for ( size_t i = 0; i < responses.size(); i++ )

modules/tracking/src/trackerModel.cpp

@@ -112,7 +112,6 @@ bool TrackerModel::runStateEstimator()
   if( stateEstimator == 0 )
   {
     CV_Error( -1, "Tracker state estimator is not setted" );
-    return false;
   }
   Ptr<TrackerTargetState> targetState = stateEstimator->estimate( confidenceMaps );
   if( targetState == 0 )

modules/tracking/src/trackerSamplerAlgorithm.cpp

@@ -83,7 +83,6 @@ Ptr<TrackerSamplerAlgorithm> TrackerSamplerAlgorithm::create( const String& trac
   }
 
   CV_Error( -1, "Tracker sampler algorithm type not supported" );
-  return Ptr<TrackerSamplerAlgorithm>();
 }
 
 String TrackerSamplerAlgorithm::getClassName() const

modules/tracking/src/trackerStateEstimator.cpp

@@ -85,7 +85,6 @@ Ptr<TrackerStateEstimator> TrackerStateEstimator::create( const String& trackeSt
   }
 
   CV_Error( -1, "Tracker state estimator type not supported" );
-  return Ptr<TrackerStateEstimator>();
 }
 
 String TrackerStateEstimator::getClassName() const