The class that I experience the problem with is the ctlSQLGrid, which extends wxGrid. I don't know wxWidgets at all, but in the grid sample class they have an example of setting a cell renderer, perhaps you could add a cell renderer that changes the background color [or even renders a string that says "Sorry, too long!"] based on the strlen? That should have fairly minimal performance impact since it will only affect the N rows being actually rendered, not the entire result set.
The example is /wxWidgets-2.8.11/samples/grid/griddemo.cpp line 299.
Jeff
On Tue, Jul 27, 2010 at 4:39 PM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:
Le 27/07/2010 22:26, Jeff Adams a écrit :
> The bug is definitely a wxWidgets bug, and exists both in 2.8.11 and 2.9.1.
> I was able to reproduce it in their "grid" sample.
>
> I submitted a bug to the wxWidgets project:
>
>
http://trac.wxwidgets.org/ticket/12276 >
> Do you think pgAdmin should implement some sort of temporary fix in the
> meanwhile?
>
It would be best to have a (even temporary) fix. I'm not sure how we can
do that. Having a special background colour could do that. But that
means we need to check the length of every cells. I suppose that, if you
have cells with more than 50k characters, you'll also have big tables.
It sure will take a lot of time to check that. And what should we do if
we find cells with more than X thousand characters? Highlight them? Draw
something? I don't know. All I can say is that it won't be cheap
(performance-wise).
--
--
Jeff Adams
Azavea (formerly Avencia) | T: 215.701.7717 | F: 215.925.2663
www.azavea.com