[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Georg Klein
Subject: Re: [libcvd-members] libcvd/cvd gl_helpers.h
Date: Tue, 8 May 2007 15:12:16 +0100 (BST)

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
- 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?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]