Hello all, particularly Cedric.
Before I explain the annoying problem in consequence of it, let me say
that I totally disagree with storing playback position in file xattr.
This kind of thing should be per application, not per file.
1- Let's say you thumbnail a file using emotion, you'll get file changed.
2- Let's say you don't want to remember it. You'll still get file changed.
3- You'll be hitting the disc (or flash) more than desired.
So please remove such thing from Emotion, it's really not there. If
you wish you can make it an optional API call that the application may
give its own xattr key and it's persisted. But really, any application
should have its own database as xattr is not available everywhere and
they will get lots of bugreports with inconsistent behavior.
But let's spot the actual bug, which is VERY annoying if you're using
generic engine:
- you operate on threads using eio, those are marked as
PTHREAD_CANCEL_ENABLE && PTHREAD_CANCEL_ASYNCHRONOUS
- you get tons of position changed, so the thread is often running
- you increment the reference count of emotion
- you quit your application (emotion_test)
- thread is running, sd->refcount is 2
- _smart_data_free is not called
- sd->module->file_close() is not called
- plugin->close() is not called
Then there is no way to communicate the slave should die and it keeps
playing until buffer fills.
Yes, there are ways to make the slave die... but damn it... this thing
with xattr is insane!
--
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: barbi...@gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202
------------------------------------------------------------------------------
Special Offer -- Download ArcSight Logger for FREE!
Finally, a world-class log management solution at an even better
price-free! And you'll get a free "Love Thy Logs" t-shirt when you
download Logger. Secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsisghtdev2dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
