URL:
  <http://gna.org/patch/?3232>

                 Summary: Do not reserve space for S_OLD_FORTRESS &
S_OLD_AIRBASE
                 Project: Freeciv
            Submitted by: cazfi
            Submitted on: Sat 31 Mar 2012 05:18:12 PM EEST
                Category: general
                Priority: 5 - Normal
                  Status: Ready For Test
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 2.5.0

    _______________________________________________________

Details:

Special biths S_OLD_FORTRESS and S_OLD_AIRBASE exist only for compatibility
with pre-2.2 savegames. They are never used in running game. There's no point
in having bits reserved for them.

Attached, still almost untested, patch makes them special values ,used only
when loading old savegames, outside legal specials number range. I don't think
this alone gives any memory or savegame size savings as number of legal
specials goes from 11 to 9, still requiring 2*8 or 3*4 bits like before. But
at some poin gen-roads will make it possible to remove also S_ROAD and
S_RAILROAD moving us below the magic limit, and even leaving us one free bit
if we want to add some new special (Nets?)
tile_special_iterate() will generate slightly less and/or faster code when
S_OLD_FORTRESS and S_OLD_AIRBASE are not part of the iteration, I think.



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Sat 31 Mar 2012 05:18:12 PM EEST  Name: FortressNoSpace.diff  Size: 4kB 
 By: cazfi

<http://gna.org/patch/download.php?file_id=15486>

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3232>

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


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to