921582 - '... may be used uninitialized' warning in intl_CompareStrings

Status: RESOLVED FIXED

(Core :: JavaScript: Internationalization API, defect)

Description

[Jim Blandy :jimb]

Attachment: Avoid a 'may be used uninitialized' warning.

Specifically:

In file included from ./../../dist/include/js/CallArgs.h:39:0,
                 from ./../../dist/include/js/CallNonGenericMethod.h:12,
                 from /home/jimb/moz/dbg/js/src/NamespaceImports.h:15,
                 from /home/jimb/moz/dbg/js/src/builtin/Intl.h:10,
                 from /home/jimb/moz/dbg/js/src/builtin/Intl.cpp:12:
./../../dist/include/js/Value.h: In function ‘bool js::intl_CompareStrings(JSContext*, unsigned int, JS::Value*)’:
./../../dist/include/js/Value.h:653:58: warning: ‘res’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     JS_RETURN_LAYOUT_FROM_BITS(((uint64_t)(uint32_t)i32) | JSVAL_SHIFTED_TAG_INT32);
                                                          ^
/home/jimb/moz/dbg/js/src/builtin/Intl.cpp:968:13: note: ‘res’ was declared here
     int32_t res;
             ^

Comment 1

[Jim Blandy :jimb]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b85ed93dee3b

Comment 2

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/b85ed93dee3b

Comment 3

[Gordon P. Hemsley [:GPHemsley]]

Mass-moving existing Intl-related bugs to the new Core :: JavaScript: Internationalization API component.

If you think this bug has been moved in error, feel free to move it back to Core :: JavaScript Engine.

[Mass change filter: core-js-intl-api-move]