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

1.12.1 parameter highlighting issue. #256

Closed
matter123 opened this issue Jun 25, 2019 · 2 comments · Fixed by #258
Closed

1.12.1 parameter highlighting issue. #256

matter123 opened this issue Jun 25, 2019 · 2 comments · Fixed by #258
Labels
🐛 Bug Something isn't working High Priority Common or breaks highlighting of more than just itself

Comments

@matter123
Copy link
Collaborator

I'll try to get a nicer repro in the morning but this shows the issue for now.

Example:

class basic_string {
	basic_string() noexcept(noexcept(Allocator())) : basic_string(Allocator()) {}
	explicit basic_string(const Allocator &alloc) noexcept
	    : __string({._s = {{}, _SSO_Cap}}, alloc){};
	// see LWG 3076 for defect report, disabling comes from libc++
	template <
	    class = std::enable_if_t<std::__is_allocator<Allocator>::value, nullptr_t>>
	basic_string(size_type count, CharT ch, const Allocator &alloc = Allocator())
	    : __string({}, alloc) {
		CharT *buf = _New_allocate(count);
		for(size_t s = 0; s < count; s++) { traits_type::assign(buf[s], ch); }
		traits_type::assign(buf[count], CharT());
	}
};

Image:
Screenshot from 2019-06-24 22-50-15

@matter123 matter123 added 🐛 Bug Something isn't working 🔍 investigating More information is being gathered High Priority Common or breaks highlighting of more than just itself labels Jun 25, 2019
@matter123
Copy link
Collaborator Author

matter123 commented Jun 25, 2019

Caused by the noexcept(noexcept()) on line 2.

@matter123 matter123 mentioned this issue Jun 25, 2019
@jeff-hykin
Copy link
Owner

alright I'll get this merged in today

@matter123 matter123 removed the 🔍 investigating More information is being gathered label Jun 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working High Priority Common or breaks highlighting of more than just itself
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants