Here is the patch.
http://www.mediafire.com/?4479dga59bs1s9j
I cleaned it up a little, and changed it from copying the string to storing a const reference. This should keep the string object from poofing without any extra overhead. For the exact reason as to why this is a problem, see the following.
http://stackoverflow.com/questions/13744...-confusion
I also included minor changes to common.h and version.cpp to facilitate compiling with ICC. The common.h change simply enables the _M_SSE define when using ICC, and the version.h change appends -ICC to the window title when compiled with ICC (as requested by neobrain). You can leave them out if you feel them unnecessary, as they aren't part of the actual bugfix. They should save other people from making some mistakes when compiling with ICC themselves though, and myself a little time when making ICC builds.
http://www.mediafire.com/?4479dga59bs1s9j
I cleaned it up a little, and changed it from copying the string to storing a const reference. This should keep the string object from poofing without any extra overhead. For the exact reason as to why this is a problem, see the following.
http://stackoverflow.com/questions/13744...-confusion
I also included minor changes to common.h and version.cpp to facilitate compiling with ICC. The common.h change simply enables the _M_SSE define when using ICC, and the version.h change appends -ICC to the window title when compiled with ICC (as requested by neobrain). You can leave them out if you feel them unnecessary, as they aren't part of the actual bugfix. They should save other people from making some mistakes when compiling with ICC themselves though, and myself a little time when making ICC builds.