Re: [libcvd-members] libcvd/cvd gl_helpers.h

[Georg Klein]

On Tue, 8 May 2007, Gerhard Reitmayr wrote:

> Georg Klein wrote:
> > > +    struct TextureStore {
> > > +        static std::map<const void *, GLuint> texIds;
> >                 ^^^^^^
> >
> > N.B., this will break if you try to use one image from two different
> > contexts (not that uncommon.)
>
> true, didn't think of that.
> I see the following options:
>
> - use current context as additional index, but adds dependency on glx
> headers
> - do nothing and warn user in documentation (just a simple tool)
> - remove again (I ended up not using it :)
>
> what do you think?

For a similar thing, I use a class which I instantiate for a context
if needed. You could even add something like that to your gl window
class. However, I think this is very much in roll-your-own territory.

If it's left in, the docs should probably also warn that if they
change the image's pixels and try to re-upload the texture, nothing
will happen unless they erase it first.

BTW, what's <LIFT> for?