[Webkit-unassigned] [Bug 35087] New port: EFL; adding files to WebCore/*/efl

[bugzilla-daemon at webkit.org]

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


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #49227|review?, commit-queue?      |review-
               Flag|                            |




--- Comment #29 from Eric Seidel <eric at webkit.org>  2010-02-22 11:44:46 PST ---
(From update of attachment 49227)
Half of your files are LGPL the other half are BSD.  Is this intentional?

Also, I don't understand why you're listing any copyright from 2008 (or before
2010).  If the work wasn't published, I'm not sure how it can have copyright.

I think normally we leave a blank line here, but I don't think there is an
official style:
+namespace WebCore {
+PassRefPtr<Clipboard> Editor::newGeneralClipboard(ClipboardAccessPolicy
policy)

This patch would be much easier to handle in smaller pieces.

See examples from other recent ports, like BREW or Haiku, how they posted
smaller patches grouped by related files.  Personally I target my patches to be
< 20k, although for simple patches like these that's not always needed.  The
problem here is that some of this patch is simple,a nd some of it actually has
real logic.  Would be much better to post the simple patches of just
"notImplemented()" code in a separate patch.

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