gnash-commit
[Top][All Lists]
Advanced

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

Re: [Gnash-commit] gnash ChangeLog server/asobj/Key.cpp server/mov...


From: Sandro Santilli
Subject: Re: [Gnash-commit] gnash ChangeLog server/asobj/Key.cpp server/mov...
Date: Tue, 17 Jul 2007 18:15:04 +0200

On Mon, Jul 16, 2007 at 05:21:47PM +0800, zou lunkai wrote:

> The problem is the std::set container just support read-only
> structure,  then the following code cann't even compile :

Argh, this is annoying.
I see these possible solutions:

1) Use a std::list (but you'd have to maintain order yourself in this case)
2) Use a std::map (but it'd take more space)
3) Keep using a std::set and const_cast the elementd pointed-to by the iterator

Option 3 seems the quicker. A comment above the const_cast use might warn
the developers about what would be dangerous and what not.
I think we index by pointer so hardly any operation would be dangerous.

--strk;




reply via email to

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