Skip to content

Expose FloatErrorKind - #164794

Merged
rust-bors[bot] merged 4 commits into
rust-lang:mainfrom
malezjaa:extend-float-error-kind
Oct 26, 2026
Merged

rust-bors[bot] merged 4 commits into
rust-lang:mainfrom
malezjaa:extend-float-error-kind

Conversation

@malezjaa

@malezjaa malezjaa commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Tracking issue: #164793
Closes rust-lang/libs-team#683 (ACP)

Right now this only makes the enum public and adds the method, but I'd like more input from libs team on what the FloatErrorKind should actually be extended with.

r? libs

@rustbot

rustbot commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

Some changes occurred in float parsing

cc @tgross35

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 23, 2026
@malezjaa
malezjaa force-pushed the extend-float-error-kind branch from 7c593ee to 4a0f132 Compare October 23, 2026 13:03
@rust-log-analyzer

This comment has been minimized.

@clarfonthey

Copy link
Copy Markdown
Contributor

I think that the Invalid variant is less than ideal, but I left a comment on the tracking issue since that can be resolved later. For now, I think this is fine. Thank you!

@bors r+ rollup

@rust-bors

rust-bors Bot commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

📌 Commit acb24f2 has been approved by clarfonthey

It is now in the queue for this repository.

🌲 The tree is currently closed for pull requests below priority 5. This pull request will be tested once the tree is reopened.

Reason for tree closure: test-aarch64-apple-macos-26-2 out of disk space

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 25, 2026
@clarfonthey

Copy link
Copy Markdown
Contributor

Also, small procedural thing: accepted ACPs can be closed as soon as the tracking issue is opened; the implementation is nice but not necessary. That's why I closed the ACP before this merged.

@clarfonthey clarfonthey changed the title extend FloatErrorKind Expose FloatErrorKind Oct 25, 2026
@malezjaa

Copy link
Copy Markdown
Contributor Author

It wasn't my ACP, so I couldn't close it myself. I didn't want to bother anyone, so I just linked it using 'Closes #...' ^^ (I hope GitHub can actually close issues between org repos)

@clarfonthey

Copy link
Copy Markdown
Contributor

Ah, I didn't notice that detail, so, makes sense. Either way I resolved that issue.

JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Oct 25, 2026
…r=clarfonthey

Expose FloatErrorKind

Tracking issue: rust-lang#164793
Closes rust-lang/libs-team#683 (ACP)

Right now this only makes the enum public and adds the method, but I'd like more input from libs team on what the `FloatErrorKind` should actually be extended with.

r? libs
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Oct 25, 2026
…r=clarfonthey

Expose FloatErrorKind

Tracking issue: rust-lang#164793
Closes rust-lang/libs-team#683 (ACP)

Right now this only makes the enum public and adds the method, but I'd like more input from libs team on what the `FloatErrorKind` should actually be extended with.

r? libs
rust-bors Bot pushed a commit that referenced this pull request Oct 25, 2026
…uwer

Rollup of 20 pull requests

Successful merges:

 - #163730 (add tests over re-entrancy guarantees for 160776)
 - #164834 (miri subtree update)
 - #164849 (Use the span of the doctest code block fence for figuring out the line number)
 - #164883 (rustdoc: don't panic on a nested use group with no path prefix)
 - #164890 (Store cold blocks in a bitset instead of a Vec of bools)
 - #163458 (stabilize {Rc, Arc}::is_unique)
 - #163499 (vec: stabilise allocators for into_iter)
 - #163805 (std: check `SO_ERROR` after poll in `TcpStream::connect_timeout`)
 - #163816 (stabilize String::into_chars)
 - #163819 (Correct documentation of what `core::ops::Code` is.)
 - #163847 (add regression test for reborrow-invariant-type)
 - #164006 (Arduously fix typo)
 - #164009 (Update `Ready` future expect messages)
 - #164522 (Add IPv6 hop limit stubs to the motor OS net backend)
 - #164602 (Improve seek_read_buf_exact() test coverage to match seek_read_exact())
 - #164772 (docs: Two nitpick changes to `HashMap`, `BTreeMap` Entry API docs)
 - #164794 (Expose FloatErrorKind)
 - #164802 (Migrate stable-specific run-make tests to UI tests)
 - #164899 (Only recover if we may in `parse_expr_tuple_parens`)
 - #164903 (rustc-dev-guide subtree update)

Failed merges:

 - #164856 (Switch from `insta` to `expect-test` in bootstrap)
@rust-bors
rust-bors Bot merged commit 0ac4e1e into rust-lang:main Oct 26, 2026
14 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Oct 26, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 26, 2026
Rollup merge of #164794 - malezjaa:extend-float-error-kind, r=clarfonthey

Expose FloatErrorKind

Tracking issue: #164793
Closes rust-lang/libs-team#683 (ACP)

Right now this only makes the enum public and adds the method, but I'd like more input from libs team on what the `FloatErrorKind` should actually be extended with.

r? libs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expose FloatErrorKind from private core::num::dec2flt to public core::num::FloatErrorKind

4 participants