[Webkit-unassigned] [Bug 64297] [Qt][WK2] Add the Web Inspector to WebKit2

[bugzilla-daemon at webkit.org]

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





--- Comment #61 from Alexis Menard <alexis.menard at openbossa.org>  2011-08-30 04:53:40 PST ---
(In reply to comment #60)
> Noam,
> 
> Please review my comments.
> 
> I'll replace vars (remove m_ prefix)
> 
> Please explain why is so principal to use OwnPtr, what benefits one will get from this change

OwnPtr will delete the object it holds for you whenever the pointer dies. It's a nice practice to use it, just as the WebKit code base is doing.

> 
> Please review again submitted patch and try send me requests to ,hope, all unacceptable elements in patch
> 
> thanks a lot

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