On Sat, Jun 26, 2010 at 1:01 AM, Davide Alberani <davide.alber...@gmail.com> wrote: > In fact it's not a great improvement: to tell the truth, the numer > of the episode should not be computed in the _build_episode function (but > right now I don't have time for a better solution, and anyway... this > one works :-)
I concur; that's not the best place to keep it. I assume that the bit that processes XML or whatever would be the ideal place to have a counter to keep track of it? But I was busy and didn't have time to figure out your entire code base. :-> For a personal fix, I ended up using just an integer instead of the unknownN type, mostly because the tool that I was writing needs an integer, and it seemed silly to take an integer counter, turn it into a string, only to have to parse it to turn it back into an integer again. I've not yet looked at yours, though. I suppose on of the problems with this is that there is no good way to guarantee an order. I imagine that most of the time it will be the same, but it'll be a the whim of the order that IMDB sends the data, rather than anything that we can predetermine. mrc ------------------------------------------------------------------------------ This SF.net email is sponsored by Sprint What will you do first with EVO, the first 4G phone? Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first _______________________________________________ Imdbpy-help mailing list Imdbpy-help@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/imdbpy-help