Skip to content
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

Remove DynamicDataProvider and expand methods into ExportableProvider #4881

Open
sffc opened this issue May 8, 2024 · 1 comment
Open

Remove DynamicDataProvider and expand methods into ExportableProvider #4881

sffc opened this issue May 8, 2024 · 1 comment
Labels
2.0-breaking Changes that are breaking API changes C-data-infra Component: provider, datagen, fallback, adapters discuss Discuss at a future ICU4X-SC meeting S-small Size: One afternoon (small bug fix or enhancement)
Milestone

Comments

@sffc
Copy link
Member

sffc commented May 8, 2024

I'm pretty sure that we're not using DynamicDataProvider anywhere except in the bounds of ExportableProvider.

We should merge DynamicDataProvider and IterableDynamicDataProvider into ExportableProvider in 2.0.

@sffc sffc added C-data-infra Component: provider, datagen, fallback, adapters S-small Size: One afternoon (small bug fix or enhancement) 2.0-breaking Changes that are breaking API changes labels May 8, 2024
@sffc sffc added this to the ICU4X 2.0 milestone May 8, 2024
@robertbastian
Copy link
Member

I see DynamicDataProvider as a core provider type, it's the equivalent to DataProvider where the key is only known at runtime. In fact, AnyProvider and BufferProvider should be instantiations of DynamicDataProvider for AnyMarker and BufferMarker.

@robertbastian robertbastian added the discuss Discuss at a future ICU4X-SC meeting label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0-breaking Changes that are breaking API changes C-data-infra Component: provider, datagen, fallback, adapters discuss Discuss at a future ICU4X-SC meeting S-small Size: One afternoon (small bug fix or enhancement)
Projects
Status: Small breakage (defer to end)
Development

No branches or pull requests

2 participants