Terrain/RasterWeather: don't reset "weather_time" in Reload()

This attribute shall be modified only by the user.  This kind of error
handling is surprising and loses data.
Terrain/RasterWeather: eliminate method Close()

Close() was only ever called by the RasterWeather destructor, and a
destructor does not need to hold a lock to itself.