[Webkit-unassigned] [Bug 6444] New: loader.* should be split into one file per class

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Mon Jan 9 00:01:56 PST 2006


http://bugzilla.opendarwin.org/show_bug.cgi?id=6444

           Summary: loader.* should be split into one file per class
           Product: WebKit
           Version: 412+
          Platform: Macintosh
        OS/Version: Mac OS X 10.4
            Status: NEW
          Severity: normal
          Priority: P4
         Component: WebKit Misc.
        AssignedTo: webkit-unassigned at opendarwin.org
        ReportedBy: macdome at opendarwin.org


loader.* should be split into one file per class

The attached patch does just that.

I used the attached script on "loader.h" to svn copy loader.* to a new file for each of the contained classes.  
Then I went through and manually cleaned up the headers a bit, and stripped down the .cpp files.  Finally I 
got the rest of WebCore back compiling by going through all the other .cpp files and fixing up the 
necessary includes.

This patch not only makes the code *much* easier to read, but also substantially reduces header 
dependencies between files.

-- 
Configure bugmail: http://bugzilla.opendarwin.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