ericniebler / range-v3

Range library for C++14/17/20, basis for C++20's std::ranges
Other
4.11k stars 440 forks source link

Update deprecation message #1619

Closed DerekTBrown closed 3 years ago

DerekTBrown commented 3 years ago

The following code triggers a CLANGTIDY lint:

return ranges::views::empty<int64_t>();

is deprecated Replace views::empty() with views::empty<>. It is now a variable template.

If I do as the lint suggests, I get a template error:

return ranges::views::empty<>;

error: too few template arguments for variable template 'empty'

I suspect we actually want the lint message to be:

is deprecated Replace views::empty() with views::empty<T>. It is now a variable template.

ericniebler commented 3 years ago

Thanks!