Closed Bug 768723 Opened 9 years ago Closed 9 years ago

Split up BasicLayers.cpp into a few files

Categories

(Core :: Graphics: Layers, defect)

15 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: nrc, Assigned: nrc)

Details

Attachments

(1 file, 1 obsolete file)

It's just too big and it makes my head hurt
Attached patch split up basiclayers.cpp (obsolete) — Splinter Review
Attachment #637000 - Flags: review?(roc)
Comment on attachment 637000 [details] [diff] [review]
split up basiclayers.cpp

Review of attachment 637000 [details] [diff] [review]:
-----------------------------------------------------------------

I think if you use hg copy to copy BasicLayers.cpp to all the new files, then delete the code each file doesn't need, we can still have usable history for each file and a cleaner diff to review here.
Attachment #637000 - Attachment is obsolete: true
Attachment #637000 - Flags: review?(roc)
Attachment #637704 - Flags: review?(roc)
https://hg.mozilla.org/mozilla-central/rev/3c5f8eb756bf
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Usable history: great.

Making diffstat think you removed 40000 lines of code: awesome.

hg log --stat -r 3c5f8eb756bf
You need to log in before you can comment on or make changes to this bug.