[Webkit-unassigned] [Bug 20953] Rework the Font handling in QtWebKit

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Dec 4 09:18:38 PST 2008


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


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #23592|review?(hyatt at apple.com)    |review+
               Flag|                            |




------- Comment #18 from darin at apple.com  2008-12-04 09:18 PDT -------
(From update of attachment 23592)
Moving to a separate file seems fine.

> +/**

We don't use that double-star format.

> + * This file is part of the html renderer for KDE.

You can leave this out of the new file.

> + * Copyright (C) 1999 Lars Knoll (knoll at kde.org)
> + *           (C) 1999 Antti Koivisto (koivisto at kde.org)
> + *           (C) 2000 Dirk Mueller (mueller at kde.org)
> + * Copyright (C) 2003, 2006 Apple Computer, Inc.
> + * Copyright (C) 2008 Holger Hans Peter Freyther

None of the code you're moving is covered by the older copyrights (older than
2003); it's all new stuff. So you can omit those older copyright notices from
this new file.
> +    return data;
> +}
> +
> +
> +
> +}

We just use one blank line in places like this, not 3.

r=me on moving this code


-- 
Configure bugmail: https://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.



More information about the webkit-unassigned mailing list