Limit the size of explainPopup

From: Clemens Eisserer <linuxhippy(at)gmail(dot)com>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: Limit the size of explainPopup
Date: 2012-05-05 11:09:49
Message-ID: CAFvQSYTTut+Q4F4Lrz49tVmXpHGb4gP6hmMxKLi2p2Z+YCdwOw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

I often use pgadmin's graphical explain to see how queries generated
be hibernate are planned&executed by postgresql, and it works really
well.
However, for very long queries pgadmin3 tries to create windows >
65536 pixels, which is not supported by X11 and therefor leads to
XErrors, which in turn cause pgadmin3 to close because of assetions.

I would propose to limit the size of the ExplainPopup to the screen
dimensions, please see the attached patch.

Thank you in advance, Clemens

Attachment Content-Type Size
patch.txt text/plain 504 bytes

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2012-05-05 20:51:32 pgAdmin III commit: Review of language object
Previous Message pgAdmin Trac 2012-05-04 21:30:42 Re: [pgAdmin III] #13: Pasting from apps on Mac OS X doesn't always work