diff options
Diffstat (limited to 'src/classlibnative/bcltype/number.cpp')
-rw-r--r-- | src/classlibnative/bcltype/number.cpp | 459 |
1 files changed, 43 insertions, 416 deletions
diff --git a/src/classlibnative/bcltype/number.cpp b/src/classlibnative/bcltype/number.cpp index 6ca24d8672..3f2d63c70e 100644 --- a/src/classlibnative/bcltype/number.cpp +++ b/src/classlibnative/bcltype/number.cpp @@ -142,12 +142,6 @@ void DoubleToNumber(double value, int precision, NUMBER* number) WRAPPER_NO_CONTRACT _ASSERTE(number != NULL); -#ifndef FEATURE_BCL_FORMATTING - number->palNumber=PAL_DoubleToNumber(value); - IfNullThrow(number->palNumber); - number->palNumberType=PALNUMBERTYPE_DOUBLE; -#endif - number->precision = precision; if (((FPDOUBLE*)&value)->exp == 0x7FF) { number->scale = (((FPDOUBLE*)&value)->mantLo || ((FPDOUBLE*)&value)->mantHi) ? SCALE_NAN: SCALE_INF; @@ -736,13 +730,6 @@ void Int32ToNumber(int value, NUMBER* number) WRAPPER_NO_CONTRACT _ASSERTE(number != NULL); -#ifndef FEATURE_BCL_FORMATTING - number->palNumber=PAL_IntToNumber(value); - IfNullThrow(number->palNumber); - number->palNumberType=PALNUMBERTYPE_INT; -#endif - - wchar buffer[INT32_PRECISION+1]; number->precision = INT32_PRECISION; if (value >= 0) { @@ -769,12 +756,6 @@ void UInt32ToNumber(unsigned int value, NUMBER* number) WRAPPER_NO_CONTRACT _ASSERTE(number != NULL); -#ifndef FEATURE_BCL_FORMATTING - number->palNumber=PAL_UIntToNumber(value); - IfNullThrow(number->palNumber); - number->palNumberType=PALNUMBERTYPE_UINT; -#endif - wchar buffer[UINT32_PRECISION+1]; number->precision = UINT32_PRECISION; number->sign = 0; @@ -880,12 +861,6 @@ void Int64ToNumber(__int64 value, NUMBER* number) { WRAPPER_NO_CONTRACT -#ifndef FEATURE_BCL_FORMATTING - number->palNumber=PAL_Int64ToNumber(value); - IfNullThrow(number->palNumber); - number->palNumberType=PALNUMBERTYPE_INT64; -#endif - wchar buffer[INT64_PRECISION+1]; number->precision = INT64_PRECISION; if (value >= 0) { @@ -934,29 +909,6 @@ void UInt64ToNumber(unsigned __int64 value, NUMBER* number) } -#ifndef FEATURE_BCL_FORMATTING -void NumberToUInt64(NUMBER * number, unsigned __int64* value) -{ - _ASSERTE(NULL != number); - _ASSERTE(NULL != value); - - if (NULL != number && NULL != value) { - (*value) = 0; - int i = 0; - while (i < NUMBER_MAXDIGITS && i < number->precision && number->digits[i] != NULL) { - _ASSERTE((number->digits[i] - '0') >= 0 && (number->digits[i] - '0') <= 9); - *value = (10 * (*value)) + (number->digits[i] - '0'); - i++; - } - while (i < number->scale) { - *value = (10 * (*value)); - i++; - } - } -} -#endif - - void RoundNumber(NUMBER* number, int pos) { LIMITED_METHOD_CONTRACT @@ -985,50 +937,6 @@ void RoundNumber(NUMBER* number, int pos) } digits[i] = 0; -#ifndef FEATURE_BCL_FORMATTING -// -// The PAL stores PALNUMBER as the actual numeric type where as NUMBER is in string form; -// Convert NUMBER back into its original type and pass it to the PAL for later use -// - if (0 != number->palNumber) { - if (PALNUMBERTYPE_DOUBLE == number->palNumberType) { - // no need to round NaN or infinity double values - if (SCALE_NAN != ((unsigned int)number->scale) && SCALE_INF != ((unsigned int)number->scale)) { - double value = 0.0; - NumberToDouble(number, &value); - // make sure the rounding didn't accidently cause the good value - // to be turned into NaN or infinity - if (((FPDOUBLE*)&value)->exp != 0x7FF) { - number->palNumber=PAL_DoubleToNumber(value); - } - } - } - else { - unsigned __int64 value = 0; - NumberToUInt64(number, &value); - switch(number->palNumberType) { - case PALNUMBERTYPE_INT: - _ASSERTE((value >> 32) == 0); - number->palNumber=PAL_IntToNumber(((unsigned int)value) * (number->sign ? -1 : 1)); - break; - case PALNUMBERTYPE_UINT: - _ASSERTE((value >> 32) == 0); - number->palNumber=PAL_UIntToNumber((unsigned int)value); - break; - case PALNUMBERTYPE_INT64: - _ASSERTE((value >> 63) == 0); - number->palNumber=PAL_Int64ToNumber(((__int64)value) * (number->sign ? -1 : 1)); - break; - case PALNUMBERTYPE_UINT64: - number->palNumber=PAL_UInt64ToNumber(value); - break; - default: - CONSISTENCY_CHECK_MSGF(0, ("This palNumberType is not understood '(%d)''\n", number->palNumberType)); - break; - } - } - } -#endif } #if defined(_MSC_VER) && defined(_TARGET_X86_) @@ -1112,29 +1020,6 @@ wchar* FormatGeneral(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NUM } } -#ifndef FEATURE_BCL_FORMATTING - if (number->palNumber) - { - WCHAR sExp[2]={expChar}; - LPCWSTR strNumberDecimal(sNumberDecimal!=NULL?sNumberDecimal->GetBuffer():NULL); - LPCWSTR strPositive(sPositive!=NULL?sPositive->GetBuffer():NULL); - LPCWSTR strNegative(sNegative!=NULL?sNegative->GetBuffer():NULL); - LPCWSTR strZero(sZero!=NULL?sZero->GetBuffer():NULL); - - int nChars; - - // nMaxDigits for Scientific are 1 more than needed - if (scientific) - nChars=PAL_FormatScientific(NULL, buffer, cchBuffer, number->palNumber,-1,nMaxDigits-1,sExp,strNumberDecimal,strPositive,strNegative,strZero); - else - nChars=PAL_FormatDecimal(NULL, buffer, cchBuffer, number->palNumber,-1,nMaxDigits,-1, -1,-1,strNumberDecimal,NULL,strNegative,strZero); - - if(nChars<0) - return NULL; - return buffer+nChars; - } -#endif - wchar* dig = GetDigitsBuffer(number); _ASSERT(dig != NULL); if (digPos > 0) { @@ -1162,26 +1047,9 @@ wchar* FormatGeneral(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NUM wchar* FormatScientific(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NUMBER* number, int nMinDigits, int nMaxDigits, wchar expChar, STRINGREF sNumberDecimal, STRINGREF sPositive, STRINGREF sNegative, STRINGREF sZero) { - WRAPPER_NO_CONTRACT - _ASSERTE(number != NULL); - _ASSERTE(buffer != NULL); -#ifndef FEATURE_BCL_FORMATTING - if (number->palNumber) - { - WCHAR sExp[2]={expChar}; - LPCWSTR strNumberDecimal(sNumberDecimal!=NULL?sNumberDecimal->GetBuffer():NULL); - LPCWSTR strPositive(sPositive!=NULL?sPositive->GetBuffer():NULL); - LPCWSTR strNegative(sNegative!=NULL?sNegative->GetBuffer():NULL); - LPCWSTR strZero(sZero!=NULL?sZero->GetBuffer():NULL); - - // nMaxDigits passed into FormatScientific are 1 more than requested - int nChars=PAL_FormatScientific(NULL, buffer, cchBuffer, number->palNumber,nMinDigits, nMaxDigits-1,sExp,strNumberDecimal,strPositive,strNegative,strZero); - if(nChars<0) - ThrowLastError(); - return buffer+nChars; - } -#endif - + WRAPPER_NO_CONTRACT + _ASSERTE(number != NULL); + _ASSERTE(buffer != NULL); wchar* dig = GetDigitsBuffer(number); _ASSERTE(dig != NULL); @@ -1207,21 +1075,6 @@ wchar* FormatFixed(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NUMBE PRECONDITION(CheckPointer(number)); } CONTRACTL_END; -#ifndef FEATURE_BCL_FORMATTING - if (number->palNumber) - { - LPCWSTR strDecimal(sDecimal!=NULL?sDecimal->GetBuffer():NULL); - LPCWSTR strNegative(sNegative!=NULL?sNegative->GetBuffer():NULL); - LPCWSTR strZero(sZero!=NULL?sZero->GetBuffer():NULL); - - int nChars=PAL_FormatDecimal(NULL, buffer, cchBuffer, number->palNumber,nMinDigits,nMaxDigits,-1, -1,-1,strDecimal,W(""),strNegative,strZero); - if(nChars<0) - return NULL; - return buffer+nChars; - } -#endif - - int digPos = number->scale; wchar* dig = GetDigitsBuffer(number); const I4* groupDigits = NULL; @@ -1328,33 +1181,6 @@ wchar* FormatNumber(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NUMB PRECONDITION(CheckPointer(buffer)); PRECONDITION(CheckPointer(number)); } CONTRACTL_END; -#ifndef FEATURE_BCL_FORMATTING - if (number->palNumber) - { - LPCWSTR strDecimal(sNumberDecimal!=NULL?sNumberDecimal->GetBuffer():NULL); - LPCWSTR strGroup(sNumberGroup!=NULL?sNumberGroup->GetBuffer():NULL); - LPCWSTR strNegative(sNegative!=NULL?sNegative->GetBuffer():NULL); - LPCWSTR strZero(sZero!=NULL?sZero->GetBuffer():NULL); - int iPrimaryGroup=0; - int iSecondaryGroup=0; - if(cNumberGroup!=NULL) - { - int nGroups = cNumberGroup->GetNumComponents(); - I4* pGroups=(I4*)cNumberGroup->GetDataPtr(); - - if(nGroups>0) - iPrimaryGroup=pGroups[0]; - if(nGroups>1) - iSecondaryGroup=pGroups[1]; - } - - int nChars=PAL_FormatDecimal(NULL, buffer, cchBuffer, number->palNumber,nMinDigits,nMaxDigits, cNegativeNumberFormat, - iPrimaryGroup,iSecondaryGroup,strDecimal,strGroup,strNegative,strZero); - if(nChars<0) - return NULL; - return buffer+nChars; - } -#endif char ch; const char* fmt; @@ -1391,36 +1217,6 @@ wchar* FormatCurrency(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NU PRECONDITION(CheckPointer(number)); } CONTRACTL_END; -#ifndef FEATURE_BCL_FORMATTING - if (number->palNumber) - { - LPCWSTR strCurrencyDecimal(sCurrencyDecimal!=NULL?sCurrencyDecimal->GetBuffer():NULL); - LPCWSTR strCurrencyGroup(sCurrencyGroup!=NULL?sCurrencyGroup->GetBuffer():NULL); - LPCWSTR strNegative(sNegative!=NULL?sNegative->GetBuffer():NULL); - LPCWSTR strCurrency(sCurrency!=NULL?sCurrency->GetBuffer():NULL); - LPCWSTR strZero(sZero!=NULL?sZero->GetBuffer():NULL); - int iPrimaryGroup=0; - int iSecondaryGroup=0; - - if(cCurrencyGroup!=NULL) - { - int nGroups = cCurrencyGroup->GetNumComponents(); - I4* pGroups=(I4*)cCurrencyGroup->GetDataPtr(); - - if(nGroups>0) - iPrimaryGroup=pGroups[0]; - if(nGroups>1) - iSecondaryGroup=pGroups[1]; - } - - int nChars=PAL_FormatCurrency(NULL, buffer, cchBuffer, number->palNumber,nMinDigits,nMaxDigits,cNegCurrencyFormat, cPosCurrencyFormat, - iPrimaryGroup, iSecondaryGroup, strCurrencyDecimal,strCurrencyGroup,strNegative, strCurrency,strZero); - if (nChars<0) - return NULL; - - return buffer+nChars; - } -#endif char ch; const char* fmt; fmt = number->sign? @@ -1458,39 +1254,6 @@ wchar* FormatPercent(__in_ecount(cchBuffer) wchar* buffer, SIZE_T cchBuffer, NUM PRECONDITION(CheckPointer(number)); } CONTRACTL_END; - -#ifndef FEATURE_BCL_FORMATTING - if (number->palNumber) - { - LPCWSTR strPercentDecimal(sPercentDecimal!=NULL?sPercentDecimal->GetBuffer():NULL); - LPCWSTR strPercentGroup(sPercentGroup!=NULL?sPercentGroup->GetBuffer():NULL); - LPCWSTR strNegative(sNegative!=NULL?sNegative->GetBuffer():NULL); - LPCWSTR strPercent(sPercent!=NULL?sPercent->GetBuffer():NULL); - LPCWSTR strZero(sZero!=NULL?sZero->GetBuffer():NULL); - - int iPrimaryGroup=0; - int iSecondaryGroup=0; - - if(cPercentGroup!=NULL) - { - int nGroups = cPercentGroup->GetNumComponents(); - I4* pGroups=(I4*)cPercentGroup->GetDataPtr(); - - if(nGroups>0) - iPrimaryGroup=pGroups[0]; - if(nGroups>1) - iSecondaryGroup=pGroups[1]; - } - - int nChars=PAL_FormatPercent(NULL, buffer, cchBuffer, number->palNumber,nMinDigits,nMaxDigits,cNegativePercentFormat, cPositivePercentFormat, - iPrimaryGroup, iSecondaryGroup,strPercentDecimal,strPercentGroup,strNegative, strPercent,strZero); - if(nChars<0) - return NULL; - - return buffer+nChars; - } -#endif - char ch; const char* fmt; fmt = number->sign? @@ -1612,12 +1375,6 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF STRINGREF sCurrencyGroup = numfmt->sCurrencyGroup; STRINGREF sNegative = numfmt->sNegative; STRINGREF sCurrency = numfmt->sCurrency; -#ifndef FEATURE_BCL_FORMATTING - if (numfmt->bIsInvariant || 0 == number->palNumber) - { - // So that FormatCurrency uses BCL format - number->palNumber = 0; -#endif // Prefix: bogus warning 22011: newBufferLen+=digCount may be smaller than MIN_BUFFER_SIZE PREFIX_ASSUME(digCount >=0 && digCount <= INT32_MAX); newBufferLen += digCount; @@ -1637,22 +1394,6 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF RoundNumber(number, number->scale + nMaxDigits); // Don't change this line to use digPos since digCount could have its sign changed. dst = FormatCurrency(dst, static_cast<SIZE_T>(newBufferLen/sizeof(WCHAR)), number, nMinDigits,nMaxDigits, cNegCurrencyFormat, cPosCurrencyFormat, cCurrencyGroup, sCurrencyDecimal, sCurrencyGroup, sNegative, sCurrency,sZero); -#ifndef FEATURE_BCL_FORMATTING - } - else - { - for ( SIZE_T nChars=128;;nChars*=2) - { - dst = buffer = (WCHAR*)buf.AllocThrows(nChars * sizeof(WCHAR)); - dst = FormatCurrency(dst, nChars, number, nMinDigits, nMaxDigits, cNegCurrencyFormat, cPosCurrencyFormat, cCurrencyGroup, sCurrencyDecimal, sCurrencyGroup, sNegative, sCurrency,sZero); - if (dst) - break; - if (GetLastError()!=ERROR_INSUFFICIENT_BUFFER) - ThrowLastError(); - } - } -#endif - break; } case 'F': @@ -1672,49 +1413,25 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF // It is critical to format with the same values that we use to calculate buffer size. STRINGREF sNumberDecimal = numfmt->sNumberDecimal; STRINGREF sNegative = numfmt->sNegative; -#ifndef FEATURE_BCL_FORMATTING - if (numfmt->bIsInvariant || 0 == number->palNumber) - { - // So that FormatFixed uses BCL format - number->palNumber = 0; -#endif - - newBufferLen += digCount; - newBufferLen += sNegative->GetStringLength(); // For number and exponent - newBufferLen += sNumberDecimal->GetStringLength(); - - _ASSERTE(newBufferLen >= MIN_BUFFER_SIZE); - if (newBufferLen > INT32_MAX) { - COMPlusThrowOM(); - } - newBufferLen = newBufferLen * sizeof(WCHAR); - dst = buffer = (WCHAR*)buf.AllocThrows(static_cast<SIZE_T>(newBufferLen)); - - RoundNumber(number, number->scale + nMaxDigits); + + newBufferLen += digCount; + newBufferLen += sNegative->GetStringLength(); // For number and exponent + newBufferLen += sNumberDecimal->GetStringLength(); + + _ASSERTE(newBufferLen >= MIN_BUFFER_SIZE); + if (newBufferLen > INT32_MAX) { + COMPlusThrowOM(); + } + newBufferLen = newBufferLen * sizeof(WCHAR); + dst = buffer = (WCHAR*)buf.AllocThrows(static_cast<SIZE_T>(newBufferLen)); + + RoundNumber(number, number->scale + nMaxDigits); if (number->sign) { AddStringRef(&dst, sNegative); } dst = FormatFixed(dst, static_cast<SIZE_T>(newBufferLen/sizeof(WCHAR)-(dst-buffer)), number, nMinDigits,nMaxDigits, NULL, sNumberDecimal, NULL, sNegative, sZero); -#ifndef FEATURE_BCL_FORMATTING - } - else - { - for( SIZE_T nChars=128;;nChars*=2) - { - dst = buffer = (WCHAR*)buf.AllocThrows(nChars * sizeof(WCHAR)); - dst = FormatFixed(dst, nChars, number, nMinDigits,nMaxDigits, - NULL, - sNumberDecimal, NULL,sNegative,sZero); - - if (dst) - break; - if (GetLastError()!=ERROR_INSUFFICIENT_BUFFER) - ThrowLastError(); - } - } -#endif break; } @@ -1737,44 +1454,21 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF STRINGREF sNumberDecimal = numfmt->sNumberDecimal; STRINGREF sNumberGroup = numfmt->sNumberGroup; int cNegativeNumberFormat = numfmt->cNegativeNumberFormat; -#ifndef FEATURE_BCL_FORMATTING - if (numfmt->bIsInvariant || 0 == number->palNumber) - { - // So that FormatNumber uses BCL format - number->palNumber = 0; -#endif - - newBufferLen += digCount; - newBufferLen += sNegative->GetStringLength(); // For number and exponent - if (!ClrSafeInt<UINT64>::addition((UINT64)sNumberGroup->GetStringLength() * digCount, newBufferLen, newBufferLen)) - COMPlusThrowOM(); - newBufferLen += sNumberDecimal->GetStringLength(); - - _ASSERTE(newBufferLen >= MIN_BUFFER_SIZE); - if (newBufferLen > INT32_MAX) { - COMPlusThrowOM(); - } - newBufferLen = newBufferLen * sizeof(WCHAR); - dst = buffer = (WCHAR*)buf.AllocThrows(static_cast<SIZE_T>(newBufferLen)); - - RoundNumber(number, number->scale + nMaxDigits); - dst = FormatNumber(dst, static_cast<SIZE_T>(newBufferLen/sizeof(WCHAR)),number, nMinDigits, nMaxDigits, cNegativeNumberFormat, cNumberGroup, sNumberDecimal, sNumberGroup, sNegative, sZero); -#ifndef FEATURE_BCL_FORMATTING - } - else - { - for( SIZE_T nChars=128;;nChars*=2) - { - dst = buffer = (WCHAR*)buf.AllocThrows(nChars * sizeof(WCHAR)); - dst = FormatNumber(dst, nChars, number, nMinDigits,nMaxDigits, cNegativeNumberFormat, cNumberGroup, sNumberDecimal, sNumberGroup, sNegative,sZero); - - if (dst) - break; - if (GetLastError()!=ERROR_INSUFFICIENT_BUFFER) - ThrowLastError(); - } + newBufferLen += digCount; + newBufferLen += sNegative->GetStringLength(); // For number and exponent + if (!ClrSafeInt<UINT64>::addition((UINT64)sNumberGroup->GetStringLength() * digCount, newBufferLen, newBufferLen)) + COMPlusThrowOM(); + newBufferLen += sNumberDecimal->GetStringLength(); + + _ASSERTE(newBufferLen >= MIN_BUFFER_SIZE); + if (newBufferLen > INT32_MAX) { + COMPlusThrowOM(); } -#endif + newBufferLen = newBufferLen * sizeof(WCHAR); + dst = buffer = (WCHAR*)buf.AllocThrows(static_cast<SIZE_T>(newBufferLen)); + + RoundNumber(number, number->scale + nMaxDigits); + dst = FormatNumber(dst, static_cast<SIZE_T>(newBufferLen/sizeof(WCHAR)),number, nMinDigits, nMaxDigits, cNegativeNumberFormat, cNumberGroup, sNumberDecimal, sNumberGroup, sNegative, sZero); break; } @@ -1792,45 +1486,22 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF nMinDigits=nMaxDigits; nMaxDigits++; -#ifndef FEATURE_BCL_FORMATTING - if (numfmt->bIsInvariant || 0 == number->palNumber) - { - // So that FormatScientific uses BCL format - number->palNumber = 0; -#endif - - newBufferLen += nMaxDigits; - newBufferLen += (((INT64)sNegative->GetStringLength() + sPositive->GetStringLength()) *2); // For number and exponent - newBufferLen += sNumberDecimal->GetStringLength(); - - _ASSERTE(newBufferLen >= MIN_BUFFER_SIZE); - if (newBufferLen > INT32_MAX) { - COMPlusThrowOM(); - } - newBufferLen = newBufferLen * sizeof(WCHAR); - dst = buffer = (WCHAR*)buf.AllocThrows(static_cast<SIZE_T>(newBufferLen)); + newBufferLen += nMaxDigits; + newBufferLen += (((INT64)sNegative->GetStringLength() + sPositive->GetStringLength()) *2); // For number and exponent + newBufferLen += sNumberDecimal->GetStringLength(); - RoundNumber(number, nMaxDigits); - if (number->sign) { - AddStringRef(&dst, sNegative); - } - dst = FormatScientific(dst, static_cast<SIZE_T>(newBufferLen * sizeof(WCHAR)-(dst-buffer)),number, nMinDigits,nMaxDigits, format, sNumberDecimal, sPositive, sNegative,sZero); -#ifndef FEATURE_BCL_FORMATTING + _ASSERTE(newBufferLen >= MIN_BUFFER_SIZE); + if (newBufferLen > INT32_MAX) { + COMPlusThrowOM(); } - else - { - for( SIZE_T nChars=128;;nChars*=2) - { - dst = buffer = (WCHAR*)buf.AllocThrows(nChars * sizeof(WCHAR)); - dst = FormatScientific(dst, nChars, number, nMinDigits, nMaxDigits, format, sNumberDecimal, sPositive, sNegative,sZero); + newBufferLen = newBufferLen * sizeof(WCHAR); + dst = buffer = (WCHAR*)buf.AllocThrows(static_cast<SIZE_T>(newBufferLen)); - if (dst) - break; - if (GetLastError()!=ERROR_INSUFFICIENT_BUFFER) - ThrowLastError(); - } + RoundNumber(number, nMaxDigits); + if (number->sign) { + AddStringRef(&dst, sNegative); } -#endif + dst = FormatScientific(dst, static_cast<SIZE_T>(newBufferLen * sizeof(WCHAR)-(dst-buffer)),number, nMinDigits,nMaxDigits, format, sNumberDecimal, sPositive, sNegative,sZero); break; } @@ -1878,30 +1549,7 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF } -#ifndef FEATURE_BCL_FORMATTING - if (numfmt->bIsInvariant || 0 == number->palNumber) - { - // So that FormatScientific uses BCL format - number->palNumber = 0; -#endif - - dst = FormatGeneral(dst, static_cast<SIZE_T>(newBufferLen/sizeof(WCHAR)), number, nMinDigits,nMaxDigits, format - ('G' - 'E'), sNumberDecimal, sPositive, sNegative, sZero, !enableRounding); -#ifndef FEATURE_BCL_FORMATTING - } - else - { - for( SIZE_T nChars=128;;nChars*=2) - { - dst = buffer = (WCHAR*)buf.AllocThrows(nChars * sizeof(WCHAR)); - dst = FormatGeneral(dst, nChars, number, nMinDigits,nMaxDigits, format - ('G' - 'E'), sNumberDecimal, sPositive, sNegative, sZero, !enableRounding); - - if (dst) - break; - if (GetLastError()!=ERROR_INSUFFICIENT_BUFFER) - ThrowLastError(); - } - } -#endif + dst = FormatGeneral(dst, static_cast<SIZE_T>(newBufferLen/sizeof(WCHAR)), number, nMinDigits,nMaxDigits, format - ('G' - 'E'), sNumberDecimal, sPositive, sNegative, sZero, !enableRounding); } break; @@ -1930,10 +1578,6 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF STRINGREF sNegative = numfmt->sNegative; STRINGREF sPercent = numfmt->sPercent; -#ifndef FEATURE_BCL_FORMATTING - // So that FormatPercent uses BCL format - number->palNumber = 0; -#endif newBufferLen += digCount; newBufferLen += sNegative->GetStringLength(); // For number and exponent if (!ClrSafeInt<UINT64>::addition((UINT64)sPercentGroup->GetStringLength() * digCount, newBufferLen, newBufferLen)) @@ -1957,7 +1601,6 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF COMPlusThrow(kFormatException, W("Argument_BadFormatSpecifier")); } // check for overflow of the preallocated buffer -#ifdef FEATURE_BCL_FORMATTING // when not defined the buffer could be resized, so skip the check // Review signed/unsigned mismatch in '<=' comparison. #pragma warning(push) #pragma warning(disable:4018) @@ -1965,7 +1608,6 @@ STRINGREF NumberToString(NUMBER* number, wchar format, int nMaxDigits, NUMFMTREF #pragma warning(pop) DoJITFailFast(); } -#endif return StringObject::NewString(buffer, (int) (dst - buffer)); } @@ -2047,12 +1689,6 @@ STRINGREF NumberToStringFormat(NUMBER* number, STRINGREF str, NUMFMTREF numfmt) section = FindSection(format, (GetDigitsBuffer(number))[0] == 0 ? 2 : number->sign ? 1 : 0); -#ifndef FEATURE_BCL_FORMATTING - // custom formatting is all done in the VM without the PAL. Blanking - // the palNumber field avoids unnecessary RoundNumber calculations - number->palNumber = 0; -#endif - ParseSection: digitCount = 0; decimalPos = -1; @@ -2607,11 +2243,6 @@ FCIMPL3_VII(Object*, COMNumber::FormatSingle, float value, StringObject* formatU //and display that. DoubleToNumber(argsValue, FLOAT_PRECISION, &number); -#ifndef FEATURE_BCL_FORMATTING - // Make sure that BCL formatting is used for Single to avoid lossy conversion to Double - number.palNumber = 0; -#endif - if (number.scale == (int) SCALE_NAN) { gc.refRetVal = gc.numfmt->sNaN; goto lExit; @@ -2654,10 +2285,6 @@ FCIMPL3_VII(Object*, COMNumber::FormatSingle, float value, StringObject* formatU } DoubleToNumber(value, precision, &number); -#ifndef FEATURE_BCL_FORMATTING - // Make sure that BCL formatting is used for Single to avoid lossy conversion to Double - number.palNumber = 0; -#endif if (number.scale == (int) SCALE_NAN) { gc.refRetVal = gc.numfmt->sNaN; |