ichthyo/LUMIERA.clone
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

note some unresolved problems with our DummyPlayer

Browse source
This commit is contained in:
Fischlurch 2013-01-04 06:00:35 +01:00
parent 3d628b6eee
commit ef9a6e6f11
2 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/include/dummy-player-facade.h
View file

@ -57,6 +57,11 @@ namespace lumiera {
* experiences regarding integration of the application layers.
* Lumiera is not yet able actually to deliver rendered video data.
*
* In hindsight, this design study highlighted some relevant problems
* with our interface layout and the way we create bindings to the
* implementation. The moment we attempt to use other abstractions
* within an interface (as we do here with the Process interface),
* we're running into serious maintenance and library dependency problems.
*/
class DummyPlayer
{

3
src/proc/play/dummy-player-interface-proxy.hpp
View file

@ -74,6 +74,7 @@ namespace lumiera {
* it directly within the client (=GUI) bypasses the C interface
* and thus leaves us only with one level of indirection,
* irrespective if using the C or C++ interface.
* @note in hindsight this turned out as a very bad idea
*/
Process start(LumieraDisplaySlot viewerHandle)
{
@ -83,7 +84,7 @@ namespace lumiera {
throw lumiera::error::State("failed to start DummyPlayer", lumiera_error());
return pP->createHandle();
}
} //////////////////TODO here we get library linking problems: createHandle is implemented in liblumieraproc.so and this is necessarily so.