On Tue, Dec 9, 2008 at 10:38 AM, <[EMAIL PROTECTED]> wrote: > > So there's a single StringValue class, but sometimes its type is > TYPE_UTF8_STRING and sometimes its type is TYPE_UTF16_STRING? So calling > Value::IsType would require two checks to see what type of string it is? > > I think it would be easier if there was only one string type and > internally it keeps track of whether to use a std::string or std::wstring > (depending on which CreateString method was used). If you call GetString > or GetWString, it'll do the conversion automatically if you created the > string using the other type. This makes it easier for consumers of the > code and shouldn't require changing any callers.
I think value should always be std::string. That's what it reads from disk anyway. And I would advocate changing the callers (but I don't know how big a job that would be). Brett --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Chromium-dev" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/chromium-dev?hl=en -~----------~----~----~----~------~----~------~--~---
