https://codereview.chromium.org/40133004/diff/1/src/i18n.js
File src/i18n.js (right):
https://codereview.chromium.org/40133004/diff/1/src/i18n.js#newcode2002
src/i18n.js:2002: * Adds a new method.
Remove * Adds a new method comment. That belongs to CL description.
https://codereview.chromium.org/40133004/diff/1/src/i18n.js#newcode2016
src/i18n.js:2016: var normForm = NORMALIZATION_FORMS.indexOf(form);
normalizationForm, don't abbreviate (except for loop indices).
https://codereview.chromium.org/40133004/diff/1/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/40133004/diff/1/src/runtime.cc#newcode14069
src/runtime.cc:14069: static const UNormalizationMode normForms[] =
normalizationForms[]
https://codereview.chromium.org/40133004/diff/1/src/runtime.cc#newcode14080
src/runtime.cc:14080: // TODO(mnita) check Normalizer2 (not available in
ICU 46)
// TODO(mnita): switch to Normalizer2 when we switch to ICU 51 or above.
https://codereview.chromium.org/40133004/diff/1/src/runtime.cc#newcode14084
src/runtime.cc:14084: if (U_FAILURE(status)) return
isolate->heap()->undefined_value();
if () {
statement;
}
not
if () statement;
https://codereview.chromium.org/40133004/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
