8000 'n' support for float format by JazzGlobal · Pull Request #4865 · RustPython/RustPython · GitHub
[go: up one dir, main page]
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
merged 14 commits into from
Apr 15, 2023
Merged

'n' support for float format #4865

Show file tree
Hide file tree
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 8000
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
cargo fmt
  • Loading branch information
@JazzGlobal
JazzGlobal committed Apr 8, 2023
commit a937e8eefe42f0fdd32776a18d550fa5adf7c03f
21 changes: 14 additions & 7 deletions common/src/float_ops.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -238,22 +238,29 @@ pub fn format_general(
pub fn format_number(magnitude: f64, precision: usize) -> String {
let magnitude_as_string = magnitude.to_string();
let separator_index = magnitude_as_string.chars().position(|c| c == '.').unwrap();
if separator_index == precision { return String::from(&magnitude_as_string[..separator_index]) }
if separator_index < precision
{
if separator_index == precision {
return String::from(&magnitude_as_string[..separator_index]);
}
if separator_index < precision {
let prefix = &magnitude_as_string[0..separator_index];
let suffix_distance = (precision - prefix.len() + 1) + separator_index;
let suffix = &magnitude_as_string[separator_index..suffix_distance];
let suffix = &magnitude_as_string[separator_index..suffix_distance];

return format!("{}{}", prefix, suffix)
return format!("{}{}", prefix, suffix);
}
format!("{}e+0{}", &magnitude_as_string[..1], get_num_of_char(&magnitude_as_string, '0'))
format!(
"{}e+0{}",
&magnitude_as_string[..1],
get_num_of_char(&magnitude_as_string, '0')
)
}

fn get_num_of_char(text: &str, value: char) -> i32 {
let mut total = 0;
for char in text.chars() {
if char == value { total += 1 }
if char == value {
total += 1
}
}
total
}
Expand Down
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 @@ -439,7 +439,7 @@ impl FormatSpec {
Some(FormatType::Number(_)) => {
let precision = if precision == 0 { 6 } else { precision };
Ok(float_ops::format_number(magnitude, precision))
},
}
Some(FormatType::GeneralFormat(case)) => {
let precision = if precision == 0 { 1 } else { precision };
Ok(float_ops::format_general(
Expand Down
0