Skip to content

Conversation

martin-g
Copy link
Contributor

Motivation

#7426 made tokio::net::unix::pipe::OpenOptions::read_write available on Android too

Solution

Update the docs to mention Android next to Linux

@Darksonn Darksonn added T-docs Topic: documentation A-tokio Area: The main tokio crate M-net Module: tokio/net labels Sep 11, 2025
/// ```
///
/// Opening a [`Sender`] on Linux when you are sure the file is a FIFO:
/// Opening a [`Sender`] on Linux/Android when you are sure the file is a FIFO:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The doc cfg stuff already makes it clear that it works on Android:

image

IMO Android is Linux and we don't need to also mention this in the text.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough!

@martin-g martin-g closed this Sep 11, 2025
@martin-g martin-g deleted the unix-pipe-read_write-on-android branch September 11, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate M-net Module: tokio/net T-docs Topic: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants