Skip to content

'n' support for float format #4865

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
youknowone merged 14 commits into from
Apr 15, 2023
Merged

'n' support for float format #4865

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
c1a390b
N and n support for float format.
JazzGlobal Apr 7, 2023
03a8990
Satisfy github actions
JazzGlobal Apr 8, 2023
a937e8e
cargo fmt
JazzGlobal Apr 8, 2023
9627142
Added tests
JazzGlobal Apr 8, 2023
d8c3f11
Fixed tests and N case functionality
JazzGlobal Apr 9, 2023
1fd13c8
Merge pull request #1 from JazzGlobal/okay
JazzGlobal Apr 9, 2023
f17f7a4
Fixed possibility of precision being larger than magnitude.
JazzGlobal Apr 9, 2023
e5fd98d
Lint error.
JazzGlobal Apr 9, 2023
d08a36e
Lint Error Part 2.
JazzGlobal Apr 10, 2023
f46e88f
Unmark as expected fail.
JazzGlobal Apr 10, 2023
ae47876
Merge branch 'format_float_code_n' of https://github.com/JazzGlobal/R…
JazzGlobal Apr 10, 2023
a6e7a55
Fixed tests
JazzGlobal Apr 10, 2023
8c76113
cargo fmt
JazzGlobal Apr 10, 2023
509e811
Reuse GeneralFormat. �
JazzGlobal Apr 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Lint error.
  • Loading branch information
@JazzGlobal
JazzGlobal committed Apr 9, 2023
commit e5fd98d2f9cae7537b1e762d3354c0792a4e5308
2 changes: 1 addition & 1 deletion common/src/format.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -438,7 +438,7 @@ impl FormatSpec {
Err(FormatSpecError::UnknownFormatCode(ch, "float"))
}
Some(FormatType::Number(Case::Lower)) => {
Comment thread
DimitrisJim marked this conversation as resolved.
Outdated
let size = magnitude.to_string().replace(".", "").len();
let size = magnitude.to_string().replace('.', '').len();
// If we get a precision larger than the magnitude (without the decimal), use the size.
let precision = if precision > size { size } else { precision };
Ok(float_ops::format_number(magnitude, precision))
Expand Down