Skip to content

Conversation

taiki-e
Copy link
Member

@taiki-e taiki-e commented Oct 20, 2022

Xaeroxe and others added 8 commits October 20, 2022 11:50
```
warning: unreachable `pub` item
  --> futures-util/src/stream/futures_unordered/mod.rs:25:32
   |
25 | pub use self::iter::{IntoIter, Iter, IterMut, IterPinMut, IterPinRef};
   | ---                            ^^^^
   | |
   | help: consider restricting its visibility: `pub(crate)`
   |
   = help: or consider exporting it for use by other crates
note: the lint level is defined here
  --> futures-util/src/lib.rs:11:5
   |
11 |     unreachable_pub
   |     ^^^^^^^^^^^^^^^

warning: unreachable `pub` item
  --> futures-util/src/stream/futures_unordered/mod.rs:25:38
   |
25 | pub use self::iter::{IntoIter, Iter, IterMut, IterPinMut, IterPinRef};
   | ---                                  ^^^^^^^
   | |
   | help: consider restricting its visibility: `pub(crate)`
   |
   = help: or consider exporting it for use by other crates
```
```
warning: casting to the same type is unnecessary (`usize` -> `usize`)
  --> futures-util/src/stream/stream/take.rs:57:37
   |
57 |         let lower = cmp::min(lower, self.remaining as usize);
   |                                     ^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.remaining`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
   = note: `#[warn(clippy::unnecessary_cast)]` on by default

warning: casting to the same type is unnecessary (`usize` -> `usize`)
  --> futures-util/src/stream/stream/take.rs:60:28
   |
60 |             Some(x) if x < self.remaining as usize => Some(x),
   |                            ^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.remaining`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast

warning: casting to the same type is unnecessary (`usize` -> `usize`)
  --> futures-util/src/stream/stream/take.rs:61:23
   |
61 |             _ => Some(self.remaining as usize),
   |                       ^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.remaining`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
```
* fix soundness hole in join macros
add a miri regression test
update failing tests (join sizes increased due to fix)

* fix `CI / cross test` by ignoring `join_size` and `try_join_size` tests on "non-64-bit pointer" targets (e.g. `i686-unknown-linux-gnu`)
(this is the same fix that was also applied in PR #2447)
@taiki-e taiki-e added the futures-0.3 Issue related to the 0.3 versions of futures label Oct 20, 2022
@taiki-e taiki-e merged commit 77d8219 into 0.3 Oct 20, 2022
@taiki-e taiki-e deleted the 0.3-next branch October 20, 2022 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
futures-0.3 Issue related to the 0.3 versions of futures
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants