Bug 1356843 - Fix -Wcomma warning in intl/uconv/util/nsUCSupport.cpp. r?smontagu draft
authorChris Peterson <cpeterson@mozilla.com>
Sat, 08 Apr 2017 01:05:19 -0700
changeset 563967 ac8f478a584f71bf18fe4df092c81bf9f3aeee60
parent 563966 206a327410b1ed367a9deae233c10578e7d8baf2
child 563968 5b5028c76ccfca2ab2cb5eea66c1c6719ef1a429
push id54487
push usercpeterson@mozilla.com
push dateTue, 18 Apr 2017 06:09:29 +0000
reviewerssmontagu
bugs1356843
milestone55.0a1
Bug 1356843 - Fix -Wcomma warning in intl/uconv/util/nsUCSupport.cpp. r?smontagu clang's -Wcomma warning warns about suspicious use of the comma operator such as between two statements. intl/uconv/util/nsUCSupport.cpp:552:34 [-Wcomma] possible misuse of comma operator here MozReview-Commit-ID: Emk05qDJHbm
intl/uconv/util/nsUCSupport.cpp
--- a/intl/uconv/util/nsUCSupport.cpp
+++ b/intl/uconv/util/nsUCSupport.cpp
@@ -544,17 +544,17 @@ nsEncoderSupport::GetMaxLength(const cha
 
 nsTableEncoderSupport::nsTableEncoderSupport(uScanClassID aScanClass,
                                              uShiftOutTable* aShiftOutTable,
                                              uMappingTable* aMappingTable,
                                              uint32_t aMaxLengthFactor)
 : nsEncoderSupport(aMaxLengthFactor)
 {
   mScanClass = aScanClass;
-  mShiftOutTable = aShiftOutTable,
+  mShiftOutTable = aShiftOutTable;
   mMappingTable = aMappingTable;
 }
 
 nsTableEncoderSupport::nsTableEncoderSupport(uScanClassID aScanClass,
                                              uMappingTable* aMappingTable,
                                              uint32_t aMaxLengthFactor)
 : nsEncoderSupport(aMaxLengthFactor)
 {