On Thu, 1 Sep 2011 21:29:51 -0300 Gustavo Sverzut Barbieri <barbi...@profusion.mobi> said:
> 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. i totally agree here. each app should be responsible for storing previous playback positions for media.. IF it wants to "pick up where it left off" and store this not ON the file itself - eg in ~/.appname/myconfig.cfg/eet/whatever or ~/.config/appname/..... sure - we can provide api's and libraries that make this easier to do - but it shouldnt be in xattr. > 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 > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ 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
