[MPlayer-dev-eng] [PATCH] EOSD support for VDPAU

[Reimar Döffinger]

On Sun, Feb 22, 2009 at 05:27:13PM +0100, Grigori G wrote:
> Also,
> 
>  > Also, you do not free those surfaces in uninit it seems?
> 
> I was under the impression that destroying a VdpDevice also destroys all 
> objects belonging to it. Reading the API docs again, this doesn't seem 
> to be the case. They're destroyed when the the display gets preempted, 
> guess I misinterpreted that last time...

I would assume that they do get freed, though I would still consider it
not good style not to free them - also all other resources are freed in
uninit already, better keep it consistent. It might also make it easier
to reinitialize some parts if it will become necessary for some reason
(e.g. handling cases when we run out of video card memory - I think
VDPAU fails hard for that currently so we can't do much about it, but
maybe NVidia will improve that).