Unknown type for bencode about bittornado

[effigies]

Thanks for the report. (I updated your comment because the type was invisible.)

Looks like file mtimes are floats, which can't be bencoded. The simplest solution will be to coerce the types to int, but I'll need to decide whether it's better to do that when writing to file or when reading the mtimes in the first place. From a quick glance, BitTornado doesn't actually use the mtime, so it probably doesn't matter, but I'll want to look a little closer before breaking something.

[effigies]

Can you verify that the issue_25 branch solves your problem?

[Yuuki2012]

Ok, I'm running the tracker, I'll report back after about 30-60min, or earlier, depending if the issue pops up again or not.

[effigies]

I was able to produce the error by hitting Control-C, forcing it to save the state file and close. And the default save interval is 300s (5min).

[Yuuki2012]

Is that the same error? (I didn't really check that, thought it was normal because I'm forcing it to close, happens with a lot of programs), sometimes it happens and sometimes it doesn't.

[effigies]

Yeah. It goes through Tracker.save_state, which is called at intervals and at shutdown.

In general, tracebacks are a bad sign. If you encounter them when exiting any BitTornado programs, feel free to start another issue.

[Yuuki2012]

Well I just use the tracker, so can't really tell. Atm I have no need for the other programs ^^

[effigies]

How'd your tests go?

[Yuuki2012]

Haven't gotten any errors until now, I restarted it a few times (force closing with CTRL+C) but it saved the dstate file just fine, so I think it's fixed now?

[effigies]

Sounds good. Merging.