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

Casey's miscellaneous branch #3293

Merged
merged 5 commits into from
Jan 11, 2023
Merged

Conversation

CaseyCarter
Copy link
Member

@CaseyCarter CaseyCarter commented Dec 15, 2022

Some changes too small to submit individually that I've been accumulating.

  • ranges_algorithm_support.hpp includes <cstddef> but uses size_t unqualified; add std::-qualifications.
  • MSVC (unreleased) and Clang (unreleased) have fixed __is_aggregate, but EDG has not. Disable the workaround in internal testing to avoid regression, and update the TRANSITION comment appropriately.
  • _To_signed_like is unused, remove it.
  • Improve error when _Verify_offset is called on an invalidated vector iterator.
  • Mention set_environment.bat in README.md, and use it in the "How To Consume" example.

* MSVC has fixed `__is_aggregate`, so we can disable the workaround for internal testing.
* Clang has implemented the `__is_aggregate` fix.
* I filed VSO-1690654 to request a similar fix for EDG.

Update the TRANSITION comment for the workaround appropriatelu.
IIRC I added this while speculatively implementing the PR of an LWG issue that was later changed to no longer need _`to-signed-like`_.
@CaseyCarter CaseyCarter added the enhancement Something can be improved label Dec 15, 2022
@CaseyCarter CaseyCarter requested a review from a team as a code owner December 15, 2022 23:51
…ME.md

... and use it to simplify the example.
@StephanTLavavej StephanTLavavej self-assigned this Jan 11, 2023
@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 9720e8f into microsoft:main Jan 11, 2023
@StephanTLavavej
Copy link
Member

Miscellaneous thanks! 😹 🐈 🐈‍⬛

@CaseyCarter CaseyCarter deleted the misc branch January 11, 2023 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Something can be improved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants