Fix review comment errors.

This commit is contained in:
John Ralls 2016-10-29 12:40:57 -07:00
parent 00880cd8f5
commit b8024fa9ea
3 changed files with 6 additions and 6 deletions

View File

@ -156,7 +156,7 @@ GncDbiSqlResult::IteratorImpl::get_time64_at_col (const char* col) const
auto type = dbi_result_get_field_type (m_inst->m_dbi_result, col);
auto attrs = dbi_result_get_field_attribs (m_inst->m_dbi_result, col);
if (type != DBI_TYPE_DATETIME)
throw (std::invalid_argument{"Requested double from non-double column."});
throw (std::invalid_argument{"Requested time64 from non-time64 column."});
gnc_push_locale (LC_NUMERIC, "C");
#if HAVE_LIBDBI_TO_LONGLONG
/* A less evil hack than the one equrie by libdbi-0.8, but

View File

@ -158,10 +158,10 @@ gnc_sql_get_object_backend(const std::string& type)
[type](const OBEEntry& entry){
return type == std::get<0>(entry);
});
auto obe = std::get<1>(*entry);
if (entry != backend_registry.end())
return obe;
return nullptr;
if (entry == backend_registry.end())
return nullptr;
return std::get<1>(*entry);
}
void

View File

@ -355,7 +355,7 @@ protected:
* Row of SQL Query results.
*
* This is a "pointer" class of a pimpl pattern, the implementation being
* GncSqlResul::IteratorImpl. It's designed to present a std::forward_iterator
* GncSqlResult::IteratorImpl. It's designed to present a std::forward_iterator
* like interface for use with range-for while allowing for wrapping a C API.
*
* Important Implementation Note: Operator++() as written requires that the