Fix check-webkit-style errors in Source/wtf/text/.

third_party/WebKit/Source/wtf/text/UTF8.cpp

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2010 Patrick Gansterer <paroga@paroga.com>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 37 matching lines @@
 }
 
 inline int inlineUTF8SequenceLength(char b0)
 {
     return isASCII(b0) ? 1 : inlineUTF8SequenceLengthNonASCII(b0);
 }
 
 // Once the bits are split out into bytes of UTF-8, this is a mask OR-ed
 // into the first byte, depending on how many bytes follow.  There are
 // as many entries in this table as there are UTF-8 sequence types.
-// (I.e., one byte sequence, two byte... etc.). Remember that sequencs
+// (I.e., one byte sequence, two byte... etc.). Remember that sequences
 // for *legal* UTF-8 will be 4 or fewer bytes total.
 static const unsigned char firstByteMark[7] = { 0x00, 0x00, 0xC0, 0xE0, 0xF0, 0xF8, 0xFC };
 
 ConversionResult convertLatin1ToUTF8(
-                                     const LChar** sourceStart, const LChar* sourceEnd,
-                                     char** targetStart, char* targetEnd)
+    const LChar** sourceStart, const LChar* sourceEnd,
+    char** targetStart, char* targetEnd)
 {
     ConversionResult result = conversionOK;
     const LChar* source = *sourceStart;
     char* target = *targetStart;
     while (source < sourceEnd) {
         UChar32 ch;
         unsigned short bytesToWrite = 0;
         const UChar32 byteMask = 0xBF;
         const UChar32 byteMark = 0x80;
         const LChar* oldSource = source; // In case we have to back up because of target overflow.
@@ skipping 82 matching lines @@
         }
 
         target += bytesToWrite;
         if (target > targetEnd) {
             source = oldSource; // Back up source pointer!
             target -= bytesToWrite;
             result = targetExhausted;
             break;
         }
         switch (bytesToWrite) { // note: everything falls through.
-            case 4: *--target = (char)((ch | byteMark) & byteMask); ch >>= 6;
-            case 3: *--target = (char)((ch | byteMark) & byteMask); ch >>= 6;
-            case 2: *--target = (char)((ch | byteMark) & byteMask); ch >>= 6;
-            case 1: *--target =  (char)(ch | firstByteMark[bytesToWrite]);
+        case 4:
+            *--target = (char)((ch | byteMark) & byteMask);
+            ch >>= 6;
+        case 3:
+            *--target = (char)((ch | byteMark) & byteMask);
+            ch >>= 6;
+        case 2:
+            *--target = (char)((ch | byteMark) & byteMask);
+            ch >>= 6;
+        case 1:
+            *--target =  (char)(ch | firstByteMark[bytesToWrite]);
         }
         target += bytesToWrite;
     }
     *sourceStart = source;
     *targetStart = target;
     return result;
 }
 
 // This must be called with the length pre-determined by the first byte.
 // If presented with a length > 4, this returns false.  The Unicode
 // definition of UTF-8 goes up to 4-byte sequences.
 static bool isLegalUTF8(const unsigned char* source, int length)
 {
     unsigned char a;
     const unsigned char* srcptr = source + length;
     switch (length) {
-        default: return false;
-        // Everything else falls through when "true"...
-        case 4: if ((a = (*--srcptr)) < 0x80 || a > 0xBF) return false;
-        case 3: if ((a = (*--srcptr)) < 0x80 || a > 0xBF) return false;
-        case 2: if ((a = (*--srcptr)) > 0xBF) return false;
+    default:
+        return false;
+    // Everything else falls through when "true"...
+    case 4:
+        if ((a = (*--srcptr)) < 0x80 || a > 0xBF)
+            return false;
+    case 3:
+        if ((a = (*--srcptr)) < 0x80 || a > 0xBF)
+            return false;
+    case 2:
+        if ((a = (*--srcptr)) > 0xBF)
+            return false;
 
+        // no fall-through in this inner switch
         switch (*source) {
-            // no fall-through in this inner switch
-            case 0xE0: if (a < 0xA0) return false; break;
-            case 0xED: if (a > 0x9F) return false; break;
-            case 0xF0: if (a < 0x90) return false; break;
-            case 0xF4: if (a > 0x8F) return false; break;
-            default:   if (a < 0x80) return false;
+        case 0xE0:
+            if (a < 0xA0)
+                return false;
+            break;
+        case 0xED:
+            if (a > 0x9F)
+                return false;
+            break;
+        case 0xF0:
+            if (a < 0x90)
+                return false;
+            break;
+        case 0xF4:
+            if (a > 0x8F)
+                return false;
+            break;
+        default:
+            if (a < 0x80)
+                return false;
         }
 
-        case 1: if (*source >= 0x80 && *source < 0xC2) return false;
+    case 1:
+        if (*source >= 0x80 && *source < 0xC2)
+            return false;
     }
     if (*source > 0xF4)
         return false;
     return true;
 }
 
 // Magic values subtracted from a buffer value during UTF8 conversion.
 // This table contains as many values as there might be trailing bytes
 // in a UTF-8 sequence.
 static const UChar32 offsetsFromUTF8[6] = { 0x00000000UL, 0x00003080UL, 0x000E2080UL, 0x03C82080UL, static_cast<UChar32>(0xFA082080UL), static_cast<UChar32>(0x82082080UL) };
 
 static inline UChar32 readUTF8Sequence(const char*& sequence, unsigned length)
 {
     UChar32 character = 0;
 
     // The cases all fall through.
     switch (length) {
-        case 6: character += static_cast<unsigned char>(*sequence++); character <<= 6;
-        case 5: character += static_cast<unsigned char>(*sequence++); character <<= 6;
-        case 4: character += static_cast<unsigned char>(*sequence++); character <<= 6;
-        case 3: character += static_cast<unsigned char>(*sequence++); character <<= 6;
-        case 2: character += static_cast<unsigned char>(*sequence++); character <<= 6;
-        case 1: character += static_cast<unsigned char>(*sequence++);
+    case 6:
+        character += static_cast<unsigned char>(*sequence++);
+        character <<= 6;
+    case 5:
+        character += static_cast<unsigned char>(*sequence++);
+        character <<= 6;
+    case 4:
+        character += static_cast<unsigned char>(*sequence++);
+        character <<= 6;
+    case 3:
+        character += static_cast<unsigned char>(*sequence++);
+        character <<= 6;
+    case 2:
+        character += static_cast<unsigned char>(*sequence++);
+        character <<= 6;
+    case 1:
+        character += static_cast<unsigned char>(*sequence++);
     }
 
     return character - offsetsFromUTF8[length - 1];
 }
 
 ConversionResult convertUTF8ToUTF16(
     const char** sourceStart, const char* sourceEnd,
     UChar** targetStart, UChar* targetEnd, bool* sourceAllASCII, bool strict)
 {
     ConversionResult result = conversionOK;
@@ skipping 20 matching lines @@
             break;
         }
 
         if (U_IS_BMP(character)) {
             // UTF-16 surrogate values are illegal in UTF-32
             if (U_IS_SURROGATE(character)) {
                 if (strict) {
                     source -= utf8SequenceLength; // return to the illegal value itself
                     result = sourceIllegal;
                     break;
-                } else {
-                    *target++ = replacementCharacter;
-                    orAllData |= replacementCharacter;
                 }
+                *target++ = replacementCharacter;
+                orAllData |= replacementCharacter;
             } else {
                 *target++ = static_cast<UChar>(character); // normal case
                 orAllData |= character;
             }
         } else if (U_IS_SUPPLEMENTARY(character)) {
             // target is a character in range 0xFFFF - 0x10FFFF
             if (target + 1 >= targetEnd) {
                 source -= utf8SequenceLength; // Back up source pointer!
                 result = targetExhausted;
                 break;
@@ skipping 39 matching lines @@
         }
 
         int utf8SequenceLength = inlineUTF8SequenceLengthNonASCII(*data);
         dataLength += utf8SequenceLength;
 
         if (!dataEnd) {
             for (int i = 1; i < utf8SequenceLength; ++i) {
                 if (!data[i])
                     return 0;
             }
-        } else if (dataEnd - data < utf8SequenceLength)
+        } else if (dataEnd - data < utf8SequenceLength) {
             return 0;
+        }
 
         if (!isLegalUTF8(reinterpret_cast<const unsigned char*>(data), utf8SequenceLength))
             return 0;
 
         UChar32 character = readUTF8Sequence(data, utf8SequenceLength);
         ASSERT(!isASCII(character));
 
         if (U_IS_BMP(character)) {
             // UTF-16 surrogate values are illegal in UTF-32
             if (U_IS_SURROGATE(character))
                 return 0;
             stringHasher.addCharacter(static_cast<UChar>(character)); // normal case
             utf16Length++;
         } else if (U_IS_SUPPLEMENTARY(character)) {
-            stringHasher.addCharacters(static_cast<UChar>(U16_LEAD(character)),
-                                       static_cast<UChar>(U16_TRAIL(character)));
+            stringHasher.addCharacters(static_cast<UChar>(U16_LEAD(character)), static_cast<UChar>(U16_TRAIL(character)));
             utf16Length += 2;
-        } else
+        } else {
             return 0;
+        }
     }
 
     return stringHasher.hashWithTop8BitsMasked();
 }
 
 template<typename CharType>
 ALWAYS_INLINE bool equalWithUTF8Internal(const CharType* a, const CharType* aEnd, const char* b, const char* bEnd)
 {
     while (b < bEnd) {
         if (isASCII(*b)) {
@@ skipping 17 matching lines @@
             // UTF-16 surrogate values are illegal in UTF-32
             if (U_IS_SURROGATE(character))
                 return false;
             if (*a++ != character)
                 return false;
         } else if (U_IS_SUPPLEMENTARY(character)) {
             if (*a++ != U16_LEAD(character))
                 return false;
             if (*a++ != U16_TRAIL(character))
                 return false;
-        } else
+        } else {
             return false;
+        }
     }
 
     return a == aEnd;
 }
 
 bool equalUTF16WithUTF8(const UChar* a, const UChar* aEnd, const char* b, const char* bEnd)
 {
     return equalWithUTF8Internal(a, aEnd, b, bEnd);
 }
 
 bool equalLatin1WithUTF8(const LChar* a, const LChar* aEnd, const char* b, const char* bEnd)
 {
     return equalWithUTF8Internal(a, aEnd, b, bEnd);
 }
 
 } // namespace Unicode
 } // namespace WTF