Hello, On Fri, Apr 12, 2013 at 9:03 AM, thomasg <tho...@gstaedtner.net> wrote: > As there's a clear lack of EFL based libraries :P I've written one for > the TheTVDB.com[1] API. I initially started with this for a tool I'm > working on, but figured it might be useful for other people, maybe > writing video players and such. > > The library is basically a C frontend to the TVDB XML api, it parses > the XML to C structures. For parsing Eina simple xml is used, for > downloading cURL; no other dependencies. It is meant to be high > level and produce directly usable data. > > So far, there are enough function to make it useful (I think), but not > the full TVDB API is represented and not all properties TVDB offers > are used yet. > > This is the first library I've ever written and I'm certainly not a > professional programmer, so it will contain bugs. I'd be very happy > for any constructive feedback and any contribution, but be kind. :)
I just did a quick review of it. I would recommend you to move to use Ecore_Con_Url for just one good reason it will force you to redesign your library with an asynchronous API. Right now, you will have blocking call for any of those network request. You also can't cancel them. I know you can trigger them in a thread, but for such a low bandwidth things it is not really useful and will add some overhead (You will still have the issue of how to cancel them). Also I think that eina xml support partial parsing, so you can parse data as you receive them and will have the result ready as soon as the last packet come in. Anyway good job, -- Cedric BAIL ------------------------------------------------------------------------------ Precog is a next-generation analytics platform capable of advanced analytics on semi-structured data. The platform includes APIs for building apps and a phenomenal toolset for data science. Developers can use our toolset for easy data analysis & visualization. Get a free account! http://www2.precog.com/precogplatform/slashdotnewsletter _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
