Skip to content

Commit

Permalink
853: Silly typo in obscure stream_to constructor.
Browse files Browse the repository at this point in the history
Fixes: #853

This has been broken (at compile time!) for _years_ but someone finally
noticed.  It's a deprecated constructor, so perhaps I ought to delete
it rather than fix it.
  • Loading branch information
jtv committed Jun 24, 2024
1 parent 0989d34 commit 139f504
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
1 change: 1 addition & 0 deletions NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
- Bump gcc/clang/postgres minimum version requirements.
- Another fix to the readthedocs documentation build. (#845)
- Work around CMake 3.30 renaming one of its functions. (#851)
- Fix obscure deprecated `stream_to` constructor. (#853)
7.9.1
- Fix bad conversion of array of empty strings to string. (#816)
- Move `[[likely]]` feature check back to compile time, to speed up configure.
Expand Down
7 changes: 3 additions & 4 deletions include/pqxx/stream_to.hxx
Original file line number Diff line number Diff line change
Expand Up @@ -481,11 +481,10 @@ inline stream_to::stream_to(
Iter columns_end) :
stream_to{
tx,
tx.quote_name(
table_name,
separated_list(",", columns_begin, columns_end, [&tx](auto col) {
tx.quote_name(table_name),
separated_list(",", columns_begin, columns_end, [&tx](auto col) {
return tx.quote_name(*col);
}))}
})}
{}
} // namespace pqxx
#endif

0 comments on commit 139f504

Please sign in to comment.