SoCo/SoCo — pull request #22 by KennethNielsen (2013-06-21 15:56:03+00:00)
Title: Made playername optional, since not providing it resulted in no action

This has been bugging me for a while, since it is one of the first methods you see, when you open up the file. Anyway, the `set_player_name` method has only one argument and it is optional. However, since all available code is inside an if that checks if you provided it, not providing it will result in the method executing absolutely no code. I don't think this makes much sense and so this pull request makes the `playername` argument mandatory.

Note, while testing the fix I noticed that this method raises an exception. It works, but raises an exception I guess since the "mission accomplished" return value has changed. In any case that was also the case before, so it should not affect this pull request. I will open an issue for it separately.

I consider this a simple fix and will pull in a few days if there are wild objections, unless someone reviews in the meantime.

Regards Kenneth