[ scummvm-Patches-1906528 ] Managed containers

SourceForge.net noreply at sourceforge.net
Mon Mar 3 22:14:03 CET 2008


Patches item #1906528, was opened at 2008-03-03 22:14
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=1906528&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Martin Doucha (next_ghost)
Assigned to: Nobody/Anonymous (nobody)
Summary: Managed containers

Initial Comment:
I've written ManagedList and ManagedArray implementation because I'll need both of them in CinE. They are noncopyable with least code redundancy possible.

I've also modified Array::remove_at() and Stack::pop() to return void because the only return type that makes sense for ManagedArray::remove_at() is void. ManagedArray patch includes necessary changes to all parts of code where remove_at() or pop() is used (and a little performance gain as side effect of the change). These changes compile cleanly but need testing.

MoviePlayer class in Sword1 engine and CursorManager in graphics could use ManagedArray (see array2.patch for details). The classes should be ready for use in Lure and Parallaction.

I know about patch 1895703, LordHoto has seen both of my patches and agreed with posting them here.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=1906528&group_id=37116




More information about the Scummvm-tracker mailing list