This reverts commit df876cedb5.
This commit broke internal builds with the error message:
----
CMake Error at CMakeLists.txt:422 (target_link_libraries):
Cannot specify link libraries for target "phonenumber" which is not built
by this project.
----
Includes corresponding changes for C++ and JavaScript as well to keep the implementations in sync. (C++ and JS didn't exhibit buggy behavior because the corresponding substring methods don't throw errors for invalid start positions.)
Bugfix for https://github.com/googlei18n/libphonenumber/issues/592
that can be followed easily (the re2 stuff with Mercurial was removed
because it's a rat-hole and currently needs a manual patch to the sources
which is not something that this document should be trying to describe.
This also requires updating code that previously took a shortcut and
depended on the Rune and char32 typedefs to be identical (which is no
longer the case, as Rune is now unsigned).
R=jia.shao.peng@gmail.com
BUG=
Review URL: https://codereview.appspot.com/136920043
With strict error checking, 32 bit compiles on some platforms fail with
the error:
"error: this decimal constant is unsigned only in ISO C90"
This patch fixes the error by adding the "u" unsigned qualifier to the
integers.
Author: daniel@pocock.proR=jia.shao.peng@gmail.com
BUG=
Review URL: https://codereview.appspot.com/123610044
They generated static initializers on Clang/Mac.
This also adds the previously missing <algorithm> include in unicodetext.cc
causing a compilation error on VS2013.
This is a squash of (very slightly modified) patches contributed by thakis@ and
yosin@.
R=roes@google.com
Review URL: https://codereview.appspot.com/14930044