Skip to content

Add support for checked #151

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

Merged
merged 6 commits into from
May 17, 2018
Merged

Add support for checked #151

merged 6 commits into from
May 17, 2018

Conversation

kevmoo
Copy link
Collaborator

@kevmoo kevmoo commented May 11, 2018

No description provided.

@kevmoo kevmoo force-pushed the checked_fun branch 6 times, most recently from 72a149f to b184e1c Compare May 17, 2018 01:01
@kevmoo kevmoo changed the title VERY WIP: Checked fun Add support for checked May 17, 2018
@kevmoo kevmoo requested review from natebosch and jakemac53 May 17, 2018 14:38
@kevmoo kevmoo force-pushed the checked_fun branch 2 times, most recently from 43ce7a1 to e1367f4 Compare May 17, 2018 14:46
@kevmoo
Copy link
Collaborator Author

kevmoo commented May 17, 2018

@natebosch @jakemac53 – I split this out into commits that should be easy to consume individually

@kevmoo kevmoo merged commit 583a0c3 into master May 17, 2018
@kevmoo kevmoo deleted the checked_fun branch May 17, 2018 15:25
@natebosch
Copy link
Member

What is checked?

/// for large object graphs.
/// [Map] and [List] instances created during serialization.
///
/// This will increase the code size, but it may improve runtime performance,
Copy link
Member

Choose a reason for hiding this comment

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

These line lengths look inconsistent. Would [Map] have fit within 80 characters on line 45 or is this line going over 80 characters?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I often wrap at the sentence start. It doesn't affect markdown rendering, but it doe make it easier to edit sentences without rewrapping the entire paragraph.

Copy link
Member

Choose a reason for hiding this comment

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

neither of these lines are wrapped at a sentence start...

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.

3 participants