[Webkit-unassigned] [Bug 28410] New: Add 'icu::' qualifier to ICU C++ names

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28410

           Summary: Add 'icu::' qualifier to ICU C++ names
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jshin at chromium.org
                CC: dglazkov at chromium.org


Chromium is going to upgrade its ICU to ICU 4.2.1. In the process, it's found
that there's a name collision between Chromium and ICU because both has
StringPiece in the global namespace. To prevent this, Chromium decided to build
its ICU with U_USING_ICU_NAMESPACE set to 0 (the default is 1 and ICU dumps all
the names to the global namespace). 

With that change, all the references to ICU's C++ names must be qualified with
'icu::'.  Fortunately, there are very few of them (most references to ICU apis
are C APIs rather than C++ APIs). Chromium on Linux and Mac build just fine
without any change. Chromium on Windows needs a minor change in a single file. 

However, there's a possibility that new references to ICU C++ names will be
introduced to Webkit in the future. I'll send an email to webkit-dev about the
issue (hoping that nobody will be against having to use 'icu::').

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.