Log In - Home Page

Patch: Allow encoding specific character incrementer.

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2011-09
Topic Performance
Patch Status Committed
Author Kyotaro Horiguchi
Reviewers Robert Haas
Committer Robert Haas
Close Date 2011-10-29
Patch by horiguti on 2011-07-12 07:00:43 AM: Initial version.
Patch by horiguti on 2011-09-14 02:46:49 AM: Rebased to master HEAD.
Char increment sanity check programs used for this patch attached.
Review by rhaas on 2011-09-22 12:35:50 AM: Review of previous discussion; is this the right approach?
Comment by tgl on 2011-09-22 02:14:07 AM: Some responses to Robert's comments.
Patch by horiguti on 2011-09-29 10:28:25 AM: Style modified. memcpy check. Make smaller.
Review by rhaas on 2011-10-03 06:14:41 PM: More review comments.
Patch by rhaas on 2011-10-07 04:16:48 PM: New patch.
Review by rhaas on 2011-10-07 04:25:38 PM: Looks pretty good - can we add a few more comments?
Patch by horiguti on 2011-10-18 01:55:27 AM: Version 4 patch. add comment to increment functions and bug fix.
Review by rhaas on 2011-10-19 02:01:29 AM: I think pg_utf8_increment() contains some bogus logic.
Comment by horiguti on 2011-10-29 05:41:44 PM: I try to describe another aspect of this code.

Add Comment

Please log in to comment on this patch.