Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(995)

Side by Side Diff: third_party/icu/README.chromium

Issue 2025010: Add a separate icu data file for ChromeOS (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | third_party/icu/cros/locales/am.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 Name: ICU 1 Name: ICU
2 URL: http://site.icu-project.org/ 2 URL: http://site.icu-project.org/
3 3
4 This directory contains the source code of ICU 4.2.1 for C/C++ 4 This directory contains the source code of ICU 4.2.1 for C/C++
5 5
6 1. It was obtained with the following: 6 1. It was obtained with the following:
7 7
8 $ svn export --native-eol LF http://source.icu-project.org/repos/icu/icu/tag s/release-4-2-1 icu42 8 $ svn export --native-eol LF http://source.icu-project.org/repos/icu/icu/tag s/release-4-2-1 icu42
9 9
10 2. The following directories were removed because they're not used by Chromium 10 2. The following directories were removed because they're not used by Chromium
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 linux needs the '@' in the preamble changed to '%'. See 125 linux needs the '@' in the preamble changed to '%'. See
126 http://codereview.chromium.org/215026. 126 http://codereview.chromium.org/215026.
127 mac/icudt42l_dat.s needs one line added after it is generated. A 127 mac/icudt42l_dat.s needs one line added after it is generated. A
128 .private_extern directive needs to be added so that the top of the 128 .private_extern directive needs to be added so that the top of the
129 file looks like: 129 file looks like:
130 130
131 .globl _icudt42_dat 131 .globl _icudt42_dat
132 .private_extern _icudt42_dat 132 .private_extern _icudt42_dat
133 .data 133 .data
134 134
135 - cros/icudt42l_dat.s : Built on Linux with
136 abriged locale source files in cros/locales put
137 in source/data/locales. Those abridged locales files are
138 for locales ChromeOS is not localized to. '@' in the preamble
139 is replaced with '%' as in linux.
140
135 12. The header files were moved as shown below: 141 12. The header files were moved as shown below:
136 142
137 source/common/unicode ==> public/common/unicode 143 source/common/unicode ==> public/common/unicode
138 source/i18n/unicode ==> public/i18n/unicode 144 source/i18n/unicode ==> public/i18n/unicode
139 145
140 13. The patch for a memory leak in i18n/timezone.cpp (Windows only): 146 13. The patch for a memory leak in i18n/timezone.cpp (Windows only):
141 see http://bugs.icu-project.org/trac/ticket/7135 147 see http://bugs.icu-project.org/trac/ticket/7135
142 148
143 - patches/tzmemory.patch 149 - patches/tzmemory.patch
144 150
145 14. The patch for a crash in common/putil.c (Linux only): 151 14. The patch for a crash in common/putil.c (Linux only):
146 see http://bugs.icu-project.org/trac/ticket/7177 152 see http://bugs.icu-project.org/trac/ticket/7177
147 153
148 - patches/linuxtz.patch 154 - patches/linuxtz.patch
149 155
150 15. The patch for Linux locale detection 156 15. The patch for Linux locale detection
151 157
152 - patches/locdet.patch 158 - patches/locdet.patch
153 159
154 16. Merge the upstream change to get rid of a few compiler warnings 160 16. Merge the upstream change to get rid of a few compiler warnings
155 161
156 - patches/remove_warnings.patch 162 - patches/remove_warnings.patch
OLDNEW
« no previous file with comments | « no previous file | third_party/icu/cros/locales/am.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698