Merge lp:~akopytov/percona-server/bug1132350 into lp:percona-server/5.5

Proposed by Alexey Kopytov
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 458
Proposed branch: lp:~akopytov/percona-server/bug1132350
Merge into: lp:percona-server/5.5
Diff against target: 34 lines (+9/-11)
1 file modified
Percona-Server/strings/ctype-simple.c (+9/-11)
To merge this branch: bzr merge lp:~akopytov/percona-server/bug1132350
Reviewer Review Type Date Requested Status
Sergei Glushchenko (community) g2 Approve
Review via email: mp+150229@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexey Kopytov (akopytov) wrote :
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

Approve

review: Approve (g2)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Percona-Server/strings/ctype-simple.c'
--- Percona-Server/strings/ctype-simple.c 2011-06-30 15:46:53 +0000
+++ Percona-Server/strings/ctype-simple.c 2013-02-24 15:11:22 +0000
@@ -76,23 +76,21 @@
76 const uchar *src, size_t srclen)76 const uchar *src, size_t srclen)
77{77{
78 uchar *map= cs->sort_order;78 uchar *map= cs->sort_order;
79 size_t dstlen= len;79 const uchar *end;
80 set_if_smaller(len, srclen);80
81 if (dest != src)81 if (likely(len <= srclen))
82 {82 {
83 const uchar *end;83 for (end = src + len; src < end ; )
84 for ( end=src+len; src < end ; )
85 *dest++= map[*src++];84 *dest++= map[*src++];
86 }85 }
87 else86 else
88 {87 {
89 const uchar *end;88 for (end = src + srclen; src < end ; )
90 for ( end=dest+len; dest < end ; dest++)89 *dest++= map[*src++];
91 *dest= (char) map[(uchar) *dest];90 memset(dest, ' ', len - srclen);
92 }91 }
93 if (dstlen > len)92
94 bfill(dest, dstlen - len, ' ');93 return len;
95 return dstlen;
96}94}
9795
9896

Subscribers

People subscribed via source and target branches