Closed Bug 96213 Opened 24 years ago Closed 24 years ago

Don't query Xinerama if Xinerama is not active

Categories

(Core :: XUL, defect)

Product:
Core
Component:
XUL
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: fred, Assigned: blizzard)

Details

(Whiteboard: have patch; ready to checkin)

Attachments

(1 file)

Don't query Xinerama if not active
573 bytes, patch
shaver
: superreview+
Details | Diff | Splinter Review
We should not query Xinerama when it is not activated. Patch is attached
Add patch and review keywords
Keywords: patch, review
Status: UNCONFIRMED → NEW
Ever confirmed: true
--> dr
Assignee: trudelle → dr
May God have mercy on us all. The 212 bug spam-o-rama is Now!
QA Contact: aegis → jrgm
[spam] dr@netscape.com's bugs subject to redistribution by chofmann. R!
Assignee: dr → chofmann
Has this patch been OK'd by one of the drivers? Could it be? It looks simple enough, I'd like to get some sense that it's "OK" and that it will be moving towards inclusion before 0.9.5...
chofmann, could you review this bug ? The patch is quite simple and prevents XFree86 warning when running an Xinerama build on a non Xinerama system.
Heh, I don't think chofmann should own this bug... -> blizzard, cc: pavlov -- can you guys have a look at the attached patch. Thanks.
Assignee: chofmann → blizzard
r/sr=blizzard
Assignee: blizzard → blizzard
Thanks Christopher. Can someone with CVS write access commit the patch ?
Comment on attachment 46588 [details] [diff] [review] Don't query Xinerama if not active sr=shaver.
Attachment #46588 - Flags: superreview+
Whiteboard: have patch; ready to checkin
Checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: