Re: Trial conversion from PLF to PLFHandle

[Ingo Ruhnke]

Gervase Lam <address@hidden> writes:

> I got an update from CVS.  However, one of the main errors I am getting is:
>
> || In file included from angel.cxx:23:
> /home/gervase/pingus/cvs/Games/Pingus/src/actions/../pingus_resource.hxx|26| 
> 26: plf_handle.hxx: No such file or directory
> 
> There has been changes to convert from PLF to PLFHandle in the past
> couple of days. I assume that plf_handle.hxx has not been committed.

File added to CVS now 

> I also assume that this basically means that all PLFs should change
> to PLFHandle? I ask because angel.cxx has PLF and not PLFHandle? Has
> the conversion been finished?

In most cases it could/should be changed, but I am not yet sure that I
like PLFHandle in its current form, after all its just a
'not-so-smart'-smartpointer and only a try to keep the PLF allocation
somewhat local. It also currently has its throuble, since its not able
to reload a file that has changed on disk (happens in the editor).
Replacing/extenting it to something more CL_Surface like might help
(aka ref-counted smartptr, that tells you its refcount).

-- 
WWW:      http://pingus.seul.org/~grumbel/ 
Games:    http://pingus.seul.org/~grumbel/gamedesigns/
JabberID: address@hidden 
ICQ:      59461927