Progress bar "fetching server status..." sometimes never ends

[GoogleCodeExporter]

What steps will reproduce the problem?
1. set up a game server with a bad config : 
  -------------------- 8< ------------------
  set sv_maxclients "12 //max clientslots available on the server, using more than 16 is not advised. It can cause lag and most maps are not built for it. Going over 24 can cause nasty bugs.
  -------------------- 8< ------------------
2. restart game server
3. make urtsb refresh the server list

What is the expected output? 
- urtsb do not fail on reading sv_maxclients value
- progress bar should end despite the error

What do you see instead?
- progress bar never ends and displays "fetching server status (1353/1354) ..." 
forever.
- clicking the stop button does not change anything and the "search" button 
stays in disabled state. The only way to start a new search is to restart the 
program.
- error : invalid literal for int() with base 10: '12 //max clientslots 
available on the server, using more than 16 is not advised. It can cause lag 
and most maps are not built for it. Going over 24 can cause nasty bugs.'

What version of the product are you using? 
- witnessed on 0.4 windows release
- reproduced out of source from trunk

On what operating system?
- Windows 7 64bit

Original issue reported on code.google.com by thomasle...@gmail.com on 21 Oct 2011 at 11:03

Attachments:

• urtsb_fetch_forever.log
• urtsb_fetching_forever.png

[GoogleCodeExporter]

attached is a fix that properly decrements the running threads counter if an 
exception is raised within a thread

Original comment by thomasle...@gmail.com on 21 Oct 2011 at 11:09

Attachments:

• issue7.patch