On 28.09.2013 12:44, David Rowley wrote:
The macro for test 4 was as follows:
#define appendStringInfoStringConst(buf, s) appendBinaryStringInfo(buf,
(s), sizeof(s)-1)
If that makes any difference in practice, I wonder if we should just do:
#define appendStringInfoString(buf, s) appendBinaryStringInfo(buf, (s),
strlen(s))
With a compiler worth its salt, the strlen(s) will be optimized into a
constant, if s is a constant. If it's not a constant, we'll just end up
calling strlen(), like appendStringInfoString would anyway. That would
turn a single function call into two in all of the non-constant
callsites, though, bloating the code, so it might not be a win overall.
- Heikki
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
