]> granicus.if.org Git - icu/commitdiff
ICU-20508 Fixing -Wextra-semi in library code.
authorShane Carr <shane@unicode.org>
Fri, 22 Mar 2019 20:58:36 +0000 (13:58 -0700)
committerShane F. Carr <shane@unicode.org>
Fri, 22 Mar 2019 22:29:45 +0000 (15:29 -0700)
22 files changed:
icu4c/source/common/cmemory.h
icu4c/source/common/dictbe.cpp
icu4c/source/common/dictionarydata.h
icu4c/source/common/rbbi.cpp
icu4c/source/common/rbbi_cache.h
icu4c/source/common/rbbiscan.h
icu4c/source/common/umutex.h
icu4c/source/common/unifiedcache.h
icu4c/source/i18n/csdetect.cpp
icu4c/source/i18n/number_decimalquantity.h
icu4c/source/i18n/number_patternstring.h
icu4c/source/i18n/numparse_types.h
icu4c/source/i18n/scriptset.h
icu4c/source/i18n/tzgnames.h
icu4c/source/i18n/tznames.cpp
icu4c/source/i18n/unicode/alphaindex.h
icu4c/source/i18n/unicode/dtitvfmt.h
icu4c/source/i18n/unicode/formattedvalue.h
icu4c/source/i18n/unicode/listformatter.h
icu4c/source/i18n/unicode/numberformatter.h
icu4c/source/i18n/unicode/numberrangeformatter.h
icu4c/source/i18n/unicode/reldatefmt.h

index f121bcb8f11a9f8aab8c4d5979401bbe421205cc..f19715caf420555830266f49402ec98b67c0de70 100644 (file)
@@ -298,7 +298,7 @@ public:
      */
     MaybeStackArray(int32_t newCapacity) : MaybeStackArray() {
         if (capacity < newCapacity) { resize(newCapacity); }
-    };
+    }
     /**
      * Destructor deletes the array (if owned).
      */
index 0c2612ad3ca6a77e240640d028e399b226da1ff7..c769138ae4b3919838073265f27c0e05447416ad 100644 (file)
@@ -103,8 +103,8 @@ private:
     int32_t   cpLengths[POSSIBLE_WORD_LIST_MAX];   // Word Lengths, in code points.
 
 public:
-    PossibleWord() : count(0), prefix(0), offset(-1), mark(0), current(0) {};
-    ~PossibleWord() {};
+    PossibleWord() : count(0), prefix(0), offset(-1), mark(0), current(0) {}
+    ~PossibleWord() {}
   
     // Fill the list of candidates if needed, select the longest, and return the number found
     int32_t   candidates( UText *text, DictionaryMatcher *dict, int32_t rangeEnd );
@@ -118,13 +118,13 @@ public:
   
     // Return the longest prefix this candidate location shares with a dictionary word
     // Return value is in code points.
-    int32_t   longestPrefix() { return prefix; };
+    int32_t   longestPrefix() { return prefix; }
   
     // Mark the current candidate as the one we like
-    void      markCurrent() { mark = current; };
+    void      markCurrent() { mark = current; }
     
     // Get length in code points of the marked word.
-    int32_t   markedCPLength() { return cpLengths[mark]; };
+    int32_t   markedCPLength() { return cpLengths[mark]; }
 };
 
 
