[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-25 Thread pepeto
Update of patch #1983 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-25 Thread Jacob Nevins
Follow-up Comment #2, patch #1983 (project freeciv): After this patch, I noticed that file #10365 from bug #16708 is loading differently on my machine: it's now coming out with the map taller than it is wide, and giving some warnings. Looking at the savefile, I think should *always* have been

[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-25 Thread pepeto
Follow-up Comment #3, patch #1983 (project freeciv): Probably it just means that some savefiles generated by medium-old versions of trunk are a bit dubious, and we shouldn't worry about it. I'm mentioning the behaviour mainly in case it puzzles anyone else. Yes, I changed how it saves/load

[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-25 Thread Matthias Pfafferodt
Follow-up Comment #4, patch #1983 (project freeciv): While staring at the patch to work out what could possibly have caused this, I noticed that it incidentally fixes a bug in setting_bool_str_to_bool(), which was using a function pointer from the wrong part of a union. This could have been

[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-25 Thread Jacob Nevins
Follow-up Comment #5, patch #1983 (project freeciv): Sorry, I wasn't clear; pepeto has already fixed it. (Unless I've missed a different bug?) ___ Reply to this item at: http://gna.org/patch/?1983

[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-23 Thread pepeto
URL: http://gna.org/patch/?1983 Summary: Cleanup for 'enum' options Project: Freeciv Submitted by: pepeto Submitted on: jeudi 23.09.2010 à 18:27 Category: general Priority: 5 - Normal

[Freeciv-Dev] [patch #1983] Cleanup for 'enum' options

2010-09-23 Thread pepeto
Update of patch #1983 (project freeciv): Depends on: = patch #1982 ___ Reply to this item at: http://gna.org/patch/?1983 ___ Message