std::format, concepts, range-for using and string_view#298
Conversation
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
|
Tick the box to add this pull request to the merge queue (same as
|
| /// Satisfied when interface T can be constructed from Args according to its InterfaceTraits. | ||
| /** | ||
| * Replaces the std::enable_if_t<class_loader::is_interface_constructible_v<...>> SFINAE guard | ||
| * on the create*Instance() methods with a named constraint, yielding clearer diagnostics. | ||
| */ | ||
| template<typename T, typename ... Args> | ||
| concept InterfaceConstructible = class_loader::is_interface_constructible_v<T, Args...>; | ||
|
|
There was a problem hiding this comment.
class_loader already provides this concept. I think we should use that, instead of reimplementing it.
Then we can replace all uses of InterfaceConstructible with class_loader::InterfaceConstructible
There was a problem hiding this comment.
@ahcorde this makes sense to me. Could you make the change?
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
|
Pulls: #298
|
3 participants
Claude Opus 4.7