Addendum #1 to the fix for bug #18359924

Removed unused variable.
Fixed long (>80 lines)

1 files changed, 11 insertions, 5 deletions

diff --git a/sql/sql_string.cc b/sql/sql_string.cc
index b9a9ce92cd6..41725128415 100644
--- a/sql/sql_string.cc
+++ b/sql/sql_string.cc
@@ -241,11 +241,17 @@ bool String::needs_conversion_on_storage(uint32 arg_length,
 {
   uint32 offset;
   return (needs_conversion(arg_length, cs_from, cs_to, &offset) ||
-          (cs_from == &my_charset_bin &&      /* force conversion when storing a binary string */
-           cs_to != &my_charset_bin &&        /* into a non-binary destination */
-           (                                  /* and any of the following is true :*/
-            cs_to->mbminlen != cs_to->mbmaxlen || /* it's a variable length encoding */
-            cs_to->mbminlen > 2 ||            /* longer than 2 bytes : neither 1 byte nor ucs2 */
+          /* force conversion when storing a binary string */
+          (cs_from == &my_charset_bin &&
+          /* into a non-binary destination */
+           cs_to != &my_charset_bin &&
+           /* and any of the following is true :*/
+           (
+            /* it's a variable length encoding */
+            cs_to->mbminlen != cs_to->mbmaxlen ||
+            /* longer than 2 bytes : neither 1 byte nor ucs2 */
+            cs_to->mbminlen > 2 ||
+            /* and is not a multiple of the char byte size */
             0 != (arg_length % cs_to->mbmaxlen)
            )
           )