The International Simutrans Forum

Simutrans Extended => Simutrans-Extended development => Topic started by: Milko on July 25, 2013, 06:32:20 PM

Title: branch 11.x double line?
Post by: Milko on July 25, 2013, 06:32:20 PM
Hello James

Last 11.x branch

file: dataobj/einstellungen.cc, line 1898/1897, there are two lines (add in one of last commits):
unprotect_abondoned_player_months = contents.get_int("unprotect_abondoned_player_months", unprotect_abondoned_player_months );   
unprotect_abondoned_player_months = contents.get_int("unprotect_abandoned_player_months", unprotect_abondoned_player_months ); // Correcting spelling error.

Giuseppe
Title: Re: branch 11.x double line?
Post by: Dwachs on July 25, 2013, 07:22:42 PM
See the tiny difference:

unprotect_abondoned_player_months = contents.get_int("unprotect_abondoned_player_months", unprotect_abondoned_player_months );   
unprotect_abondoned_player_months = contents.get_int("unprotect_abandoned_player_months", unprotect_abondoned_player_months );

both spellings of the setting are read to keep backward compatibility. This is now also fixed in standard.
Title: Re: branch 11.x double line?
Post by: jamespetts on July 25, 2013, 08:28:21 PM
Thank you for pointing this out, but Dwachs is correct: it is intentional.

Dwachs - presumably the fix in Standard has a similar effect?
Title: Re: branch 11.x double line?
Post by: Dwachs on July 26, 2013, 06:01:37 AM
Quote from: jamespetts on July 25, 2013, 08:28:21 PM
Dwachs - presumably the fix in Standard has a similar effect?
Yes, implementation is similar, I only took the freedom to correct the variable name as well.