Re: Bug with exceptionally long values.

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(at)lelarge(dot)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).
>
>
> --
> Guillaume
> http://www.postgresql.fr
> http://dalibo.com
>

--
Jeff Adams
Azavea (formerly Avencia) | T: 215.701.7717 | F: 215.925.2663
www.azavea.com