update working draft revision to N4917

[fsb4000]

https://wg21.link/n4917 is not working currently but probably will be updated before this PR will be merged.

github release: https:/cplusplus/draft/releases/download/n4917/n4917.pdf

[AlexGuteniev]

The link works fine for me

[frederick-vs-ja]

How about consistently updating the references to working drafts in comments/static_assert messages? I've updated the referenced number to N4910 in <xstring> and <cmath>, but not in other headers.

(It's weird that even change to README.md requires running tests...)

[StephanTLavavej]

@frederick-vs-ja

How about consistently updating the references to working drafts in comments/static_assert messages?

That's tracked by #182. It can't be automated (since we have to look out for paragraph numbers changing and wording being moved), but if you'd like to prepare a PR, go ahead! I'd say that it can be done incrementally (no need to update the entire codebase all at once; individual files already cite a mix of WP numbers so I don't even think intra-file consistency is necessary, although of course it's nice). I'd also say that the static_assert messages which are displayed to users are somewhat higher priority than code comments.

(It's weird that even change to README.md requires running tests...)

We need to run Code Format Validation to detect disallowed characters etc. It would be possible to configure Azure Pipelines to skip the build and test stages for readme changes - I see how to do it, but it would take some time, and README-only updates haven't been that frequent.

[fsb4000]

Yes, https://wg21.link/n4917 works for me via proxy but not directly.
Probably it's my ISP's problem, not that site's.

[StephanTLavavej]

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

[StephanTLavavej]

Thanks for updating the WP link! ⛓️ ✅ 😺