1568175 - Simplify "availableLocales" functions by directly using CallArgs::rval

Status: RESOLVED FIXED

(Core :: JavaScript: Internationalization API, task)

Description

[André Bargull [:anba]]

From js::intl_Collator_availableLocales

RootedValue result(cx);
if (!GetAvailableLocales(cx, ucol_countAvailable, ucol_getAvailable, &result)) {
  return false;
}
args.rval().set(result);
return true;

can be simplified to:

return GetAvailableLocales(cx, ucol_countAvailable, ucol_getAvailable, args.rval());

The same pattern appears in all Intl classes.

Comment 1

[André Bargull [:anba]]

Attachment: Bug 1568175: Directly store result of 'available-locales' function in args.rval(). r=khyperia!

Comment 2

[André Bargull [:anba]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e0759132b55b8e5717cf82f21d1c0428a5360448

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9cca1ac4fe97
Directly store result of 'available-locales' function in args.rval(). r=khyperia

Comment 4

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/9cca1ac4fe97