[Webkit-unassigned] [Bug 141655] New: Consolidate soft-linked CMTimeMakeWithSeconds() function

[bugzilla-daemon at webkit.org]

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

            Bug ID: 141655
           Summary: Consolidate soft-linked CMTimeMakeWithSeconds()
                    function
    Classification: Unclassified
           Product: WebKit
           Version: 528+ (Nightly build)
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ddkilzer at webkit.org
            Blocks: 141625

Consolidate soft-linked CMTimeMakeWithSeconds() function into a single header/source file.

Advantages:
- In the future, using a soft-linked function that is already defined is as easy as including a new header file in any source file.
- Reduces duplicate code for soft-linking the same framework and the same method in multiple source files.
- Reduces static variables that cause dirty pages for soft-linking the same framework and the same method in multiple source files.
- New code puts static soft-link methods into the WebCore namespace.
- Inline method moved to header, making it more likely to be inlined and thus fixing weak external symbol issues worked around in Bug 141607.

Disadvantages:
- This strategy requires separate macros for header (declaration) and source (implementation) files, but this is mitigated by extra declarations which will cause the compile to fail if they don't match.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150216/4e135ba0/attachment-0002.html>