Update comment.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 28 Jan 2014 17:34:29 +0000 (12:34 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 28 Jan 2014 17:34:29 +0000 (12:34 -0500)
generate_normalized_query() no longer needs to truncate text, but this
one comment didn't get the memo.  Per Peter Geoghegan.

contrib/pg_stat_statements/pg_stat_statements.c

index aa6ae375773a6a550ec96af72ff7851c9b48a2e6..3c6e84d0c40b865ffd71e92c4e758f884073ab88 100644 (file)
@@ -2726,7 +2726,7 @@ generate_normalized_query(pgssJumbleState *jstate, const char *query,
        if (tok_len < 0)
            continue;           /* ignore any duplicates */
 
-       /* Copy next chunk, or as much as will fit */
+       /* Copy next chunk (what precedes the next constant) */
        len_to_wrt = off - last_off;
        len_to_wrt -= last_tok_len;
 
@@ -2734,6 +2734,7 @@ generate_normalized_query(pgssJumbleState *jstate, const char *query,
        memcpy(norm_query + n_quer_loc, query + quer_loc, len_to_wrt);
        n_quer_loc += len_to_wrt;
 
+       /* And insert a '?' in place of the constant token */
        norm_query[n_quer_loc++] = '?';
 
        quer_loc = off + tok_len;