Skip to content

Commit

Permalink
added clarification comments
Browse files Browse the repository at this point in the history
  • Loading branch information
silverqx committed Nov 13, 2022
1 parent 106af9d commit 5993ac2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 2 additions & 1 deletion include/orm/ormconcepts.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ namespace Query
using QueryBuilder = Query::Builder;

/* Don't test for eg. QString and also const QString & in the std::converible_to<>,
test only for the QString, it doesn't make sense to test for both. */
test only for the QString, it doesn't make sense to test for both.
The std::convertible_to<T, Orm::QueryBuilder &> is correct. */

/*! Concept for a queryable parameter. */
template<typename T>
Expand Down
1 change: 1 addition & 0 deletions tom/include/tom/terminal.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ TINYORM_BEGIN_COMMON_NAMESPACE

namespace Tom
{
/* std::w/ostream don't have copy ctor, so need to check for const std::ostream &. */
/*! Concept for the ostream and wostream. */
template<typename T>
concept OStreamConcept = std::convertible_to<T, const std::ostream &> ||
Expand Down

0 comments on commit 5993ac2

Please sign in to comment.