Check return value form CreateFeedsFetchesManager. (#1653)

Also cleanup a couple of unused variables.
This commit is contained in:
Scott McKay 2019-08-20 12:20:21 -07:00 committed by GitHub
parent 7be5695fad
commit 5311c1b2b5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 2 additions and 3 deletions

View file

@ -257,7 +257,6 @@ Status LoopImpl::CreateFeedsFetchesManager(std::unique_ptr<FeedsFetchesManager>&
feed_names.push_back(entry.first);
}
FeedsFetchesInfo ffi(feed_names, subgraph_output_names_);
auto status =
FeedsFetchesManager::Create(feed_names, subgraph_output_names_, session_state_.GetOrtValueNameIdxMap(), ffm);

View file

@ -125,7 +125,6 @@ Status CreateFeedsFetchesManager(const GraphViewer& subgraph, int num_variadic_i
feed_names.push_back(entry.first);
}
FeedsFetchesInfo ffi(feed_names, subgraph_output_names);
auto status = FeedsFetchesManager::Create(feed_names, subgraph_output_names, ort_value_name_idx_map, ffm);
return status;

View file

@ -26,7 +26,8 @@ common::Status SubgraphExecuteHelper(std::unique_ptr<FeedsFetchesManager>& cache
} else {
// use a local instance until we know we're successful, and cache if it is
std::unique_ptr<FeedsFetchesManager> new_ffm;
impl.CreateFeedsFetchesManager(new_ffm);
ORT_RETURN_IF_ERROR(impl.CreateFeedsFetchesManager(new_ffm));
status = impl.Execute(&*new_ffm, nullptr);
if (status.IsOK()) {
cached_feeds_fetches_manager = std::move(new_ffm);