380196 - Kill QT graphics code

Status: RESOLVED FIXED

(Core :: Widget, defect)

Description

[Ryan Jones-Ward [:sciguyryan]]

This is the same idea as bug 326152 but this time with the QT code.

Comment 1

[Ryan Jones-Ward [:sciguyryan]]

Attachment: Patch v1

Patch v1

This basically kills the QT make stuff (and a few other things) but doesn't include the removed files to ease the reviewer.

Comment 2

[Ryan Jones-Ward [:sciguyryan]]

Attachment: Removed file list v1

Removed file list v1

This is a list of the files to be removed. I will upload a complete diff patch if this gets a positive review.

Comment 3

[Peter Weilbacher]

Just wondering why you remove the comment in widget/src/os2/nsDeviceContextSpecOS2.cpp. I agree that the OS/2 widget code needs some cleanup, but this shouldn't happen in a QT-removal bug...

Comment 4

[Ryan Jones-Ward [:sciguyryan]]

(In reply to comment #3)
> Just wondering why you remove the comment in
> widget/src/os2/nsDeviceContextSpecOS2.cpp. I agree that the OS/2 widget code
> needs some cleanup, but this shouldn't happen in a QT-removal bug...
> 

Its referring to files that no longer exist so its pretty useless keeping them there (xlib and the gtx files were removed in bug 326152 and the QT one will be removed here.

If anyone objects it can be easily removed though :)

Comment 5

[Ryan Jones-Ward [:sciguyryan]]

Attachment: Removed file patch v1

Removed file patch v1

Comment 6

[Ryan Jones-Ward [:sciguyryan]]

Attachment: Removed file patch v1.1

Un-bitrotted.

Comment 7

[Peter Weilbacher]

I was just considering to check this in but I noticed that attachment 266463 [details] [diff] [review] and attachment 264840 [details] [diff] [review] are identical, even the file name of the patches?! Well, for the check-in it's not important what the files contain just that they get removed. So did you forget any files in the earlier patch that should be included?

Comment 8

[Ryan Jones-Ward [:sciguyryan]]

(In reply to comment #7)
> I was just considering to check this in but I noticed that attachment 266463 [details] [diff] [review]
> and attachment 264840 [details] [diff] [review] are identical, even the file name of the patches?! Well,
> for the check-in it's not important what the files contain just that they get
> removed. So did you forget any files in the earlier patch that should be
> included?
> 

No. The file list was the same but the contents of the files were changed but that wouldn't matter if they were CVS removed anyway. I updated in case anyone wanted to use the patch version instead.

Comment 9

[Peter Weilbacher]

That's what I meant, even the content of the files in the patch are identical:
   $ diff -s attachment264840 [details] [diff] [review].cgi attachment266463 [details] [diff] [review].cgi
   Files attachment264840 [details] [diff] [review].cgi and attachment266463 [details] [diff] [review].cgi are identical
Perhaps you by mistake uploaded the older file again?

Anyway, I checked in the removal patch (attachment 264378 [details] [diff] [review]) and cvs removed the files (attachment 264379 [details]). Marking fixed.