Merge pull request #23904 from kai-waang:removing-unreachable

asmorkalov · web-flow · commit 748279d5f102 · 2023-09-04T17:06:45.000+03:00
removing unreachable codes in `gbackend`
diff --git a/modules/gapi/src/api/gbackend.cpp b/modules/gapi/src/api/gbackend.cpp
@@ -36,7 +36,6 @@ cv::gapi::GBackend::Priv::compile(const ade::Graph&,
 {
     // ...and this method is here for the same reason!
     GAPI_Error("InternalError");
-    return {};
 }
 
 std::unique_ptr<cv::gimpl::GIslandExecutable>
@@ -224,7 +223,6 @@ void bindOutArg(Mag& mag, const RcDesc &rc, const GRunArgP &arg, HandleRMat hand
 
     default:
         util::throw_error(std::logic_error("Unsupported GShape type"));
-        break;
     }
 }
 
@@ -256,7 +254,6 @@ void resetInternalData(Mag& mag, const Data &d)
 
     default:
         util::throw_error(std::logic_error("Unsupported GShape type"));
-        break;
     }
 }
 
@@ -284,7 +281,6 @@ cv::GRunArg getArg(const Mag& mag, const RcDesc &ref)
                        mag.meta<cv::MediaFrame>().at(ref.id));
     default:
         util::throw_error(std::logic_error("Unsupported GShape type"));
-        break;
     }
 }
 
@@ -327,7 +323,6 @@ cv::GRunArgP getObjPtr(Mag& mag, const RcDesc &rc, bool is_umat)
 
     default:
         util::throw_error(std::logic_error("Unsupported GShape type"));
-        break;
     }
 }
 
@@ -359,7 +354,6 @@ void writeBack(const Mag& mag, const RcDesc &rc, GRunArgP &g_arg)
 
     default:
         util::throw_error(std::logic_error("Unsupported GShape type"));
-        break;
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,6 @@ cv::gapi::GBackend::Priv::compile(const ade::Graph&,`
`36`	`36`	`{`
`37`	`37`	`// ...and this method is here for the same reason!`
`38`	`38`	`GAPI_Error("InternalError");`
`39`		`- return {};`
`40`	`39`	`}`
`41`	`40`
`42`	`41`	`std::unique_ptr<cv::gimpl::GIslandExecutable>`
`@@ -224,7 +223,6 @@ void bindOutArg(Mag& mag, const RcDesc &rc, const GRunArgP &arg, HandleRMat hand`
`224`	`223`
`225`	`224`	`default:`
`226`	`225`	`util::throw_error(std::logic_error("Unsupported GShape type"));`
`227`		`- break;`
`228`	`226`	`}`
`229`	`227`	`}`
`230`	`228`
`@@ -256,7 +254,6 @@ void resetInternalData(Mag& mag, const Data &d)`
`256`	`254`
`257`	`255`	`default:`
`258`	`256`	`util::throw_error(std::logic_error("Unsupported GShape type"));`
`259`		`- break;`
`260`	`257`	`}`
`261`	`258`	`}`
`262`	`259`
`@@ -284,7 +281,6 @@ cv::GRunArg getArg(const Mag& mag, const RcDesc &ref)`
`284`	`281`	`mag.meta<cv::MediaFrame>().at(ref.id));`
`285`	`282`	`default:`
`286`	`283`	`util::throw_error(std::logic_error("Unsupported GShape type"));`
`287`		`- break;`
`288`	`284`	`}`
`289`	`285`	`}`
`290`	`286`
`@@ -327,7 +323,6 @@ cv::GRunArgP getObjPtr(Mag& mag, const RcDesc &rc, bool is_umat)`
`327`	`323`
`328`	`324`	`default:`
`329`	`325`	`util::throw_error(std::logic_error("Unsupported GShape type"));`
`330`		`- break;`
`331`	`326`	`}`
`332`	`327`	`}`
`333`	`328`
`@@ -359,7 +354,6 @@ void writeBack(const Mag& mag, const RcDesc &rc, GRunArgP &g_arg)`
`359`	`354`
`360`	`355`	`default:`
`361`	`356`	`util::throw_error(std::logic_error("Unsupported GShape type"));`
`362`		`- break;`
`363`	`357`	`}`
`364`	`358`	`}`
`365`	`359`