URL:
  <http://gna.org/bugs/?24467>

                 Summary: decoding error
                 Project: Freeciv
            Submitted by: dunnoob
            Submitted on: Thu 25 Feb 2016 11:03:52 PM UTC
                Category: None
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: None
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
                 Release: 
         Discussion Lock: Any
        Operating System: None
         Planned Release: 

    _______________________________________________________

Details:

2.5.1 server "decoding error", see the attached decoding logfile.

Background:  The experimental ruleset has four city radius effects
with corresponding city vision effects.  I adopted this in smaller
steps 4 (2,0), 5 (2,1), 8 (2,2), 9 (3,0), etc.; instead of bigger
jumps 2 (1,1), 5 (2,1), 10 (3,1), 17 (4,1).

Minor nit, for radius > vision the invisible tiles are additionally
unworkable, that should be documented somewhere.  I now always have
radius < vision in seven steps up to 16 (4,0) < 17 (4,1).

The experimental ruleset has an initial "vision 0" effect (value 2)
and a game.ruleset init_vis_radius_sq (value 5).  On the other hand
it has no initial "radius 0" for the similar init_city_radius_sq.

In my version I started at "radius 2" (value 1) and got 4 + 1 = 5
for init_city_radius_sq 4.  Then I tested to start with "vision 2"
(value 3) expecting 5 + 3 = 8 for init_vis_radius_sq 5.

At that point I got the unclear "decoding error" for a save game
with an explicit "vision 0".  Wild guess:  init_city_radius_sq does
what its name says, and initializes City_Radius_Sq.  The similar
init_vis_radius_sq does NOT initialize City_Vision_Radius_Sq.

IMO this can't be as it should be, either city radius should work
like the city vision, or city vision should work like the radius.

CAVEAT, city radius (4,1) with vision (3,3) would not work, the
smaller radius sqrt(17) for (4,1) covers more iso tiles than the
bigger radius sqrt(18) for (3,3), shown on Wiki page [[Radius]].





    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?24467>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to