Mathias Weber wrote: > So finally i got a first version of the resource system as discussed > before. I'm sure it isn't yet perfect. There are some position in the > code where i don't know how to do it better. So i thought i post it > here and some one may take a look at it.
Looks good. I made two changes: core/src/resource.py uses yield_execution to suspend all blocking modules now. This removed the callback as parameter and we don't need the internal callback function. ui/src/video/player.py does not implement suspend and resume (and returns False on can_suspend) because we can't resume the video (yet). One other idea: maybe add a parameter resume_other=True/False to free_ressources and by doing that merge resume_all into that function. And maybe we can add a force=True/False to reserve that will return False (not possible), True (all clear) or InProgress (possible, takes a while). This would remove the suspend_all function. I can do that if you agree that this sounds like a good idea. Dischi -- If you're not part of the solution, be part of the problem!
pgp2IN1YOOHAN.pgp
Description: PGP signature
------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________ Freevo-devel mailing list Freevo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freevo-devel
