Bug Summary

File:out/../deps/icu-small/source/i18n/number_usageprefs.cpp
Warning:line 147, column 9
Value stored to 'status' is never read

Annotated Source Code

Press '?' to see keyboard shortcuts

clang -cc1 -cc1 -triple x86_64-unknown-linux-gnu -analyze -disable-free -clear-ast-before-backend -disable-llvm-verifier -discard-value-names -main-file-name number_usageprefs.cpp -analyzer-checker=core -analyzer-checker=apiModeling -analyzer-checker=unix -analyzer-checker=deadcode -analyzer-checker=cplusplus -analyzer-checker=security.insecureAPI.UncheckedReturn -analyzer-checker=security.insecureAPI.getpw -analyzer-checker=security.insecureAPI.gets -analyzer-checker=security.insecureAPI.mktemp -analyzer-checker=security.insecureAPI.mkstemp -analyzer-checker=security.insecureAPI.vfork -analyzer-checker=nullability.NullPassedToNonnull -analyzer-checker=nullability.NullReturnedFromNonnull -analyzer-output plist -w -setup-static-analyzer -mrelocation-model pic -pic-level 2 -pic-is-pie -mframe-pointer=all -fmath-errno -ffp-contract=on -fno-rounding-math -mconstructor-aliases -funwind-tables=2 -target-cpu x86-64 -tune-cpu generic -debugger-tuning=gdb -fcoverage-compilation-dir=/home/maurizio/node-v18.6.0/out -resource-dir /usr/local/lib/clang/16.0.0 -D V8_DEPRECATION_WARNINGS -D V8_IMMINENT_DEPRECATION_WARNINGS -D _GLIBCXX_USE_CXX11_ABI=1 -D NODE_OPENSSL_CONF_NAME=nodejs_conf -D NODE_OPENSSL_HAS_QUIC -D __STDC_FORMAT_MACROS -D OPENSSL_NO_PINSHARED -D OPENSSL_THREADS -D U_COMMON_IMPLEMENTATION=1 -D U_I18N_IMPLEMENTATION=1 -D U_IO_IMPLEMENTATION=1 -D U_TOOLUTIL_IMPLEMENTATION=1 -D U_ATTRIBUTE_DEPRECATED= -D _CRT_SECURE_NO_DEPRECATE= -D U_STATIC_IMPLEMENTATION=1 -D UCONFIG_NO_SERVICE=1 -D U_ENABLE_DYLOAD=0 -D U_HAVE_STD_STRING=1 -D UCONFIG_NO_BREAK_ITERATION=0 -I ../deps/icu-small/source/common -I ../deps/icu-small/source/i18n -I ../deps/icu-small/source/tools/toolutil -internal-isystem /usr/lib/gcc/x86_64-redhat-linux/8/../../../../include/c++/8 -internal-isystem /usr/lib/gcc/x86_64-redhat-linux/8/../../../../include/c++/8/x86_64-redhat-linux -internal-isystem /usr/lib/gcc/x86_64-redhat-linux/8/../../../../include/c++/8/backward -internal-isystem /usr/local/lib/clang/16.0.0/include -internal-isystem /usr/local/include -internal-isystem /usr/lib/gcc/x86_64-redhat-linux/8/../../../../x86_64-redhat-linux/include -internal-externc-isystem /include -internal-externc-isystem /usr/include -O3 -Wno-unused-parameter -Wno-deprecated-declarations -Wno-strict-aliasing -std=gnu++17 -fdeprecated-macro -fdebug-compilation-dir=/home/maurizio/node-v18.6.0/out -ferror-limit 19 -fgnuc-version=4.2.1 -vectorize-loops -vectorize-slp -analyzer-output=html -faddrsig -D__GCC_HAVE_DWARF2_CFI_ASM=1 -o /tmp/scan-build-2022-08-22-142216-507842-1 -x c++ ../deps/icu-small/source/i18n/number_usageprefs.cpp
1// © 2020 and later: Unicode, Inc. and others.
2// License & terms of use: http://www.unicode.org/copyright.html
3
4#include "unicode/utypes.h"
5
6#if !UCONFIG_NO_FORMATTING0
7
8#include "number_usageprefs.h"
9#include "cstring.h"
10#include "number_decimalquantity.h"
11#include "number_microprops.h"
12#include "number_roundingutils.h"
13#include "number_skeletons.h"
14#include "unicode/char16ptr.h"
15#include "unicode/currunit.h"
16#include "unicode/fmtable.h"
17#include "unicode/measure.h"
18#include "unicode/numberformatter.h"
19#include "unicode/platform.h"
20#include "unicode/unum.h"
21#include "unicode/urename.h"
22#include "units_data.h"
23
24using namespace icu;
25using namespace icu::number;
26using namespace icu::number::impl;
27using icu::StringSegment;
28using icu::units::ConversionRates;
29
30// Copy constructor
31StringProp::StringProp(const StringProp &other) : StringProp() {
32 this->operator=(other);
33}
34
35// Copy assignment operator
36StringProp &StringProp::operator=(const StringProp &other) {
37 if (this == &other) { return *this; } // self-assignment: no-op
38 fLength = 0;
39 fError = other.fError;
40 if (fValue != nullptr) {
41 uprv_freeuprv_free_71(fValue);
42 fValue = nullptr;
43 }
44 if (other.fValue == nullptr) {
45 return *this;
46 }
47 if (U_FAILURE(other.fError)) {
48 // We don't bother trying to allocating memory if we're in any case busy
49 // copying an errored StringProp.
50 return *this;
51 }
52 fValue = (char *)uprv_mallocuprv_malloc_71(other.fLength + 1);
53 if (fValue == nullptr) {
54 fError = U_MEMORY_ALLOCATION_ERROR;
55 return *this;
56 }
57 fLength = other.fLength;
58 uprv_strncpy(fValue, other.fValue, fLength + 1):: strncpy(fValue, other.fValue, fLength + 1);
59 return *this;
60}
61
62// Move constructor
63StringProp::StringProp(StringProp &&src) U_NOEXCEPTnoexcept : fValue(src.fValue),
64 fLength(src.fLength),
65 fError(src.fError) {
66 // Take ownership away from src if necessary
67 src.fValue = nullptr;
68}
69
70// Move assignment operator
71StringProp &StringProp::operator=(StringProp &&src) U_NOEXCEPTnoexcept {
72 if (this == &src) {
73 return *this;
74 }
75 if (fValue != nullptr) {
76 uprv_freeuprv_free_71(fValue);
77 }
78 fValue = src.fValue;
79 fLength = src.fLength;
80 fError = src.fError;
81 // Take ownership away from src if necessary
82 src.fValue = nullptr;
83 return *this;
84}
85
86StringProp::~StringProp() {
87 if (fValue != nullptr) {
88 uprv_freeuprv_free_71(fValue);
89 fValue = nullptr;
90 }
91}
92
93void StringProp::set(StringPiece value) {
94 if (fValue != nullptr) {
95 uprv_freeuprv_free_71(fValue);
96 fValue = nullptr;
97 }
98 fLength = value.length();
99 fValue = (char *)uprv_mallocuprv_malloc_71(fLength + 1);
100 if (fValue == nullptr) {
101 fLength = 0;
102 fError = U_MEMORY_ALLOCATION_ERROR;
103 return;
104 }
105 uprv_strncpy(fValue, value.data(), fLength):: strncpy(fValue, value.data(), fLength);
106 fValue[fLength] = 0;
107}
108
109// Populates micros.mixedMeasures and modifies quantity, based on the values in
110// measures.
111void mixedMeasuresToMicros(const MaybeStackVector<Measure> &measures, DecimalQuantity *quantity,
112 MicroProps *micros, UErrorCode status) {
113 micros->mixedMeasuresCount = measures.length();
114
115 if (micros->mixedMeasures.getCapacity() < micros->mixedMeasuresCount) {
116 if (micros->mixedMeasures.resize(micros->mixedMeasuresCount) == nullptr) {
117 status = U_MEMORY_ALLOCATION_ERROR;
118 return;
119 }
120 }
121
122 for (int32_t i = 0; i < micros->mixedMeasuresCount; i++) {
123 switch (measures[i]->getNumber().getType()) {
124 case Formattable::kInt64:
125 micros->mixedMeasures[i] = measures[i]->getNumber().getInt64();
126 break;
127
128 case Formattable::kDouble:
129 U_ASSERT(micros->indexOfQuantity < 0)(void)0;
130 quantity->setToDouble(measures[i]->getNumber().getDouble());
131 micros->indexOfQuantity = i;
132 break;
133
134 default:
135 U_ASSERT(0 == "Found a Measure Number which is neither a double nor an int")(void)0;
136 UPRV_UNREACHABLE_EXITabort();
137 break;
138 }
139
140 if (U_FAILURE(status)) {
141 return;
142 }
143 }
144
145 if (micros->indexOfQuantity < 0) {
146 // There is no quantity.
147 status = U_INTERNAL_PROGRAM_ERROR;
Value stored to 'status' is never read
148 }
149}
150
151UsagePrefsHandler::UsagePrefsHandler(const Locale &locale,
152 const MeasureUnit &inputUnit,
153 const StringPiece usage,
154 const MicroPropsGenerator *parent,
155 UErrorCode &status)
156 : fUnitsRouter(inputUnit, StringPiece(locale.getCountry()), usage, status),
157 fParent(parent) {
158}
159
160void UsagePrefsHandler::processQuantity(DecimalQuantity &quantity, MicroProps &micros,
161 UErrorCode &status) const {
162 fParent->processQuantity(quantity, micros, status);
163 if (U_FAILURE(status)) {
164 return;
165 }
166
167 quantity.roundToInfinity(); // Enables toDouble
168 const units::RouteResult routed = fUnitsRouter.route(quantity.toDouble(), &micros.rounder, status);
169 if (U_FAILURE(status)) {
170 return;
171 }
172 const MaybeStackVector<Measure>& routedMeasures = routed.measures;
173 micros.outputUnit = routed.outputUnit.copy(status).build(status);
174 if (U_FAILURE(status)) {
175 return;
176 }
177
178 mixedMeasuresToMicros(routedMeasures, &quantity, &micros, status);
179}
180
181UnitConversionHandler::UnitConversionHandler(const MeasureUnit &targetUnit,
182 const MicroPropsGenerator *parent, UErrorCode &status)
183 : fOutputUnit(targetUnit), fParent(parent) {
184 MeasureUnitImpl tempInput, tempOutput;
185
186 ConversionRates conversionRates(status);
187 if (U_FAILURE(status)) {
188 return;
189 }
190
191 const MeasureUnitImpl &targetUnitImpl =
192 MeasureUnitImpl::forMeasureUnit(targetUnit, tempOutput, status);
193 fUnitConverter.adoptInsteadAndCheckErrorCode(
194 new ComplexUnitsConverter(targetUnitImpl, conversionRates, status), status);
195}
196
197void UnitConversionHandler::processQuantity(DecimalQuantity &quantity, MicroProps &micros,
198 UErrorCode &status) const {
199 fParent->processQuantity(quantity, micros, status);
200 if (U_FAILURE(status)) {
201 return;
202 }
203 quantity.roundToInfinity(); // Enables toDouble
204 MaybeStackVector<Measure> measures =
205 fUnitConverter->convert(quantity.toDouble(), &micros.rounder, status);
206 micros.outputUnit = fOutputUnit;
207 if (U_FAILURE(status)) {
208 return;
209 }
210
211 mixedMeasuresToMicros(measures, &quantity, &micros, status);
212}
213
214#endif /* #if !UCONFIG_NO_FORMATTING */