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

Value-initialize channel and pixel value members #273

Conversation

mloskot
Copy link
Member

@mloskot mloskot commented Mar 30, 2019

Add tests to ensure member values of channel and pixel classes are determined, zero-initialized.

Tidy up formatting.

References

Fixes #270

Tasklist

  • Add test case(s)
  • Review
  • Adjust for comments
  • All CI builds and checks have passed

Add tests to ensure member values of channel and pixel classes are
determined, zero-initialized.
Fixes boostorg#270

Tidy up formatting.
@mloskot mloskot added cat/bug cat/refactoring core labels Mar 30, 2019
@mloskot mloskot self-assigned this Mar 30, 2019
@mloskot mloskot requested a review from stefanseefeld Mar 30, 2019
@mloskot mloskot added this to the Boost 1.71 milestone Mar 30, 2019
@mloskot mloskot merged commit c6c5336 into boostorg:develop Apr 1, 2019
9 checks passed
@mloskot mloskot deleted the ml/fix-default-initialization-in-pixel-classes branch Apr 1, 2019
mloskot added a commit that referenced this issue Jan 22, 2021
…ts (#542)

If `Element` is a reference, then Element v{} is ill-formed.

Refines #273 which aimed to correctly value-initialize channel and pixel value members
meshtag pushed a commit to meshtag/gil that referenced this issue Apr 21, 2021
…ts (boostorg#542)

If `Element` is a reference, then Element v{} is ill-formed.

Refines boostorg#273 which aimed to correctly value-initialize channel and pixel value members
meshtag pushed a commit to meshtag/gil that referenced this issue Apr 22, 2021
…ts (boostorg#542)

If `Element` is a reference, then Element v{} is ill-formed.

Refines boostorg#273 which aimed to correctly value-initialize channel and pixel value members
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/bug cat/refactoring core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants