# Libs-API Meeting 2022-02-09

###### tags: `Libs Meetings` `Minutes`

**Meeting Link**: https://meet.jit.si/rust-libs-meeting-crxoz2at8hiccp7b3ixf89qgxfymlbwr
**Attendees**: Amanieu, David, Jane, Josh Triplett, Mara, bstrie, Mark, The 8472, scottmcm

## Agenda

- Triage
- Anything else?

## Triage

### FCPs

10 open T-libs-api FCPs:
<details><summary><a href="https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs-api+label%3Aproposed-final-comment-period">1 <code>rust-lang/rfcs</code> FCPs</a></summary>

- [[merge 3192](https://github.com/rust-lang/rfcs/issues/3192#issuecomment-1015775980)] *Add provide\_any module to core* - (4 checkboxes left)
</details>
<details><summary><a href="https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs-api+label%3Aproposed-final-comment-period">9 <code>rust-lang/rust</code> FCPs</a></summary>

- [[merge 89238](https://github.com/rust-lang/rust/issues/89238#issuecomment-927177703)] *deprecate f{32,64}::DIGITS* - (1 checkboxes left)
- [[merge 86845](https://github.com/rust-lang/rust/issues/86845#issuecomment-950843091)] *Tracking Issue for owned locked stdio handles* - (4 checkboxes left)
- [[merge 90291](https://github.com/rust-lang/rust/issues/90291#issuecomment-955599591)] *Loosen the bound on the Debug implementation of Weak.* - (2 checkboxes left)
- [[merge 80697](https://github.com/rust-lang/rust/issues/80697#issuecomment-1022585839)] *Tracking Issue for const\_intrinsic\_copy* - (1 checkboxes left)
- [[merge 92663](https://github.com/rust-lang/rust/issues/92663#issuecomment-1023878238)] *Implement \`Write for Cursor\<\[u8; N\]\>\`, plus \`A: Allocator\` cursor support* - (3 checkboxes left)
- [[merge 93263](https://github.com/rust-lang/rust/issues/93263#issuecomment-1023928386)] *Consistently present absent stdio handles on Windows as NULL handles.* - (4 checkboxes left)
- [[merge 91789](https://github.com/rust-lang/rust/issues/91789#issuecomment-1026130086)] *Tracking Issue for try\_reserve method on more containers* - (5 checkboxes left)
- [[merge 80967](https://github.com/rust-lang/rust/issues/80967#issuecomment-1033026809)] *Tracking Issue for bool\_to\_option* - (3 checkboxes left)
- [[merge 92472](https://github.com/rust-lang/rust/issues/92472#issuecomment-1033086212)] *proc\-macro: Stop wrapping \`ident\` matchers into groups* - (7 checkboxes left)
</details>
<p></p>

[nikomatsakis (1)](https://rfcbot.rs/fcp/nikomatsakis), [Amanieu (4)](https://rfcbot.rs/fcp/Amanieu), [joshtriplett (4)](https://rfcbot.rs/fcp/joshtriplett), [BurntSushi (10)](https://rfcbot.rs/fcp/BurntSushi), [yaahc (5)](https://rfcbot.rs/fcp/yaahc), [m-ou-se (4)](https://rfcbot.rs/fcp/m-ou-se), [pnkfelix (1)](https://rfcbot.rs/fcp/pnkfelix), [cramertj (1)](https://rfcbot.rs/fcp/cramertj), [dtolnay (3)](https://rfcbot.rs/fcp/dtolnay), [scottmcm (1)](https://rfcbot.rs/fcp/scottmcm)

### Nominated

- [8 `rust-lang/rust` `T-libs-api` `I-libs-api-nominated` items](https://github.com/rust-lang/rust/issues?q=label:T-libs-api+label:I-libs-api-nominated+is:open)
- [[43301](https://github.com/rust-lang/rust/issues/43301)] *Tracking issue for RFC 1937: \`?\` in \`main\`*
- No updates since last week.
- Jane working on stabilization.
- [[77174](https://github.com/rust-lang/rust/issues/77174)] *Tracking Issue for inherent\_ascii\_escape*
FCP finished. Should be merged?
- Just needs stabilization PR.
- Deprecating std::ascii can be proposed separately.
- Josh posted summary comment
- [[87067](https://github.com/rust-lang/rust/issues/87067)] *Std implementations of PartialOrd are violating the conditions regarding transitivity and symmetry. And the transitivity requirements are i̶m̶p̶o̶s̶s̶i̶b̶l̶e̶ hard to ensure in a multi\-crate ecosystem anyways.*
- Already changed the docs for PartialEq in #81198. We should accept the same change for PartialOrd.
- [[89926](https://github.com/rust-lang/rust/pull/89926)] *make \`Instant::{duration\_since, elapsed, sub}\` saturating and remove workarounds*
FCP finished. Should be merged?
https://github.com/rust-lang/rust/pull/89926#discussion_r781455890
- [[92472](https://github.com/rust-lang/rust/pull/92472)] *proc\-macro: Stop wrapping \`ident\` matchers into groups*
- Discussed in last meeting. FCP was already started.
- [[93176](https://github.com/rust-lang/rust/pull/93176)] *Add a stack\-\`pin!\`\-ning macro to \`core::pin\`.*
- We might have `unsafe` fields in the future.
- For now, this hack seems fine, as it can be replaced by something better/stable in the future.
- [[93563](https://github.com/rust-lang/rust/pull/93563)] *Merge crossbeam\-channel into \`std::sync::mpsc\`*
- Deprecate mpsc in the future, maintain it, or even add e.g. mpmc?
- No reason to block this PR itself, as it is a bug fix.
- No need for FCP. Crater run might be useful. More than one reviewer might be good.
- [[93653](https://github.com/rust-lang/rust/pull/93653)] *Added \`Box::take()\` method*
- Lots of APIs around uninit/allocators/etc., starting to become a bit of a mess.
- This particular Box::take() API seems a bit too specific. A small amount of unsafe code covers this use case. Might be fine for uncommon situations.
- Mark to close this PR.
- Mara to write something in the tracking issue/zulip stream about all the uninit functions.

### Waiting on team

- [1 `rust-lang/rust` `T-libs-api` `S-waiting-on-team` items](https://github.com/rust-lang/rust/issues?q=label:T-libs-api+label:S-waiting-on-team+is:open)
- [[81642](https://github.com/rust-lang/rust/pull/81642)] *Automatically implement AsRepr and allow deriving FromRepr for fieldless enums*
- Not waiting for us.

### Needs decision

- [1 `rust-lang/rust` `T-libs-api` `I-needs-decision` items](https://github.com/rust-lang/rust/issues?q=label:T-libs-api+label:I-needs-decision+is:open)
- [[91226](https://github.com/rust-lang/rust/issues/91226)] *\`std::env::vars\` and \`std::env::args\` should silently ignore non\-UTF8 values*
- Lossy conversion instead of panicking?
- Might be a security issue. A path might be implicitly converted to another valid path.
- Omitting an env var might also be a security issue.
- Strongly recommend vars_os over vars? Deprecate even maybe?
- Close this issue, because we don't want silently ignoring/transforming environment variables. Instead, we should strongly recommend `vars_os` over `vars`.
- Josh to reply.

### Stalled Tracking Issues

- [30 `rust-lang/rust` `T-libs-api` `C-tracking-issue` items](https://github.com/rust-lang/rust/issues?q=label:T-libs-api+label:C-tracking-issue+is:open+sort:updated-asc)
- [[72505](https://github.com/rust-lang/rust/issues/72505)] *Tracking issue for \`#!\[feature(const\_float\_classify)\]\`*
- [[26350](https://github.com/rust-lang/rust/issues/26350)] *Clarify story on libm bindings*
- [[29553](https://github.com/rust-lang/rust/issues/29553)] *Tracking issue for #\[bench\] and benchmarking support*
- [[41263](https://github.com/rust-lang/rust/issues/41263)] *Tracking issue for \`ToOwned::clone\_into\` (\`toowned\_clone\_into\`)*
- [[71213](https://github.com/rust-lang/rust/issues/71213)] *Tracking Issue for \`wasi\_ext\`*

## Actions

- [ ] Reply to all issues/PRs discussed in this meeting, or add them to the [open action items](https://hackmd.io/ovrbJj6CRduRgSA0Wzg2zg).

_Generated by [fully-automatic-rust-libs-team-triage-meeting-agenda-generator](https://github.com/rust-lang/libs-team/tree/main/tools/agenda-generator)_