1319464 - Replace flattened strings with linear strings in Intl.cpp

Status: RESOLVED FIXED

(Core :: JavaScript: Internationalization API, defect)

Description

[André Bargull [:anba]]

http://logs.glob.uno/?c=mozilla%23jsapi#c811362

Comment 1

[André Bargull [:anba]]

Attachment: bug1319464.patch

NewUNumberFormat():
We're already explicitly passing uCurrency's length to unum_setTextAttribute, so we don't need to null-terminate uCurrency.

Time zone lookups:
Again no null-termination required because we're always passing the string's length (HashStringIgnoreCaseASCII, EqualCharsIgnoreCaseASCII).

js::intl_patternForSkeleton():
I've changed this method so it matches the normal ICU call pattern. As a side-effect the null-termination is no longer required.

NewUDateFormat():
Also changed to the normal ICU call pattern, so we no longer need to null-terminate the strings.


I've also replaced some "int" with "int32_t" and added assertions for consistency with other calls to ICU functions.

Comment 2

[André Bargull [:anba]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3aeb07223e84e3eba3c61aa769ea67be0da3066a
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7996d18045a2f21b706fd6cb25994e80cf61bd3f

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7251919f6a23
Replace flattened with linear strings in Intl.cpp file. r=Waldo

Comment 4

[Sebastian Hengst [:aryx] (needinfo on intermittent or backout)]

Also backed out (in addition to bug 1319465) for failures in Intl.h in Sm-tc(nu):

https://hg.mozilla.org/integration/mozilla-inbound/rev/9caa83bd1f8353ca9c61f663d5023f61326e18e1

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=aabf76e98464f96bcee92c111286ffe939b98700
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=39721183&repo=mozilla-inbound

[task 2016-11-23T16:02:36.957660Z] In file included from /home/worker/workspace/build/src/js/src/builtin/Intl.cpp:12:0:
[task 2016-11-23T16:02:36.957731Z] /home/worker/workspace/build/src/js/src/builtin/Intl.h:85:43: error: expected ')' before '*' token
[task 2016-11-23T16:02:36.957756Z]              explicit Lookup(JSLinearString* timeZone);
[task 2016-11-23T16:02:36.957788Z]                                            ^
[task 2016-11-23T16:02:38.971359Z] /home/worker/workspace/build/src/js/src/builtin/Intl.cpp:2045:1: error: prototype for 'js::SharedIntlData::TimeZoneHasher::Lookup::Lookup(JSLinearString*)' does not match any in class 'js::SharedIntlData::TimeZoneHasher::Lookup'
[task 2016-11-23T16:02:38.971428Z]  js::SharedIntlData::TimeZoneHasher::Lookup::Lookup(JSLinearString* timeZone)
[task 2016-11-23T16:02:38.971442Z]  ^
[task 2016-11-23T16:02:38.971473Z] In file included from /home/worker/workspace/build/src/js/src/builtin/Intl.cpp:12:0:
[task 2016-11-23T16:02:38.971542Z] /home/worker/workspace/build/src/js/src/builtin/Intl.h:74:16: error: candidates are: constexpr js::SharedIntlData::TimeZoneHasher::Lookup::Lookup(js::SharedIntlData::TimeZoneHasher::Lookup&&)
[task 2016-11-23T16:02:38.971566Z]          struct Lookup
[task 2016-11-23T16:02:38.971584Z]                 ^
[task 2016-11-23T16:02:38.971633Z] /home/worker/workspace/build/src/js/src/builtin/Intl.h:74:16: error:                 constexpr js::SharedIntlData::TimeZoneHasher::Lookup::Lookup(const js::SharedIntlData::TimeZoneHasher::Lookup&)
[task 2016-11-23T16:02:38.971686Z] /home/worker/workspace/build/src/js/src/builtin/Intl.h:74:16: error:                 js::SharedIntlData::TimeZoneHasher::Lookup::Lookup()
[task 2016-11-23T16:02:38.971746Z] /home/worker/workspace/build/src/js/src/builtin/Intl.h:72:12: error: 'struct js::SharedIntlData::TimeZoneHasher' is private
[task 2016-11-23T16:02:38.971775Z]      struct TimeZoneHasher
[task 2016-11-23T16:02:38.971797Z]             ^
...

Comment 5

[André Bargull [:anba]]

Looks like a missing include for vm/String.h.

Comment 6

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/00df72027708
Backed out changeset 7251919f6a23 to fix Sm-Tc(nu). r=backout

Comment 7

[André Bargull [:anba]]

Attachment: bug1319464.patch

Added forward declaration to fix non-unified build bustage. Carrying r+ from Waldo.

Comment 8

[André Bargull [:anba]]

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

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/968fe7fa5d93
Replace flattened with linear strings in Intl.cpp file. r=Waldo

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/968fe7fa5d93