index 73b7d6753ba2881fb8db4e7a8315e51b1a04ceed..0d303d9a8dcfdbcb255c24465d7cd8c509140923 100644 (file)
@@ -68,7 +68,7 @@ public:
  */
 class U_COMMON_API DictionaryMatcher : public UMemory {
 public:
-    DictionaryMatcher() {};
+    DictionaryMatcher() {}
     virtual ~DictionaryMatcher();
     // this should emulate CompactTrieDictionary::matches()
     /*  @param text      The text in which to look for matching words. Matching begins
index 41490250b0b21ae6140f4e2fe881057a19ae5e52..3c2f7203e3a8825f43fb658b99a39422c67e2105 100644 (file)
@@ -720,7 +720,7 @@ struct LookAheadResults {
     int32_t    fPositions[8];
     int16_t    fKeys[8];
 
-    LookAheadResults() : fUsedSlotLimit(0), fPositions(), fKeys() {};
+    LookAheadResults() : fUsedSlotLimit(0), fPositions(), fKeys() {}
 
     int32_t getPosition(int16_t key) {
         for (int32_t i=0; i<fUsedSlotLimit; ++i) {
index b4338c37490e2b3cdf1aa6343aa1f434748a0c86..7991d6c0c7bf4c9c06c1612801b4306a9d9736bd 100644 (file)
@@ -95,7 +95,7 @@ class RuleBasedBreakIterator::BreakCache: public UMemory {
                                 fTextIdx = fBI->fPosition = fBoundaries[fBufIdx];
                                 fBI->fRuleStatusIndex = fStatuses[fBufIdx];
                             }
-                };
+                }
 
 
     void        nextOL();
@@ -178,7 +178,7 @@ class RuleBasedBreakIterator::BreakCache: public UMemory {
     void dumpCache();
 
   private:
-    static inline int32_t   modChunkSize(int index) { return index & (CACHE_SIZE - 1); };
+    static inline int32_t   modChunkSize(int index) { return index & (CACHE_SIZE - 1); }
 
     static constexpr int32_t CACHE_SIZE = 128;
     static_assert((CACHE_SIZE & (CACHE_SIZE-1)) == 0, "CACHE_SIZE must be power of two.");
index 8bec8234e58f1e8b8fbb7cfd562e7d81cfda55dc..c51b4cf2a68aa18c0e0621a2c83fbeadfe5e1a03 100644 (file)
@@ -54,7 +54,7 @@ public:
     struct RBBIRuleChar {
         UChar32             fChar;
         UBool               fEscaped;
-        RBBIRuleChar() : fChar(0), fEscaped(FALSE) {};
+        RBBIRuleChar() : fChar(0), fEscaped(FALSE) {}
     };
 
     RBBIRuleScanner(RBBIRuleBuilder  *rb);
index 7858b893784e041505f4ca6f3adad6cc6fc72863..d0a7e7afc5bed4974cf1bcbcc4dbceab26b67e17 100755 (executable)
@@ -95,8 +95,8 @@ inline int32_t umtx_atomic_dec(u_atomic_int32_t *var) {
 struct UInitOnce {
     u_atomic_int32_t   fState;
     UErrorCode       fErrCode;
-    void reset() {fState = 0;};
-    UBool isReset() {return umtx_loadAcquire(fState) == 0;};
+    void reset() {fState = 0;}
+    UBool isReset() {return umtx_loadAcquire(fState) == 0;}
 // Note: isReset() is used by service registration code.
 //                 Thread safety of this usage needs review.
 };
index 4c9992be25eb572cf049dcfa8380ec6873af20f8..d6c9945126c2222b6c05c128fc231c865b6952b2 100644 (file)
@@ -137,7 +137,7 @@ class LocaleCacheKey : public CacheKey<T> {
  protected:
    Locale   fLoc;
  public:
-   LocaleCacheKey(const Locale &loc) : fLoc(loc) {};
+   LocaleCacheKey(const Locale &loc) : fLoc(loc) {}
    LocaleCacheKey(const LocaleCacheKey<T> &other)
            : CacheKey<T>(other), fLoc(other.fLoc) { }
    virtual ~LocaleCacheKey() { }
index 36552c4b76227caac4286a450cf5c54739c5e222..2436760a9b2ef12b9b8b4a8ede9872db8c1a946b 100644 (file)
@@ -36,9 +36,9 @@ U_NAMESPACE_BEGIN
 
 struct CSRecognizerInfo : public UMemory {
     CSRecognizerInfo(CharsetRecognizer *recognizer, UBool isDefaultEnabled)
-        : recognizer(recognizer), isDefaultEnabled(isDefaultEnabled) {};
+        : recognizer(recognizer), isDefaultEnabled(isDefaultEnabled) {}
 
-    ~CSRecognizerInfo() {delete recognizer;};
+    ~CSRecognizerInfo() {delete recognizer;}
 
     CharsetRecognizer *recognizer;
     UBool isDefaultEnabled;
index 13613cf569fe8fdbb87263f1b71e8dfa571c96b9..06cc836c7796f22ea7cf3628bc0f654291053a1e 100644 (file)
@@ -275,7 +275,7 @@ class U_I18N_API DecimalQuantity : public IFixedDecimal, public UMemory {
     inline bool isUsingBytes() { return usingBytes; }
 
     /** Visible for testing */
-    inline bool isExplicitExactDouble() { return explicitExactDouble; };
+    inline bool isExplicitExactDouble() { return explicitExactDouble; }
 
     bool operator==(const DecimalQuantity& other) const;
 
index 387f41bfd1596a186320ca8c9e19ff5793c77340..0c908679f3cb636ba2a945aed7df7305a8aeb67a 100644 (file)
@@ -98,7 +98,7 @@ struct U_I18N_API ParsedPatternInfo : public AffixPatternProvider, public UMemor
         int32_t offset = 0;
 
         explicit ParserState(const UnicodeString& _pattern)
-                : pattern(_pattern) {};
+                : pattern(_pattern) {}
 
         ParserState& operator=(ParserState&& src) U_NOEXCEPT {
             // Leave pattern reference alone; it will continue to point to the same place in memory,
index 0fd70faca991ff27c10659fbd1aa5597dadb1d1f..f837d8d2795a2b4c32421c5c3ea47624c1179b7f 100644 (file)
@@ -347,7 +347,7 @@ class U_I18N_API NumberParseMatcher {
      */
     virtual void postProcess(ParsedNumber&) const {
         // Default implementation: no-op
-    };
+    }
 
     // String for debugging
     virtual UnicodeString toString() const = 0;
index 385c3e3e534086869242915fd05647ed6b8cc7bb..99a71ec803f36da6a67098b87f9a78c3d0f98a89 100644 (file)
@@ -40,7 +40,7 @@ class U_I18N_API ScriptSet: public UMemory {
     ~ScriptSet();
 
     UBool operator == (const ScriptSet &other) const;
-    UBool operator != (const ScriptSet &other) const {return !(*this == other);};
+    UBool operator != (const ScriptSet &other) const {return !(*this == other);}
     ScriptSet & operator = (const ScriptSet &other);
 
     UBool      test(UScriptCode script, UErrorCode &status) const;
index 2bd76135d8a5e896f2baff5f4507244e2dbb2966..71d9d84dc0402f1ec643af6a12ae16f172175fdf 100644 (file)
@@ -46,7 +46,7 @@ public:
     static TimeZoneGenericNames* createInstance(const Locale& locale, UErrorCode& status);
 
     virtual UBool operator==(const TimeZoneGenericNames& other) const;
-    virtual UBool operator!=(const TimeZoneGenericNames& other) const {return !operator==(other);};
+    virtual UBool operator!=(const TimeZoneGenericNames& other) const {return !operator==(other);}
     virtual TimeZoneGenericNames* clone() const;
 
     UnicodeString& getDisplayName(const TimeZone& tz, UTimeZoneGenericNameType type,
index 36d27ca699b1a2713b7d7eb2bf79711695e25731..acd6aecdc0ce7e9ce2b489c999d182dc0409bb41 100644 (file)
@@ -108,7 +108,7 @@ public:
     virtual ~TimeZoneNamesDelegate();
 
     virtual UBool operator==(const TimeZoneNames& other) const;
-    virtual UBool operator!=(const TimeZoneNames& other) const {return !operator==(other);};
+    virtual UBool operator!=(const TimeZoneNames& other) const {return !operator==(other);}
     virtual TimeZoneNames* clone() const;
 
     StringEnumeration* getAvailableMetaZoneIDs(UErrorCode& status) const;
index 1c50ba217fc3109e4521b54c2879af72e5ad1318..91524a1ae52c995c5a2037b9d73db68965888389 100644 (file)
@@ -651,7 +651,7 @@ private:
      /**
       *   No assignment.
       */
-     AlphabeticIndex &operator =(const AlphabeticIndex & /*other*/) { return *this;};
+     AlphabeticIndex &operator =(const AlphabeticIndex & /*other*/) { return *this;}
 
     /**
      * No Equality operators.
index d4b9f93657f381c4c855fc97155b50e977a02b28..42d77d041f8b28574a3e2f7b07e94af9cebd205d 100644 (file)
@@ -60,7 +60,7 @@ class U_I18N_API FormattedDateInterval : public UMemory, public FormattedValue {
      * Default constructor; makes an empty FormattedDateInterval.
      * @draft ICU 64
      */
-    FormattedDateInterval() : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {};
+    FormattedDateInterval() : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {}
 
     /**
      * Move constructor: Leaves the source FormattedDateInterval in an undefined state.
index ce59335e5f9b5ecd9d77588f9ea11a187c36a0c7..60f97edd4b2686865b1b018ac8de82e31b1bc17d 100644 (file)
@@ -122,7 +122,7 @@ class U_I18N_API ConstrainedFieldPosition : public UMemory {
      */
     inline int32_t getCategory() const {
         return fCategory;
-    };
+    }
 
     /**
      * Gets the field for the current position.
@@ -135,7 +135,7 @@ class U_I18N_API ConstrainedFieldPosition : public UMemory {
      */
     inline int32_t getField() const {
         return fField;
-    };
+    }
 
     /**
      * Gets the INCLUSIVE start index for the current position.
@@ -147,7 +147,7 @@ class U_I18N_API ConstrainedFieldPosition : public UMemory {
      */
     inline int32_t getStart() const {
         return fStart;
-    };
+    }
 
     /**
      * Gets the EXCLUSIVE end index stored for the current position.
@@ -159,7 +159,7 @@ class U_I18N_API ConstrainedFieldPosition : public UMemory {
      */
     inline int32_t getLimit() const {
         return fLimit;
-    };
+    }
 
     ////////////////////////////////////////////////////////////////////
     //// The following methods are for FormattedValue implementers; ////
