[Webkit-unassigned] [Bug 17971] [Curl] FormData processing should be moved to its own class

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=17971


julien.chaffraix at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #19904|0                           |1
        is obsolete|                            |
  Attachment #20992|                            |review?
               Flag|                            |




------- Comment #3 from julien.chaffraix at gmail.com  2008-05-06 17:27 PDT -------
Created an attachment (id=20992)
 --> (http://bugs.webkit.org/attachment.cgi?id=20992&action=view)
Second version (address Eric's comments)

> My only question was if the ownership was right for
> ResourceHandle*, i.e. why is it safe to hold a weak pointer?

Checked and added a comment about why a weak pointer is alright (basically we
can have that since the ResourceHandle has a strong reference on its
FormDataStream).

> Using FILE*
> directly kinda sucks too.  But I guess we don't have any nice C++ wrapper for
> it.
I am afraid we will have to use FILE* until it exists.


> integer overflow?  We really should have some sort of safe-multiply function.

Added an overflow check.

> What the heck does "nmemb" stand for?  A more clear variable name would be
> helpful here.

Number of MEMory Block is my guess. It comes from libcURL documentation.
Updated the name in FormDataStream but left it in ResourceHandleManager
callback as IMHO it is more intuitive there.

Corrected the coding style issues.


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