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

Add ListBuilder #1

Merged
merged 1 commit into from
Mar 5, 2019
Merged

Add ListBuilder #1

merged 1 commit into from
Mar 5, 2019

Conversation

benbjohnson
Copy link
Owner

@benbjohnson benbjohnson commented Mar 5, 2019

This commit adds a builder for more efficiently creating List objects. It works by mutating the list in-place until the current list is requested. After that request, the next mutation will occur on a copy but then continue to mutate in place. This allows lists to be returned and used while continuing to use the builder.

Closes #2

@benbjohnson benbjohnson self-assigned this Mar 5, 2019
@codecov
Copy link

codecov bot commented Mar 5, 2019

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines         814    887   +73     
=====================================
+ Hits          814    887   +73
Impacted Files Coverage Δ
immutable.go 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc1145d...08218d3. Read the comment docs.

This commit adds a builder for more efficiently creating `List`
objects. It works by mutating the list in-place until the current
list is requested. After that request, the next mutation will occur
on a copy but then continue to mutate in place. This allows lists
to be returned and used while continuing to use the builder.
@benbjohnson benbjohnson merged commit dd9ebb8 into master Mar 5, 2019
@benbjohnson benbjohnson deleted the list-builder branch March 5, 2019 22:12
benbjohnson pushed a commit that referenced this pull request Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant