Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Replace dictionary proxies with nested dictionaries 06/N #678

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

ienkovich
Copy link
Contributor

Add StringDictionaryProxy::copyStrings. This is to avoid special handling of transient strings (special ID handling will be removed later).


// Iterate over transient strings, then non-transients.
void eachStringSerially(StringDictionary::StringCallback&) const;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the iteration order significant?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we provide any particular order. And this API is not actually used anyway. Method is moved to match the Dictionary API.

@ienkovich ienkovich merged commit c9676c0 into main Sep 28, 2023
@ienkovich ienkovich deleted the ienkovich/nested-dict-06 branch September 28, 2023 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants