[Webkit-unassigned] [Bug 66293] [EFL] Fix and update doxygen documentation for ewk_view

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 17 07:58:52 PDT 2011


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





--- Comment #5 from Grzegorz <g.czajkowski at samsung.com>  2011-08-17 07:58:52 PST ---
(In reply to comment #4)
> (In reply to comment #3)
> > (In reply to comment #2)
> > > Could you please try splitting this into multiple patches?
> > 
> > I agree with you that this patch is huge. What is your proposition to splitting this patch? What will you say about:
> > 
> > patch #1 - changes in ewk_view.cpp + ChangeLog
> > patch #2 - changes in ewk_view.h (typedefs and structs description) + ChangeLog
> > patch #3 - API documentation part 1 + ChangeLog (ewk_view.h)
> > patch #4 - API documentation part 2 (settings) + ChangeLog (ewk_view.h)
> > 
> > These patches will be related with the same bug, they will have the same 
> > ChangeLog.
> >
> 
> Looks like a plan.

It's only proposition :) I didn't split a patch into smaller ones before. I don't know what are general rules for that. Have you ever experienced similar case?

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



More information about the webkit-unassigned mailing list