Menu

#1 use stringstream

open
nobody
None
5
2004-02-09
2004-02-09
Anonymous
No

Hi.

Thanks for making this library!

Why not use std::stringstream instead of "std::string
SQL, nextSQL;" (or somehow use std::stringstream) in
query.hh?

This change can allow for type safety and simplicity,
and may reduce the need for "Some stream operators for
stuffing SQL into the Query."

This may help eliminate the rationale mentioned in
query.cc (and referenced below) regarding "overall less
work" with respect to stream operators:

/*
* An oldie but a goodie: formatstr's for putting
together queries
* with esoteric types that aren't covered by the usual
stream
* operators above (which could be expanded, but won't,
because this
* method is more useful and overall less work than to
do each
* conversion ourselves).
*/

What do you think? If you wish to correspond, you can
send to deckrider at yahoo dot com.

Thanks,
Dave.

Discussion


Log in to post a comment.