-
Notifications
You must be signed in to change notification settings - Fork 174
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.
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
Datagen: move HashSet to the trait and other small cleanups #4473
Conversation
impl IterableDataProvider<$marker> for crate::DatagenProvider { | ||
fn supported_locales(&self) -> Result<Vec<DataLocale>, DataError> { | ||
Ok(vec![DataLocale::default()]) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: I left the trivial impls directly with IterableDataProvider in the previous PR but accidentally migrated this trivial impl.
fn supported_locales(&self) -> Result<Vec<DataLocale>, DataError> { | ||
fn supported_locales_for_numbers(&self) -> Result<HashSet<DataLocale>, DataError> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This poorly named function has been around for a long time and I hope it didn't cause confusion. It dates to the time when each transformer was implemented on its own struct.
while you're in here please put the performance improvements in the changelog |
Changelog in #4484 |
Just some more things I had on my branch. No appreciable impact on performance (54.17, 58.70, 52.65).