Skip to content

Commit

Permalink
fix styles
Browse files Browse the repository at this point in the history
  • Loading branch information
cwharris committed Sep 20, 2023
1 parent dfea856 commit 1702de9
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions python/mrc/_pymrc/src/subscriber.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -144,13 +144,14 @@ PyObjectObservable ObservableProxy::pipe(const PyObjectObservable* self, py::arg
operators.emplace_back(op.get_operate_fn());
}

if (operators.empty()) {
if (operators.empty())
{
throw std::runtime_error("pipe() must be given at least one argument");

Check warning on line 149 in python/mrc/_pymrc/src/subscriber.cpp

View check run for this annotation

Codecov / codecov/patch

python/mrc/_pymrc/src/subscriber.cpp#L149

Added line #L149 was not covered by tests
}

auto result = *self | operators[0];

for (auto i = 1 ; i < operators.size(); i++)
for (auto i = 1; i < operators.size(); i++)
{
result = result | operators[i];
}
Expand Down

0 comments on commit 1702de9

Please sign in to comment.