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 section about Enumerable#lazy #938

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ydakuka
Copy link
Contributor

@ydakuka ydakuka commented Dec 15, 2023

No description provided.

@ydakuka ydakuka force-pushed the add-enumerable-lazy branch 3 times, most recently from 806054e to cd1789d Compare December 15, 2023 14:18
@@ -4950,6 +4950,28 @@ LOREM
"when an unknown printer took a galley of type and scrambled it to make a type specimen book."
----

== Enumerable
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might be good to check if something else can't be moved under this section as well, as it's kind of weird to have a section with a single entry in it.

README.adoc Outdated
@@ -4950,6 +4950,28 @@ LOREM
"when an unknown printer took a galley of type and scrambled it to make a type specimen book."
----

== Enumerable

=== lazy [[lazy]]
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd use a more descriptive heading here - something like Leverage lazy to optimize memory usage or whatever.

SIZE = 10
MAX = 10_000_000

# bad
Copy link
Collaborator

Choose a reason for hiding this comment

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

You might also want to mention that this will instantiate a huge collection first, so it's clearer what the problem and value proposition is.

@ydakuka ydakuka force-pushed the add-enumerable-lazy branch 2 times, most recently from e86c786 to 214df9c Compare December 17, 2023 21:28
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.

2 participants