Skip to content

Commit 5ad5023

Browse files
committed
Use safe negation in a few places
1 parent 90da1b5 commit 5ad5023

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

include/flux/op/begin_end.hpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ struct sequence_iterator {
102102
constexpr auto operator-=(difference_type n) -> sequence_iterator&
103103
requires random_access_sequence<S>
104104
{
105-
flux::inc(*seq_, cur_, -n);
105+
flux::inc(*seq_, cur_, num::neg(n));
106106
return *this;
107107
}
108108

@@ -153,7 +153,7 @@ struct sequence_iterator {
153153
-> sequence_iterator
154154
requires random_access_sequence<S>
155155
{
156-
flux::inc(*self.seq_, self.cur_, -n);
156+
flux::inc(*self.seq_, self.cur_, num::neg(n));
157157
return self;
158158
}
159159

include/flux/op/reverse.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ struct reverse_adaptor : inline_sequence_base<reverse_adaptor<Base>>
103103

104104
static constexpr auto inc(auto& self, cursor_type& cur, distance_t dist) -> void
105105
{
106-
flux::inc(self.base_, cur.base_cur, num::sub(distance_t{}, dist));
106+
flux::inc(self.base_, cur.base_cur, num::neg(dist));
107107
}
108108

109109
static constexpr auto distance(auto& self, cursor_type const& from, cursor_type const& to)

include/flux/op/stride.hpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,8 @@ inline constexpr struct advance_fn {
5151
flux::inc(seq, cur, dist);
5252
return num::sub(offset, dist);
5353
} else if (offset < 0) {
54-
auto dist = num::sub(distance_t{0},
55-
(cmp::min)(flux::distance(seq, flux::first(seq), cur), -offset));
54+
auto dist = num::neg((cmp::min)(flux::distance(seq, flux::first(seq), cur),
55+
num::neg(offset)));
5656
flux::inc(seq, cur, dist);
5757
return num::sub(offset, dist);
5858
} else {

0 commit comments

Comments
 (0)