index 121dd192ceb158bbd29a1440c5ea8cac8d81becc..9ce8ec8617ce800d4ed03c26361026693ab72fa2 100644 (file)
@@ -85,7 +85,7 @@ class U_I18N_API FormattedList : public UMemory, public FormattedValue {
      * Default constructor; makes an empty FormattedList.
      * @draft ICU 64
      */
-    FormattedList() : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {};
+    FormattedList() : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {}
 
     /**
      * Move constructor: Leaves the source FormattedList in an undefined state.
index 4fd6830518d91123af6b115cba6e94ad036a3d7b..72f474886a079ecf0299bfc6e87b4fdadf0d4346 100644 (file)
@@ -2411,7 +2411,7 @@ class U_I18N_API FormattedNumber : public UMemory, public FormattedValue {
      * @draft ICU 64
      */
     FormattedNumber()
-        : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {};
+        : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {}
 
     /**
      * Move constructor: Leaves the source FormattedNumber in an undefined state.
index 4a386b8039416e7b7117d5ce6c3982be8ecc0778..a67ee72666ea8a86e1de91b8e17104b5b334eeae 100644 (file)
@@ -481,7 +481,7 @@ class U_I18N_API NumberRangeFormatterSettings {
         }
         fMacros.copyErrorTo(outErrorCode);
         return U_FAILURE(outErrorCode);
-    };
+    }
 
     // NOTE: Uses default copy and move constructors.
 
index 84d7c0a34d54c549da1da493964cad248abbd602..32344cd5f18d6a0bb56c180f1293f2dae0ce3f4a 100644 (file)
@@ -264,7 +264,7 @@ class U_I18N_API FormattedRelativeDateTime : public UMemory, public FormattedVal
      * Default constructor; makes an empty FormattedRelativeDateTime.
      * @draft ICU 64
      */
-    FormattedRelativeDateTime() : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {};
+    FormattedRelativeDateTime() : fData(nullptr), fErrorCode(U_INVALID_STATE_ERROR) {}
 
     /**
      * Move constructor: Leaves the source FormattedRelativeDateTime in an undefined state.