Commit Graph

6 Commits

Author SHA1 Message Date
Davide Andreoli 902307ccaf Emotion Generic: receiving the size on opening is correct.
Also: more comments to simplify command debugging.
2014-02-20 23:01:06 +01:00
Davide Andreoli 1a9952eca7 Emotion generic interface: FILE_SET_DONE do not have any param. 2014-02-16 14:14:23 +01:00
Gustavo Sverzut Barbieri 91a2711225 emotion_generic: fix permissions, resource leaks and error handling.
- create permissions as 0700, this should be private to the running user.

 - do shm_unlink() on error, prevents resource leaks;

 - apply close() on error, prevents resource leaks;

 - handle strerror() properly.

 - shm name uses process id, object name and ecore's time, should be
   cleaner and avoid possible clashes

 - ecore_exe do not need to read from slave stdout/err, this removing
   the flags allows messages to come visible at terminal.
2013-12-21 17:08:35 -02:00
Gustavo Sverzut Barbieri 376ca3f09b fix doc typo, ernno usage and add missing close().
- fix typo, it's slave not slayer... :-D

 - errno: you can't use errno to report error from failed mmap() if
   you do another call that sets errno after that, in that case
   fprintf(). So have a single print so we can use the errno value of
   the proper call.

 - although the slave generic player is not that important if it leaks
   its single shmfd, let's be pedantic and close it. Manpage says it
   is safe to close filedescriptor after mmap() worked.
2013-12-21 15:02:52 -02:00
Davide Andreoli 60a9d16676 emotion_generic: tabs-- 2013-09-01 18:43:13 +02:00
Gustavo Sverzut Barbieri dfb84c1657 efl: merge emotion.
this one was quite a huge work, but hopefully it's correct.

NOTES:
 * removed vlc generic module, it should go into a separate package.
 * gstreamer is enabled by default (see --disable-gstreamer)
 * xine is disabled by default (see --enable-gstreamer)
 * generic is always built statically if supported
 * gstreamer and xine can't be configured as static (just lacks command line options, build system supports it)
 * v4l2 is enabled by default on linux if eeze is built (see --disable-v4l2)
 * emotion_test moved to src/tests/emotion and depends on EFL_ENABLE_TESTS (--with-tests), but is still installed if enabled.

TODO (need your help!):
 * fix warnings with gstreamer and xine engine
 * call engine shutdown functions if building as static
 * remove direct usage of PACKAGE_*_DIR and use eina_prefix
 * add eina_prefix checkme file as evas and others
 * add support for $EFL_RUN_IN_TREE
 * create separate package for emotion_generic_modules
 * check docs hierarchy (doxygen is segv'in here)



SVN revision: 82501
2013-01-10 03:43:32 +00:00