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