From ScummVM :: Wiki
Revision as of 21:38, 18 December 2006 by Clone2727 (talk | contribs) (Engine TODO Pages)
Jump to navigation Jump to search

Speaking of: "Get rid of the incRef/decRef API of class File. Instead, add a clone() method which generates a new (independant) File object for the same file (only would work for files in read mode, obviously)." - Wouldn't it be nicer to make the copy constructor and the operator = of Common::File public and making them doing a clone of the file? Since it wouldn't be possible to implement a Common::File clone() like method until those are protected. If the copy constructor and operator = would be doing the same like clone there would be no need for clone again, but if those don't do the same like clone it would be unsafe for code like: "void foo(Common::File bar);" -- LordHoto 4:47, 26 July 2006 (UTC)

Just thought about an Common::File* as return value, but I don't like it to have to clean up with delete after a Common::File::clone call, I would prefer a custom copy constructor and operator = -- LordHoto 20:54, 27 August 2006 (UTC)

Engine TODO Pages

I was looking at one of the individual engine todo pages and I thought of something. Why not make their respective todo pages subpages of the engine page? (And, yes, I would do the moving) -Clone2727 21:38, 18 December 2006 (UTC)