[Webkit-unassigned] [Bug 141625] New: Consolidate duplicate soft-linked functions

[bugzilla-daemon at webkit.org]

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

            Bug ID: 141625
           Summary: Consolidate duplicate soft-linked functions
    Classification: Unclassified
           Product: WebKit
           Version: 528+ (Nightly build)
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Misc.
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ddkilzer at webkit.org
                CC: akling at apple.com, benjamin at webkit.org,
                    bfulgham at webkit.org

Consolidating duplicate soft-linked functions would remove duplicate code created for each function, and remove/consolidate some static pointers that would cause dirty pages when the soft-linked method is used.

Here's a rough estimation of the current duplicate functions:

$ grep -r 'SOFT_LINK(' Source/WebCore | cut -d, -f2 | sed -e 's/^ *//' -e 's/ *$//' | sort | uniq -c | sort -n

   2 CMNotificationCenterAddListener
   2 CMNotificationCenterGetDefaultLocalCenter
   2 CMNotificationCenterRemoveListener
   2 CMTimeCompare
   2 CMTimeCopyAsDictionary
   2 CMTimeRangeGetEnd
   2 CloseThemeData
   2 DrawThemeBackground
   2 IsThemeActive
   2 IsThemeBackgroundPartiallyTransparent
   2 OpenThemeData
   2 QTMakeTime
   2 paintThemePart
   3 CMSampleBufferGetPresentationTimeStamp
   3 CMTimeMake
   3 CMTimebaseGetTime
   4 CMSampleBufferGetFormatDescription
   4 CMTimeGetSeconds
   4 CMTimeMakeWithSeconds
   4 UTTypeCopyPreferredTagWithClass
   4 UTTypeCreatePreferredIdentifierForTag

I'm not sure how much code space each method takes, though.

-- 
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/20150215/47b471ae/attachment-0002.html>