[webkit-reviews] review denied: [Bug 19578] [CURL] problem in parseDataUrl : [Attachment 22628] fix the base64 problem ( using fastMalloc/fastFree )

[bugzilla-daemon at webkit.org]

Holger Freyther <freyther at handhelds.org> has denied Mario Bensi
<mbensi at pleyo.com>'s request for review:
Bug 19578: [CURL] problem in parseDataUrl
https://bugs.webkit.org/show_bug.cgi?id=19578

Attachment 22628: fix the base64 problem ( using fastMalloc/fastFree )
https://bugs.webkit.org/attachment.cgi?id=22628&action=edit

------- Additional Comments from Holger Freyther <freyther at handhelds.org>
Thanks for the patch. The for loop needs some more spaces to comply with our
Coding Style Guidelines and I wonder why memcpy can not be used?

I also wonder what problem is getting fixed? Vector<char> out is a temporary
variable, and String(out.data(), out.size()) will make a deep copy of the
string. So how does this fail?