xenia/home
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use decltype(auto) return type for some to_string functions

Browse Source 
Remove explicit return type thats just duplicating the return
expression.
This commit is contained in:
Maxime Coste 2017-08-14 11:29:55 +07:00
parent 1b1239b25a
commit 6aa2388700
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/string.hh
View File

@ -399,8 +399,7 @@ InplaceString<23> to_string(float val);
InplaceString<7> to_string(Codepoint c); InplaceString<7> to_string(Codepoint c);
template<typename RealType, typename ValueType> template<typename RealType, typename ValueType>
decltype(to_string(std::declval<ValueType>())) decltype(auto) to_string(const StronglyTypedNumber<RealType, ValueType>& val)
to_string(const StronglyTypedNumber<RealType, ValueType>& val)
{ {
return to_string((ValueType)val); return to_string((ValueType)val);
} }
@ -411,7 +410,7 @@ namespace detail
template<typename T> constexpr bool is_string = std::is_convertible<T, StringView>::value; template<typename T> constexpr bool is_string = std::is_convertible<T, StringView>::value;
template<typename T, class = std::enable_if_t<not is_string<T>>> template<typename T, class = std::enable_if_t<not is_string<T>>>
auto format_param(const T& val) -> decltype(to_string(val)) { return to_string(val); } decltype(auto) format_param(const T& val) { return to_string(val); }
template<typename T, class = std::enable_if_t<is_string<T>>> template<typename T, class = std::enable_if_t<is_string<T>>>
StringView format_param(const T& val) { return val; } StringView format_param(const T& val) { return val; }