[Webkit-unassigned] [Bug 7582] c_utility.cpp contains platform dependant code

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Sun Mar 5 21:19:24 PST 2006


http://bugzilla.opendarwin.org/show_bug.cgi?id=7582


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6886|review?                     |review+
               Flag|                            |




------- Comment #6 from darin at apple.com  2006-03-05 21:19 PDT -------
(From update of attachment 6886)
Looks good, r=me.

We could simplify this by doing the malloc outside either if statement, but
it's good as is.

At some point we should test that ucnv_open is fast enough when used over and
over again like this.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list