[DBD::Pg 2/5] Use lookup table for placeholder string

[dbdpg-...@bucardo.org]

Committed by =?UTF-8?q?Dagfinn=20Ilmari=20Manns=C3=A5ker?= <ilm...@ilmari.org>

Subject: [DBD::Pg 2/5] Use lookup table for placeholder string

---
dbdimp.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/dbdimp.c b/dbdimp.c
index a8959fc..9e97701 100644
--- a/dbdimp.c
+++ b/dbdimp.c
@@ -1698,6 +1698,10 @@ int dbd_st_prepare_sv (SV * sth, imp_sth_t * imp_sth, SV * statement_sv, SV * at
} /* end of dbd_st_prepare */


+static const char *placeholder_string[4] = {
+ "", "?", "$1", ":foo"
+};
+
/* ================================================================== */
static void pg_st_split_statement (pTHX_ imp_sth_t * imp_sth, int version, char * statement)
{
@@ -2066,8 +2070,8 @@ static void pg_st_split_statement (pTHX_ imp_sth_t * imp_sth, int version, char
if (placeholder_type!=0) {
if (imp_sth->placeholder_type && placeholder_type != imp_sth->placeholder_type)
croak("Cannot mix placeholder styles \"%s\" and \"%s\"",
- 1==imp_sth->placeholder_type ? "?" : 2==imp_sth->placeholder_type ? "$1" : ":foo",
- 1==placeholder_type ? "?" : 2==placeholder_type ? "$1" : ":foo");
+ placeholder_string[imp_sth->placeholder_type],
+ placeholder_string[placeholder_type]);
}

/* Move on to the next letter unless we found a placeholder, or we are at the end of the string */
--
1.8.4