rssanity.c

Go to the documentation of this file.

/***********************************************************************
 Freeciv - Copyright (C) 1996 - A Kjeldberg, L Gregersen, P Unold
   This program is free software; you can redistribute it and/or modify
   it under the terms of the GNU General Public License as published by
   the Free Software Foundation; either version 2, or (at your option)
   any later version.
 
   This program is distributed in the hope that it will be useful,
   but WITHOUT ANY WARRANTY; without even the implied warranty of
   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
   GNU General Public License for more details.
***********************************************************************/
 
#ifdef HAVE_CONFIG_H
#include <fc_config.h>
#endif
 
/* utility */
#include "astring.h"
#include "deprecations.h"
 
/* common */
#include "achievements.h"
#include "actions.h"
#include "effects.h"
#include "game.h"
#include "government.h"
#include "map.h"
#include "movement.h"
#include "nation.h"
#include "player.h"
#include "specialist.h"
#include "tech.h"
 
/* server */
#include "ruleload.h"
#include "settings.h"
 
#include "rssanity.h"
 
/* These effects are always needed in the ruleset.
 * First set are those that are mandatory even in compatibility mode. */
enum effect_type req_base_effects[] =
  {
    EFT_CITY_VISION_RADIUS_SQ,
    EFT_MAX_RATES,
    EFT_UPKEEP_PCT,
    EFT_COUNT
  };
 
/* These have been made mandatory in freeciv-3.4 */
enum effect_type req_base_effects_3_4[] =
  {
    /* None yet */
    EFT_COUNT
  };
 
/**********************************************************************/
static bool sanity_check_metadata(rs_conversion_logger logger)
{
  if (game.ruleset_summary != NULL
      && strlen(game.ruleset_summary) > MAX_LEN_CONTENT) {
    ruleset_error(logger,
                  LOG_ERROR,
                  _("Too long ruleset summary. It can be only %d bytes long. "
                    "Put longer explanations to ruleset description."),
                  MAX_LEN_CONTENT);
    return FALSE;
  }
 
  return TRUE;
}
 
/**********************************************************************/
static bool nation_has_initial_tech(struct nation_type *pnation,
                                    struct advance *tech)
{
  int i;
 
  /* See if it's given as global init tech */
  for (i = 0; i < MAX_NUM_TECH_LIST
       && game.rgame.global_init_techs[i] != A_LAST; i++) {
    if (game.rgame.global_init_techs[i] == advance_number(tech)) {
      return TRUE;
    }
  }
 
  /* See if it's given as national init tech */
  for (i = 0;
       i < MAX_NUM_TECH_LIST && pnation->init_techs[i] != A_LAST;
       i++) {
    if (pnation->init_techs[i] == advance_number(tech)) {
      return TRUE;
    }
  }
 
  return FALSE;
}
 
/**********************************************************************/
static bool sanity_check_setting_is_seen(struct setting *pset)
{
  return setting_is_visible_at_level(pset, ALLOW_INFO);
}
 
/**********************************************************************/
static bool sanity_check_setting_is_game_rule(struct setting *pset)
{
  if ((setting_category(pset) == SSET_INTERNAL
            || setting_category(pset) == SSET_NETWORK)
           /* White list for SSET_INTERNAL and SSET_NETWORK settings. */
           && !(pset == setting_by_name("phasemode")
                || pset == setting_by_name("timeout")
                || pset == setting_by_name("timeaddenemymove")
                || pset == setting_by_name("unitwaittime")
                || pset == setting_by_name("victories"))) {
    /* The given server setting is a server operator related setting (like
     * the compression type of savegames), not a game rule. */
    return FALSE;
  }
 
  if (pset == setting_by_name("naturalcitynames")) {
    /* This setting is about "look", not rules. */
    return FALSE;
  }
 
  return TRUE;
}
 
/**********************************************************************/
bool sanity_check_server_setting_value_in_req(ssetv ssetval)
{
  server_setting_id id;
  struct setting *pset;
 
  /* TODO: use ssetv_setting_get() if setting value becomes multiplexed with
   * the server setting id. */
  id = (server_setting_id)ssetval;
  fc_assert_ret_val(server_setting_exists(id), FALSE);
 
  if (server_setting_type_get(id) != SST_BOOL) {
    /* Not supported yet. */
    return FALSE;
  }
 
  pset = setting_by_number(id);
 
  return (sanity_check_setting_is_seen(pset)
          && sanity_check_setting_is_game_rule(pset));
}
 
/**********************************************************************/
static bool sanity_check_req_individual(rs_conversion_logger logger,
                                        struct requirement *preq,
                                        const char *list_for)
{
  switch (preq->source.kind) {
  case VUT_IMPROVEMENT:
  case VUT_SITE:
    /* This check corresponds to what is_req_active() will support.
     * It can't be done in req_from_str(), as we may not have
     * loaded all building information at that time. */
    {
      const struct impr_type *pimprove = preq->source.value.building;
 
      if (preq->range == REQ_RANGE_WORLD && !is_great_wonder(pimprove)) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: World-ranged requirement not supported for "
                        "%s (only great wonders supported)"), list_for,
                      improvement_name_translation(pimprove));
        return FALSE;
      } else if (preq->range > REQ_RANGE_TRADE_ROUTE && !is_wonder(pimprove)) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: %s-ranged requirement not supported for "
                        "%s (only wonders supported)"), list_for,
                      req_range_name(preq->range),
                      improvement_name_translation(pimprove));
        return FALSE;
      }
    }
    break;
  case VUT_MINCALFRAG:
    /* Currently [calendar] is loaded after some requirements are
     * parsed, so we can't do this in universal_value_from_str(). */
    if (game.calendar.calendar_fragments < 1) {
      ruleset_error(logger, LOG_ERROR,
                    _("%s: MinCalFrag requirement used in ruleset without "
                      "calendar fragments"), list_for);
      return FALSE;
    } else if (preq->source.value.mincalfrag >= game.calendar.calendar_fragments) {
      ruleset_error(logger, LOG_ERROR,
                    _("%s: MinCalFrag requirement %d out of range (max %d in "
                      "this ruleset)"), list_for, preq->source.value.mincalfrag,
                    game.calendar.calendar_fragments-1);
      return FALSE;
    }
    break;
  case VUT_SERVERSETTING:
    /* There is currently no way to check a server setting's category and
     * access level that works in both the client and the server. */
    {
      server_setting_id id;
      struct setting *pset;
 
      id = ssetv_setting_get(preq->source.value.ssetval);
      fc_assert_ret_val(server_setting_exists(id), FALSE);
      pset = setting_by_number(id);
 
      if (!sanity_check_setting_is_seen(pset)) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: ServerSetting requirement %s isn't visible enough "
                        "to appear in a requirement. Everyone should be able to "
                        "see the value of a server setting that appears in a "
                        "requirement."), list_for, server_setting_name_get(id));
        return FALSE;
      }
 
      if (!sanity_check_setting_is_game_rule(pset)) {
        /* This is a server operator related setting (like the compression
         * type of savegames), not a game rule. */
        ruleset_error(logger, LOG_ERROR,
                      _("%s: ServerSetting requirement setting %s isn't about a "
                        "game rule."),
                      list_for, server_setting_name_get(id));
        return FALSE;
      }
    }
    break;
  default:
    /* No other universals have checks that can't be done at ruleset
     * load time. See req_from_str(). */
    break;
  }
  return TRUE;
}
 
/**********************************************************************/
static bool sanity_check_req_set(rs_conversion_logger logger,
                                 int reqs_of_type[],
                                 int local_reqs_of_type[],
                                 int tile_reqs_of_type[],
                                 struct requirement *preq, bool conjunctive,
                                 int max_tiles, const char *list_for)
{
  int rc;
 
  fc_assert_ret_val(universals_n_is_valid(preq->source.kind), FALSE);
 
  if (!sanity_check_req_individual(logger, preq, list_for)) {
    return FALSE;
  }
 
  if (!conjunctive) {
    /* All the checks below are only meaningful for conjunctive lists. */
    /* FIXME: we could add checks suitable for disjunctive lists. */
    return TRUE;
  }
 
  /* Add to counter for positive requirements. */
  if (preq->present) {
    reqs_of_type[preq->source.kind]++;
  }
  rc = reqs_of_type[preq->source.kind];
 
  if (preq->range == REQ_RANGE_LOCAL && preq->present) {
    local_reqs_of_type[preq->source.kind]++;
 
    switch (preq->source.kind) {
    case VUT_EXTRA:
      if (local_reqs_of_type[VUT_EXTRA] > 1) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: Requirement list has multiple local-ranged extra "
                        "requirements (did you mean to make them tile-ranged?)"),
                      list_for);
        return FALSE;
      }
      break;
    default:
      break;
    }
  }
 
  if (preq->range == REQ_RANGE_TILE && preq->present) {
    tile_reqs_of_type[preq->source.kind]++;
 
    switch (preq->source.kind) {
     case VUT_TERRAINCLASS:
       if (tile_reqs_of_type[VUT_TERRAIN] > 0) {
         ruleset_error(logger, LOG_ERROR,
                       _("%s: Requirement list has both tile terrain and terrainclass requirement"),
                       list_for);
         return FALSE;
       }
       break;
     case VUT_TERRAIN:
       if (tile_reqs_of_type[VUT_TERRAINCLASS] > 0) {
         ruleset_error(logger, LOG_ERROR,
                       _("%s: Requirement list has both tile terrain and terrainclass requirement"),
                       list_for);
         return FALSE;
       }
       break;
     case VUT_MINLATITUDE:
     case VUT_MAXLATITUDE:
       if (tile_reqs_of_type[preq->range] > 1) {
         ruleset_error(logger, LOG_ERROR,
                       _("%s: Requirement list has duplicate %s requirement at Tile range"),
                       list_for, universal_type_rule_name(&preq->source));
         return FALSE;
       }
       break;
     default:
       break;
    }
  }
 
  if (rc > 1 && preq->present) {
    /* Multiple requirements of the same type */
    switch (preq->source.kind) {
    case VUT_GOVERNMENT:
    case VUT_ACTION:
    case VUT_ACTIVITY:
    case VUT_OTYPE:
    case VUT_SPECIALIST:
    case VUT_MINSIZE: /* Breaks nothing, but has no sense either */
    case VUT_MINCITIES:
    case VUT_MINFOREIGNPCT:
    case VUT_MINMOVES: /* Breaks nothing, but has no sense either */
    case VUT_MINVETERAN: /* Breaks nothing, but has no sense either */
    case VUT_MINHP: /* Breaks nothing, but has no sense either */
    case VUT_MINYEAR:
    case VUT_MINCALFRAG:
    case VUT_AI_LEVEL:
    case VUT_TERRAINALTER: /* Local range only */
    case VUT_STYLE:
    case VUT_IMPR_GENUS:
    case VUT_ORIGINAL_OWNER: /* City range -> only one original owner */
    case VUT_FORM_AGE:
    case VUT_MAX_DISTANCE_SQ: /* Breaks nothing, but has no sense either */
      /* There can be only one requirement of these types (with current
       * range limitations)
       * Requirements might be identical, but we consider multiple
       * declarations error anyway. */
 
      ruleset_error(logger, LOG_ERROR,
                    _("%s: Requirement list has multiple %s requirements"),
                    list_for, universal_type_rule_name(&preq->source));
      return FALSE;
      break;
 
    case VUT_TERRAIN:
      /* There can be only up to max_tiles requirements of these types */
      if (max_tiles != -1 && rc > max_tiles) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: Requirement list has more %s requirements than "
                        "can ever be fulfilled."), list_for,
                      universal_type_rule_name(&preq->source));
        return FALSE;
      }
      break;
 
    case VUT_TERRAINCLASS:
      if (rc > 2 || (max_tiles != -1 && rc > max_tiles)) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: Requirement list has more %s requirements than "
                        "can ever be fulfilled."), list_for,
                      universal_type_rule_name(&preq->source));
        return FALSE;
      }
      break;
 
    case VUT_AGE:
      /* There can be age of the city, unit, and player */
      if (rc > 3) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: Requirement list has more %s requirements than "
                        "can ever be fulfilled."), list_for,
                      universal_type_rule_name(&preq->source));
        return FALSE;
      }
      break;
 
    case VUT_MINTECHS:
    case VUT_FUTURETECHS:
      /* At ranges 'Player' and 'World' */
      if (rc > 2) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s: Requirement list has more %s requirements than "
                        "can ever be fulfilled."), list_for,
                      universal_type_rule_name(&preq->source));
        return FALSE;
      }
      break;
 
    case VUT_COUNTER:
      /* Can have multiple, since many counters (also of the same range)
       * can meet checkpoint */
    case VUT_SERVERSETTING:
      /* Can have multiple, since there are many settings. */
    case VUT_TOPO:
      /* Can have multiple, since it's flag based (iso & hex) */
    case VUT_WRAP:
      /* Can have multiple, since it's flag based (wrapx & wrapy) */
    case VUT_EXTRA:
    case VUT_TILEDEF:
      /* Note that there can be more than 1 extra / tile. */
    case VUT_MAXTILETOTALUNITS:
    case VUT_MAXTILETOPUNITS:
      /* Can require different numbers on e.g. local/adjacent tiles. */
    case VUT_NATION:
      /* Can require multiple nations at Team/Alliance/World range. */
    case VUT_NATIONGROUP:
      /* Nations can be in multiple groups. */
    case VUT_NONE:
    case VUT_ADVANCE:
    case VUT_TECHFLAG:
    case VUT_IMPROVEMENT:
    case VUT_SITE:
    case VUT_UNITSTATE:
    case VUT_CITYTILE:
    case VUT_GOOD:
    case VUT_UTYPE:
    case VUT_UCLASS:
    case VUT_TILE_REL:
      /* Can check different properties. */
    case VUT_GOVFLAG:
    case VUT_UTFLAG:
    case VUT_UCFLAG:
    case VUT_TERRFLAG:
    case VUT_ROADFLAG:
    case VUT_EXTRAFLAG:
    case VUT_IMPR_FLAG:
    case VUT_PLAYER_FLAG:
    case VUT_PLAYER_STATE:
    case VUT_NATIONALITY:
    case VUT_MINCULTURE:
    case VUT_ACHIEVEMENT:
    case VUT_DIPLREL:
    case VUT_DIPLREL_TILE:
    case VUT_DIPLREL_TILE_O:
    case VUT_DIPLREL_UNITANY:
    case VUT_DIPLREL_UNITANY_O:
      /* Can have multiple requirements of these types */
    case VUT_MINLATITUDE:
    case VUT_MAXLATITUDE:
    case VUT_MAX_REGION_TILES:
      /* Can have multiple requirements at different ranges.
       *  TODO: Compare to number of legal ranges? */
      break;
    case VUT_CITYSTATUS:
      /* Could check "CITYS_LAST * number of ranges" ? */
      break;
    case VUT_COUNT:
      /* Should never be in requirement vector */
      fc_assert(FALSE);
      return FALSE;
      break;
      /* No default handling here, as we want compiler warning
       * if new requirement type is added to enum and it's not handled
       * here. */
    }
  }
 
  return TRUE;
}
 
/**********************************************************************/
static bool sanity_check_req_vec(rs_conversion_logger logger,
                                 const struct requirement_vector *preqs,
                                 bool conjunctive, int max_tiles,
                                 const char *list_for)
{
  struct req_vec_problem *problem;
  int reqs_of_type[VUT_COUNT];
  int local_reqs_of_type[VUT_COUNT];
  int tile_reqs_of_type[VUT_COUNT];
 
  /* Initialize requirement counters */
  memset(reqs_of_type, 0, sizeof(reqs_of_type));
  memset(tile_reqs_of_type, 0, sizeof(tile_reqs_of_type));
 
  requirement_vector_iterate(preqs, preq) {
    if (!sanity_check_req_set(logger, reqs_of_type, local_reqs_of_type,
                              tile_reqs_of_type, preq,
                              conjunctive, max_tiles, list_for)) {
      return FALSE;
    }
  } requirement_vector_iterate_end;
 
  problem = req_vec_suggest_repair(preqs, req_vec_vector_number, preqs);
  if (problem != NULL) {
    ruleset_error(logger, LOG_ERROR, "%s: %s.", list_for, problem->description);
    req_vec_problem_free(problem);
    return FALSE;
  }
 
  return TRUE;
}
 
typedef struct {
  struct {
    bool effect_present[EFT_COUNT];
  } base_effects;
  rs_conversion_logger logger;
} els_data;
 
/**********************************************************************/
static bool effect_list_sanity_cb(struct effect *peffect, void *data)
{
  int one_tile = -1; /* TODO: Determine correct value from effect.
                      *       -1 disables checking */
  els_data *els = (els_data *)data;
  struct astring astr;
  int i;
 
  for (i = 0; req_base_effects[i] != EFT_COUNT; i++) {
    if (peffect->type == req_base_effects[i]) {
      els->base_effects.effect_present[peffect->type] = TRUE;
      break;
    }
  }
  for (i = 0; req_base_effects_3_4[i] != EFT_COUNT; i++) {
    if (peffect->type == req_base_effects_3_4[i]) {
      els->base_effects.effect_present[peffect->type] = TRUE;
      break;
    }
  }
 
  if (peffect->type == EFT_ACTION_SUCCESS_TARGET_MOVE_COST) {
    /* Only unit targets can pay in move fragments. */
    requirement_vector_iterate(&peffect->reqs, preq) {
      if (preq->source.kind == VUT_ACTION) {
        if (action_get_target_kind(preq->source.value.action) != ATK_UNIT) {
          /* TODO: support for ATK_STACK could be added. That would require
           * manually calling action_success_target_pay_mp() in each
           * supported unit stack targeted action performer (like
           * action_consequence_success() does) or to have the unit stack
           * targeted actions return a list of targets. */
          ruleset_error(els->logger, LOG_ERROR,
                        _("The effect Action_Success_Target_Move_Cost has the"
                          " requirement {%s} but the action %s isn't"
                          " (single) unit targeted."),
                        req_to_fstring(preq, &astr),
                        universal_rule_name(&preq->source));
          astr_free(&astr);
          return FALSE;
        }
      }
    } requirement_vector_iterate_end;
  } else if (peffect->type == EFT_ACTION_SUCCESS_MOVE_COST) {
    /* Only unit actors can pay in move fragments. */
    requirement_vector_iterate(&peffect->reqs, preq) {
      if (preq->source.kind == VUT_ACTION && preq->present) {
        if (action_get_actor_kind(preq->source.value.action) != AAK_UNIT) {
          ruleset_error(els->logger, LOG_ERROR,
                        _("The effect Action_Success_Actor_Move_Cost has the"
                          " requirement {%s} but the action %s isn't"
                          " performed by a unit."),
                        req_to_fstring(preq, &astr),
                        universal_rule_name(&preq->source));
          astr_free(&astr);
          return FALSE;
        }
      }
    } requirement_vector_iterate_end;
  } else if (peffect->type == EFT_ACTION_ODDS_PCT
             || peffect->type == EFT_ACTION_RESIST_PCT) {
    /* Catch trying to set Action_Odds_Pct for non supported actions. */
    requirement_vector_iterate(&peffect->reqs, preq) {
      if (preq->source.kind == VUT_ACTION && preq->present) {
        if (action_dice_roll_initial_odds(preq->source.value.action)
            == ACTION_ODDS_PCT_DICE_ROLL_NA) {
          ruleset_error(els->logger, LOG_ERROR,
                        _("The effect %s has the"
                          " requirement {%s} but the action %s doesn't"
                          " roll the dice to see if it fails."),
                        effect_type_name(peffect->type),
                        req_to_fstring(preq, &astr),
                        universal_rule_name(&preq->source));
          astr_free(&astr);
          return FALSE;
        }
      }
    } requirement_vector_iterate_end;
  }
 
  if (!sanity_check_req_vec(els->logger, &peffect->reqs, TRUE, one_tile,
                            effect_type_name(peffect->type))) {
    ruleset_error(els->logger, LOG_ERROR,
                  _("Effects have conflicting or invalid requirements!"));
 
    return FALSE;
  }
 
  return TRUE;
}
 
/**********************************************************************/
static bool rs_barbarian_units(rs_conversion_logger logger)
{
  if (num_role_units(L_BARBARIAN) > 0) {
    if (num_role_units(L_BARBARIAN_LEADER) == 0) {
      ruleset_error(logger, LOG_ERROR, _("No role barbarian leader units"));
      return FALSE;
    }
    if (num_role_units(L_BARBARIAN_BUILD) == 0) {
      ruleset_error(logger, LOG_ERROR, _("No role barbarian build units"));
      return FALSE;
    }
    if (num_role_units(L_BARBARIAN_BOAT) == 0) {
      ruleset_error(logger, LOG_ERROR, _("No role barbarian ship units"));
      return FALSE;
    } else if (num_role_units(L_BARBARIAN_BOAT) > 0) {
      bool sea_capable = FALSE;
      struct unit_type *u = get_role_unit(L_BARBARIAN_BOAT, 0);
 
      terrain_re_active_iterate(pterr) {
        if (is_ocean(pterr)
            && BV_ISSET(pterr->native_to, uclass_index(utype_class(u)))) {
          sea_capable = TRUE;
          break;
        }
      } terrain_re_active_iterate_end;
 
      if (!sea_capable) {
        ruleset_error(logger, LOG_ERROR,
                      _("Barbarian boat (%s) needs to be able to move at sea."),
                      utype_rule_name(u));
        return FALSE;
      }
    }
    if (num_role_units(L_BARBARIAN_SEA) == 0) {
      ruleset_error(logger, LOG_ERROR, _("No role sea raider barbarian units"));
      return FALSE;
    }
 
    unit_type_re_active_iterate(ptype) {
      if (utype_has_role(ptype, L_BARBARIAN_BOAT)) {
        if (ptype->transport_capacity <= 1) {
          ruleset_error(logger, LOG_ERROR,
                        _("Barbarian boat %s has no capacity for both "
                          "leader and at least one man."),
                        utype_rule_name(ptype));
          return FALSE;
        }
 
        unit_type_re_active_iterate(pbarb) {
          if (utype_has_role(pbarb, L_BARBARIAN_SEA)
              || utype_has_role(pbarb, L_BARBARIAN_SEA_TECH)
              || utype_has_role(pbarb, L_BARBARIAN_LEADER)) {
            if (!can_unit_type_transport(ptype, utype_class(pbarb))) {
              ruleset_error(logger, LOG_ERROR,
                            _("Barbarian boat %s cannot transport "
                              "barbarian cargo %s."),
                            utype_rule_name(ptype),
                            utype_rule_name(pbarb));
              return FALSE;
            }
          }
        } unit_type_re_active_iterate_end;
      }
    } unit_type_re_active_iterate_end;
  }
 
  return TRUE;
}
 
/**********************************************************************/
static bool rs_common_units(rs_conversion_logger logger)
{
  /* Check some required flags and roles etc: */
  if (num_role_units(UTYF_WORKERS) == 0) {
    ruleset_error(logger, LOG_ERROR, _("No flag Worker units"));
    return FALSE;
  }
  if (num_role_units(L_START_EXPLORER) == 0) {
    ruleset_error(logger, LOG_ERROR, _("No role Start Explorer units"));
    return FALSE;
  }
  if (num_role_units(L_FERRYBOAT) == 0) {
    ruleset_error(logger, LOG_ERROR, _("No role Ferryboat units"));
    return FALSE;
  }
  if (num_role_units(L_FIRSTBUILD) == 0) {
    ruleset_error(logger, LOG_ERROR, _("No role Firstbuild units"));
    return FALSE;
  }
 
  if (num_role_units(L_FERRYBOAT) > 0) {
    bool sea_capable = FALSE;
    struct unit_type *u = get_role_unit(L_FERRYBOAT, 0);
 
    terrain_re_active_iterate(pterr) {
      if (is_ocean(pterr)
          && BV_ISSET(pterr->native_to, uclass_index(utype_class(u)))) {
        sea_capable = TRUE;
        break;
      }
    } terrain_re_active_iterate_end;
 
    if (!sea_capable) {
      ruleset_error(logger, LOG_ERROR,
                    _("Ferryboat (%s) needs to be able to move at sea."),
                    utype_rule_name(u));
      return FALSE;
    }
  }
 
  if (num_role_units(L_PARTISAN) == 0
      && effect_cumulative_max(EFT_INSPIRE_PARTISANS, NULL, 0) > 0) {
    ruleset_error(logger, LOG_ERROR,
                  _("Inspire_Partisans effect present, "
                    "but no units with partisan role."));
    return FALSE;
  }
 
  unit_type_iterate(ptype) {
    bool cargo = FALSE;
 
    unit_class_iterate(pclass) {
      if (BV_ISSET(ptype->cargo, uclass_index(pclass))) {
        cargo = TRUE;
        break;
      }
    } unit_class_iterate_end;
 
    if (ptype->transport_capacity > 0) {
      if (!cargo) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s has transport capacity %d, but no cargo types."),
                      utype_rule_name(ptype), ptype->transport_capacity);
        return FALSE;
      }
    } else if (cargo) {
      ruleset_error(logger, LOG_ERROR,
                    _("%s has cargo types, but no transport capacity."),
                    utype_rule_name(ptype));
      return FALSE;
    }
  } unit_type_iterate_end;
 
  return TRUE;
}
 
/**********************************************************************/
static bool rs_buildings(rs_conversion_logger logger)
{
  /* Special Genus */
  improvement_re_active_iterate(pimprove) {
    if (improvement_has_flag(pimprove, IF_GOLD)) {
      if (pimprove->genus != IG_CONVERT) {
        ruleset_error(logger, LOG_ERROR,
                      _("Gold producing improvement %s with genus other than \"Convert\""),
                      improvement_rule_name(pimprove));
 
        return FALSE;
      }
      if (improvement_has_flag(pimprove, IF_INFRA)) {
        ruleset_error(logger, LOG_ERROR,
                      _("The same improvement has both \"Gold\" and \"Infra\" flags"));
        return FALSE;
      }
    } else if (improvement_has_flag(pimprove, IF_INFRA)) {
      if (pimprove->genus != IG_CONVERT) {
        ruleset_error(logger, LOG_ERROR,
                      _("Infrapoints producing improvement %s with genus other than \"Convert\""),
                      improvement_rule_name(pimprove));
 
        return FALSE;
      }
    } else if (pimprove->genus == IG_CONVERT) {
      ruleset_error(logger, LOG_ERROR,
                    _("Improvement %s with no conversion target with genus \"Convert\""),
                    improvement_rule_name(pimprove));
      return FALSE;
    }
 
    if (improvement_has_flag(pimprove, IF_DISASTER_PROOF)
        && pimprove->genus != IG_IMPROVEMENT) {
      ruleset_error(logger, LOG_ERROR,
                    _("Disasterproof improvement %s with genus other than \"Improvement\""),
                    improvement_rule_name(pimprove));
 
      return FALSE;
    }
    if (pimprove->genus != IG_SPECIAL
        && (get_potential_improvement_bonus(pimprove, NULL, EFT_SS_STRUCTURAL,
                                            RPT_POSSIBLE, FALSE)
            || get_potential_improvement_bonus(pimprove, NULL, EFT_SS_COMPONENT,
                                               RPT_POSSIBLE, FALSE)
            || get_potential_improvement_bonus(pimprove, NULL, EFT_SS_MODULE,
                                               RPT_POSSIBLE, FALSE))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Space part %s with genus other than \"Special\""),
                    improvement_rule_name(pimprove));
      return FALSE;
    }
 
    if (!is_building_sellable(pimprove) && pimprove->upkeep != 0) {
      ruleset_error(logger, LOG_ERROR,
                    _("%s is a nonsellable building with a nonzero upkeep value"),
                    improvement_rule_name(pimprove));
      return FALSE;
    }
  } improvement_re_active_iterate_end;
 
  return TRUE;
}
 
/**********************************************************************/
static bool sanity_check_boolean_effects(rs_conversion_logger logger)
{
  enum effect_type boolean_effects[] =
    {
      EFT_ANY_GOVERNMENT,
      EFT_ENABLE_NUKE,
      EFT_ENABLE_SPACE,
      EFT_HAVE_EMBASSIES,
      EFT_NO_ANARCHY,
      EFT_NUKE_PROOF,
      EFT_REVEAL_CITIES,
      EFT_REVEAL_MAP,
      EFT_SIZE_UNLIMIT,
      EFT_SS_STRUCTURAL,
      EFT_SS_COMPONENT,
      EFT_NO_UNHAPPY,
      EFT_RAPTURE_GROW,
      EFT_HAS_SENATE,
      EFT_INSPIRE_PARTISANS,
      EFT_HAPPINESS_TO_GOLD,
      EFT_FANATICS,
      EFT_NO_DIPLOMACY,
      EFT_GOV_CENTER,
      EFT_NOT_TECH_SOURCE,
      EFT_VICTORY,
      EFT_HAVE_CONTACTS,
      EFT_COUNT
    };
  int i;
  bool ret = TRUE;
 
  for (i = 0; boolean_effects[i] != EFT_COUNT; i++) {
    if (effect_cumulative_min(boolean_effects[i], NULL) < 0
        && effect_cumulative_max(boolean_effects[i], NULL, 0) == 0) {
      ruleset_error(logger, LOG_ERROR,
                    _("Boolean effect %s can get disabled, but it can't get "
                      "enabled before that."),
                    effect_type_name(boolean_effects[i]));
      ret = FALSE;
    }
  }
 
  return ret;
}
 
/**********************************************************************/
bool sanity_check_ruleset_data(struct rscompat_info *compat)
{
  int num_utypes;
  int i;
  bool ok = TRUE; /* Store failures to variable instead of returning
                   * immediately so all errors get printed, not just first
                   * one. */
  bool default_gov_failed = FALSE;
  bool obsoleted_by_loop = FALSE;
  els_data els;
  rs_conversion_logger logger = ((compat != NULL) ? compat->log_cb : NULL);
 
  if (!sanity_check_metadata(logger)) {
    ok = FALSE;
  }
 
  if (game.info.tech_cost_style == TECH_COST_CIV1CIV2
      && game.info.free_tech_method == FTM_CHEAPEST) {
    ruleset_error(logger, LOG_ERROR,
                  _("Cost based free tech method, but tech cost style "
                    "\"Civ I|II\" so all techs cost the same."));
    ok = FALSE;
  }
 
  /* Advances. */
  advance_re_active_iterate(padvance) {
    for (i = AR_ONE; i < AR_SIZE; i++) {
      const struct advance *preq;
 
      if (i == AR_ROOT) {
        /* Self rootreq is a feature. */
        continue;
      }
 
      preq = advance_requires(padvance, i);
 
      if (A_NEVER == preq) {
        continue;
      } else if (preq == padvance) {
        ruleset_error(logger, LOG_ERROR, _("Tech \"%s\" requires itself."),
                      advance_rule_name(padvance));
        ok = FALSE;
        continue;
      }
 
      advance_req_iterate(preq, preqreq) {
        if (preqreq == padvance) {
          ruleset_error(logger, LOG_ERROR,
                        _("Tech \"%s\" requires itself indirectly via \"%s\"."),
                        advance_rule_name(padvance),
                        advance_rule_name(preq));
          ok = FALSE;
        }
      } advance_req_iterate_end;
    }
 
    requirement_vector_iterate(&(padvance->research_reqs), preq) {
      if (preq->source.kind == VUT_ADVANCE) {
        /* Don't allow this even if allowing changing reqs. Players will
         * expect all tech reqs to appear in the client tech tree. That
         * should be taken care of first. */
        ruleset_error(logger, LOG_ERROR,
                      _("Tech \"%s\" requires a tech in its research_reqs."
                        " This isn't supported yet. Please keep using req1"
                        " and req2 like before."),
                      advance_rule_name(padvance));
        ok = FALSE;
      } else if (is_req_unchanging(NULL, preq) < REQUCH_HACK
                 /* If we get an obsolete improvement before the game,
                  * almost surely it is going to become not obsolete later.
                  * This check must catch it. */) {
        struct astring astr;
 
        /* Only support unchanging requirements until the reachability code
         * can handle it and the tech tree can display changing
         * requirements. */
        ruleset_error(logger, LOG_ERROR,
                      _("Tech \"%s\" has the requirement %s in its"
                        " research_reqs. This requirement may change during"
                        " the game. Changing requirements aren't supported"
                        " yet."),
                      advance_rule_name(padvance),
                      req_to_fstring(preq, &astr));
        astr_free(&astr);
        ok = FALSE;
      }
    } requirement_vector_iterate_end;
 
    if (padvance->bonus_message != NULL) {
      if (!formats_match(padvance->bonus_message, "%s")) {
        ruleset_error(logger, LOG_ERROR,
                      _("Tech \"%s\" bonus message is not format with %%s "
                        "for a bonus tech name."),
                      advance_rule_name(padvance));
        ok = FALSE;
      }
    }
  } advance_re_active_iterate_end;
 
  if (game.default_government == game.government_during_revolution) {
    ruleset_error(logger, LOG_ERROR,
                  _("The government form %s reserved for revolution handling "
                    "has been set as default_government."),
                  government_rule_name(game.government_during_revolution));
    ok = FALSE;
    default_gov_failed = TRUE;
  }
 
  /* Check that all players can have their initial techs */
  nations_re_active_iterate(pnation) {
    int techi;
 
    /* Check global initial techs */
    for (techi = 0; techi < MAX_NUM_TECH_LIST
         && game.rgame.global_init_techs[techi] != A_LAST; techi++) {
      Tech_type_id tech = game.rgame.global_init_techs[techi];
      struct advance *a = valid_advance_by_number(tech);
 
      if (a == NULL) {
        ruleset_error(logger, LOG_ERROR,
                      _("Tech %s does not exist, but is initial "
                        "tech for everyone."),
                      advance_rule_name(advance_by_number(tech)));
        ok = FALSE;
      } else if (advance_by_number(A_NONE) != a->require[AR_ROOT]
          && !nation_has_initial_tech(pnation, a->require[AR_ROOT])) {
        /* Nation has no root_req for tech */
        ruleset_error(logger, LOG_ERROR,
                      _("Tech %s is initial for everyone, but %s has "
                        "no root_req for it."),
                      advance_rule_name(a),
                      nation_rule_name(pnation));
        ok = FALSE;
      }
    }
 
    /* Check national initial techs */
    for (techi = 0;
         techi < MAX_NUM_TECH_LIST && pnation->init_techs[techi] != A_LAST;
         techi++) {
      Tech_type_id tech = pnation->init_techs[techi];
      struct advance *a = valid_advance_by_number(tech);
 
      if (a == NULL) {
        ruleset_error(logger, LOG_ERROR,
                      _("Tech %s does not exist, but is initial tech for %s."),
                      advance_rule_name(advance_by_number(tech)),
                      nation_rule_name(pnation));
        ok = FALSE;
      } else if (advance_by_number(A_NONE) != a->require[AR_ROOT]
          && !nation_has_initial_tech(pnation, a->require[AR_ROOT])) {
        /* Nation has no root_req for tech */
        ruleset_error(logger, LOG_ERROR,
                      _("Tech %s is initial for %s, but they have "
                        "no root_req for it."),
                      advance_rule_name(a),
                      nation_rule_name(pnation));
        ok = FALSE;
      }
    }
 
    /* Check national initial buildings */
    if (nation_barbarian_type(pnation) != NOT_A_BARBARIAN
        && pnation->init_buildings[0] != B_LAST) {
      ruleset_error(logger, LOG_ERROR,
                    _("Nation %s has init_buildings set but as barbarians will "
                      "never get them."), nation_rule_name(pnation));
    }
 
    if (!default_gov_failed && pnation->init_government == game.government_during_revolution) {
      ruleset_error(logger, LOG_ERROR,
                    _("The government form %s reserved for revolution "
                      "handling has been set as initial government for %s."),
                    government_rule_name(game.government_during_revolution),
                    nation_rule_name(pnation));
      ok = FALSE;
    }
  } nations_re_active_iterate_end;
 
  /* Check against unit upgrade loops */
  num_utypes = game.control.num_unit_types;
  unit_type_re_active_iterate(putype) {
    int chain_length = 0;
    const struct unit_type *upgraded = putype;
 
    while (upgraded != NULL && !obsoleted_by_loop) {
      upgraded = upgraded->obsoleted_by;
      chain_length++;
      if (chain_length > num_utypes) {
        ruleset_error(logger, LOG_ERROR,
                      _("There seems to be obsoleted_by loop in update "
                        "chain that starts from %s"),
                      utype_rule_name(putype));
        ok = FALSE;
        obsoleted_by_loop = TRUE;
      }
    }
  } unit_type_re_active_iterate_end;
 
  /* Some unit type properties depend on other unit type properties to work
   * properly. */
  unit_type_re_active_iterate(putype) {
    /* "Spy" is a better "Diplomat". Until all the places that assume that
     * "Diplomat" is set if "Spy" is set is changed this limitation must be
     * kept. */
    if (utype_has_flag(putype, UTYF_SPY)
        && !utype_has_flag(putype, UTYF_DIPLOMAT)) {
      ruleset_error(logger, LOG_ERROR,
                    _("The unit type '%s' has the 'Spy' unit type flag but "
                      "not the 'Diplomat' unit type flag."),
                    utype_rule_name(putype));
      ok = FALSE;
    }
  } unit_type_re_active_iterate_end;
 
  /* Check that unit type fields are in range. */
  unit_type_re_active_iterate(putype) {
    if (putype->paratroopers_range < 0
        || putype->paratroopers_range > UNIT_MAX_PARADROP_RANGE) {
      /* Paradrop range is limited by the network protocol. */
      ruleset_error(logger, LOG_ERROR,
                    _("The paratroopers_range of the unit type '%s' is %d. "
                      "That is out of range. Max range is %d."),
                    utype_rule_name(putype),
                    putype->paratroopers_range, UNIT_MAX_PARADROP_RANGE);
      ok = FALSE;
    }
    /* never fires if game.scenario.prevent_new_cities is TRUE */
    if ((putype->city_size <= 0 || putype->city_size > MAX_CITY_SIZE)
        && utype_is_cityfounder(putype)) {
      ruleset_error(logger, LOG_ERROR,
                    _("Unit type '%s' would build size %d cities. "
                      "City sizes must be from 1 to %d."),
                    utype_rule_name(putype), putype->city_size,
                    MAX_CITY_SIZE);
      ok = FALSE;
    }
  } unit_type_re_active_iterate_end;
 
  memset(&els, 0, sizeof(els));
  els.logger = logger;
 
  /* Check requirement sets against conflicting requirements.
   * For effects check also other sanity in the same iteration */
  if (!iterate_effect_cache(effect_list_sanity_cb, &els)) {
    ok = FALSE;
  }
 
  for (i = 0; req_base_effects[i] != EFT_COUNT; i++) {
    if (!els.base_effects.effect_present[req_base_effects[i]]) {
      ruleset_error(logger, LOG_ERROR,
                    _("There is no base %s effect."),
                    effect_type_name(req_base_effects[i]));
      ok = FALSE;
    }
  }
  for (i = 0; req_base_effects_3_4[i] != EFT_COUNT; i++) {
    if (!els.base_effects.effect_present[req_base_effects_3_4[i]]) {
      const char *ename = effect_type_name(req_base_effects_3_4[i]);
 
      if (compat != nullptr && compat->compat_mode && compat->version < RSFORMAT_3_4) {
        log_deprecation("There is no base %s effect.", ename);
        if (compat->log_cb != nullptr) {
          char buf[512];
 
          fc_snprintf(buf, sizeof(buf), _("Missing base %s effect. Please add one."), ename);
          compat->log_cb(buf);
        }
      } else {
        ruleset_error(logger, LOG_ERROR,
                      _("There is no base %s effect."), ename);
        ok = FALSE;
      }
    }
  }
 
  if (!sanity_check_boolean_effects(logger)) {
    ok = FALSE;
  }
 
  /* Others use requirement vectors */
 
  /* Disasters */
  disaster_type_re_active_iterate(pdis) {
    if (!sanity_check_req_vec(logger, &pdis->reqs, TRUE, -1,
                              disaster_rule_name(pdis))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Disasters have conflicting or invalid requirements!"));
      ok = FALSE;
    }
  } disaster_type_re_active_iterate_end;
 
  /* Goods */
  goods_type_re_active_iterate(pgood) {
    if (!sanity_check_req_vec(logger, &pgood->reqs, TRUE, -1,
                              goods_rule_name(pgood))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Goods have conflicting or invalid requirements!"));
      ok = FALSE;
    }
  } goods_type_re_active_iterate_end;
 
  /* Buildings */
  improvement_re_active_iterate(pimprove) {
    if (!sanity_check_req_vec(logger, &pimprove->reqs, TRUE, -1,
                              improvement_rule_name(pimprove))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Buildings have conflicting or invalid requirements!"));
      ok = FALSE;
    }
    if (!sanity_check_req_vec(logger, &pimprove->obsolete_by, FALSE, -1,
                              improvement_rule_name(pimprove))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Buildings have conflicting or invalid obsolescence req!"));
      ok = FALSE;
    }
  } improvement_re_active_iterate_end;
 
  /* Governments */
  governments_re_active_iterate(pgov) {
    if (!sanity_check_req_vec(logger, &pgov->reqs, TRUE, -1,
                              government_rule_name(pgov))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Governments have conflicting or invalid requirements!"));
      ok = FALSE;
    }
  } governments_re_active_iterate_end;
 
  /* Specialists */
  specialist_type_re_active_iterate(psp) {
    if (!sanity_check_req_vec(logger, &psp->reqs, TRUE, -1,
                              specialist_rule_name(psp))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Specialists have conflicting or invalid requirements!"));
      ok = FALSE;
    }
  } specialist_type_re_active_iterate_end;
 
  /* Extras */
  extra_type_re_active_iterate(pextra) {
    if (!sanity_check_req_vec(logger, &pextra->reqs, TRUE, -1,
                              extra_rule_name(pextra))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Extras have conflicting or invalid requirements!"));
      ok = FALSE;
    }
    if (!sanity_check_req_vec(logger, &pextra->rmreqs, TRUE, -1,
                              extra_rule_name(pextra))) {
      ruleset_error(logger, LOG_ERROR,
                    _("Extras have conflicting or invalid removal requirements!"));
      ok = FALSE;
    }
    if ((requirement_vector_size(&pextra->rmreqs) > 0)
        && !(pextra->rmcauses
             & (ERM_ENTER | ERM_CLEAN | ERM_PILLAGE))) {
      ruleset_error(logger, LOG_WARN,
                    _("Requirements for extra removal defined but not "
                      "a valid remove cause!"));
    }
  } extra_type_re_active_iterate_end;
 
  /* Roads */
  extra_type_by_cause_iterate(EC_ROAD, pextra) {
    struct road_type *proad = extra_road_get(pextra);
 
    extra_type_list_iterate(proad->integrators, iextra) {
      struct road_type *iroad = extra_road_get(iextra);
      int pnbr = road_number(proad);
 
      if (pnbr != road_number(iroad)
          && !BV_ISSET(iroad->integrates, pnbr)) {
        /* We don't support non-symmetric integrator relationships yet. */
        ruleset_error(logger, LOG_ERROR,
                      _("Road '%s' integrates with '%s' but not vice versa!"),
                      extra_rule_name(pextra),
                      extra_rule_name(iextra));
        ok = FALSE;
      }
    } extra_type_list_iterate_end;
  } extra_type_by_cause_iterate_end;
 
  /* City styles */
  for (i = 0; i < game.control.num_city_styles; i++) {
    if (!sanity_check_req_vec(logger, &city_styles[i].reqs, TRUE, -1,
                              city_style_rule_name(i))) {
      ruleset_error(logger, LOG_ERROR,
                    _("City styles have conflicting or invalid requirements!"));
      ok = FALSE;
    }
  }
 
  /* Actions */
  action_iterate(act) {
    struct action *paction = action_by_number(act);
 
    if (!actres_legal_target_kind(paction->result, paction->target_kind)) {
      ruleset_error(logger, LOG_ERROR,
                    _("Action \"%s\": unsupported target kind %s."),
                    action_id_rule_name(act),
                    action_target_kind_name(paction->target_kind));
      ok = FALSE;
    }
 
    if (paction->min_distance < 0) {
      ruleset_error(logger, LOG_ERROR,
                    _("Action %s: negative min distance (%d)."),
                    action_id_rule_name(act), paction->min_distance);
      ok = FALSE;
    }
 
    if (paction->min_distance > ACTION_DISTANCE_LAST_NON_SIGNAL) {
      ruleset_error(logger, LOG_ERROR,
                    _("Action %s: min distance (%d) larger than "
                      "any distance on a map can be (%d)."),
                    action_id_rule_name(act), paction->min_distance,
                    ACTION_DISTANCE_LAST_NON_SIGNAL);
      ok = FALSE;
    }
 
    if (paction->max_distance > ACTION_DISTANCE_MAX) {
      ruleset_error(logger, LOG_ERROR,
                    _("Action %s: max distance is %d. "
                      "A map can't be that big."),
                    action_id_rule_name(act), paction->max_distance);
      ok = FALSE;
    }
 
    if (!action_distance_inside_max(paction, paction->min_distance)) {
      ruleset_error(logger, LOG_ERROR,
                    _("Action %s: min distance is %d but max distance is %d."),
                    action_id_rule_name(act),
                    paction->min_distance, paction->max_distance);
      ok = FALSE;
    }
 
    action_iterate(blocker) {
      if (BV_ISSET(paction->blocked_by, blocker)
          && action_id_get_target_kind(blocker) == ATK_UNIT
          && action_id_get_target_kind(act) != ATK_UNIT) {
        /* Can't find an individual unit target to evaluate the blocking
         * action against. (A tile may have more than one individual
         * unit) */
        ruleset_error(logger, LOG_ERROR,
                      _("The action %s can't block %s."),
                      action_id_rule_name(blocker),
                      action_id_rule_name(act));
        ok = FALSE;
      }
    } action_iterate_end;
 
    action_enabler_list_re_iterate(action_enablers_for_action(act), enabler) {
      if (!sanity_check_req_vec(logger, &(enabler->actor_reqs), TRUE, -1,
                                "Action Enabler Actor Reqs")
          || !sanity_check_req_vec(logger, &(enabler->target_reqs), TRUE, -1,
                                   "Action Enabler Target Reqs")) {
        ruleset_error(logger, LOG_ERROR,
                      _("Action enabler for %s has conflicting or invalid "
                        "requirements!"), action_id_rule_name(act));
        ok = FALSE;
      }
 
      if (action_get_target_kind(enabler_get_action(enabler)) == ATK_SELF) {
        /* Special test for self targeted actions. */
 
        if (requirement_vector_size(&(enabler->target_reqs)) > 0) {
          /* Shouldn't have target requirements since the action doesn't
           * have a target. */
          ruleset_error(logger, LOG_ERROR,
                        _("An action enabler for %s has a target "
                          "requirement vector. %s doesn't have a target."),
                        action_id_rule_name(act),
                        action_id_rule_name(act));
          ok = FALSE;
        }
      }
 
      requirement_vector_iterate(&(enabler->target_reqs), preq) {
        if (preq->source.kind == VUT_DIPLREL
            && preq->range == REQ_RANGE_LOCAL) {
          struct astring astr;
 
          /* A Local DiplRel requirement can be expressed as a requirement
           * in actor_reqs. Demand that it is there. This avoids breaking
           * code that reasons about actions. */
          ruleset_error(logger, LOG_ERROR,
                        _("Action enabler for %s has a local DiplRel "
                          "requirement %s in target_reqs! Please read the "
                          "section \"Requirement vector rules\" in "
                          "doc/README.actions"),
                        action_id_rule_name(act),
                        req_to_fstring(preq, &astr));
          astr_free(&astr);
          ok = FALSE;
        } else if (preq->source.kind == VUT_MAX_DISTANCE_SQ
                   && preq->range == REQ_RANGE_TILE) {
          struct astring astr;
 
          /* A Tile-ranged MaxDistanceSq requirement can be expressed as a
           * requirement in actor_reqs. Demand that it is there. */
          ruleset_error(logger, LOG_ERROR,
                        _("Action enabler for %s has a tile MaxDistanceSq "
                          "requirement %s in target_reqs! Please read the "
                          "section \"Requirement vector rules\" in "
                          "doc/README.actions"),
                        action_id_rule_name(act),
                        req_to_fstring(preq, &astr));
          astr_free(&astr);
          ok = FALSE;
        }
      } requirement_vector_iterate_end;
 
      if (compat == nullptr || !compat->compat_mode
          || compat->version >= RSFORMAT_3_4) {
        /* Support for letting some of the following hard requirements be
         * implicit were retired in Freeciv 3.0. Others were retired later.
         * Make sure that the opposite of each hard action requirement
         * blocks all its action enablers. */
 
        struct req_vec_problem *problem
          = action_enabler_suggest_repair(enabler);
 
        if (problem != nullptr) {
          ruleset_error(logger, LOG_ERROR, "%s", problem->description);
          req_vec_problem_free(problem);
          ok = FALSE;
        }
 
        problem = action_enabler_suggest_improvement(enabler);
        if (problem != nullptr) {
          /* There is a potential for improving this enabler. */
          log_deprecation("%s", problem->description);
          req_vec_problem_free(problem);
        }
      }
    } action_enabler_list_re_iterate_end;
 
    if (BV_ISSET(game.info.diplchance_initial_odds, paction->id)
        /* The action performer, action_dice_roll_initial_odds() and the
         * action probability calculation in action_prob() must probably all
         * be updated to add a new action here. */
        && !(action_has_result_safe(paction, ACTRES_STRIKE_BUILDING)
             || action_has_result_safe(paction, ACTRES_STRIKE_PRODUCTION)
             || action_has_result_safe(paction, ACTRES_SPY_SPREAD_PLAGUE)
             || action_has_result_safe(paction, ACTRES_SPY_POISON)
             || action_has_result_safe(paction, ACTRES_SPY_STEAL_TECH)
             || action_has_result_safe(paction,
                                       ACTRES_SPY_TARGETED_STEAL_TECH)
             || action_has_result_safe(paction, ACTRES_SPY_INCITE_CITY)
             || action_has_result_safe(paction, ACTRES_SPY_SABOTAGE_CITY)
             || action_has_result_safe(paction,
                                       ACTRES_SPY_TARGETED_SABOTAGE_CITY)
             || action_has_result_safe(paction,
                                       ACTRES_SPY_SABOTAGE_CITY_PRODUCTION)
             || action_has_result_safe(paction, ACTRES_SPY_STEAL_GOLD)
             || action_has_result_safe(paction, ACTRES_STEAL_MAPS)
             || action_has_result_safe(paction, ACTRES_SPY_NUKE))) {
      ruleset_error(logger, LOG_ERROR,
                    _("diplchance_initial_odds: \"%s\" not supported."),
                    action_rule_name(paction));
      ok = FALSE;
    }
 
    if (BV_ISSET(paction->sub_results, ACT_SUB_RES_HUT_ENTER)
        && BV_ISSET(paction->sub_results, ACT_SUB_RES_HUT_FRIGHTEN)) {
      ruleset_error(logger, LOG_ERROR,
                    _("%s both enters and frightens a hut at the same time."),
                    action_rule_name(paction));
      ok = FALSE;
    }
  } action_iterate_end;
 
  /* Auto attack */
  {
    struct action_auto_perf *auto_perf;
 
    auto_perf = action_auto_perf_slot_number(ACTION_AUTO_MOVED_ADJ);
 
    action_auto_perf_actions_iterate(auto_perf, act_id) {
      struct action *paction = action_by_number(act_id);
 
      if (!action_has_result(paction, ACTRES_CAPTURE_UNITS)
          && !action_has_result(paction, ACTRES_BOMBARD)
          && !action_has_result(paction, ACTRES_ATTACK)
          && !action_has_result(paction, ACTRES_COLLECT_RANSOM)) {
        /* Only allow removing and changing the order of old auto
         * attack actions for now. Other actions need more testing and
         * fixing of issues caused by a worst case action probability of
         * 0%. */
        ruleset_error(logger, LOG_ERROR,
                      _("auto_attack: %s not supported in"
                        " attack_actions."),
                      action_rule_name(paction));
        ok = FALSE;
      }
    } action_auto_perf_actions_iterate_end;
  }
 
  /* There must be basic city style for each nation style to start with */
  styles_re_active_iterate(pstyle) {
    if (basic_city_style_for_style(pstyle) < 0) {
      ruleset_error(logger, LOG_ERROR,
                    _("There's no basic city style for nation style %s"),
                    style_rule_name(pstyle));
      ok = FALSE;
    }
  } styles_re_active_iterate_end;
 
  /* Music styles */
  music_styles_re_active_iterate(pmus) {
    if (!sanity_check_req_vec(logger, &pmus->reqs, TRUE, -1, "Music Style")) {
      ruleset_error(logger, LOG_ERROR,
                    _("Music Styles have conflicting or invalid requirements!"));
      ok = FALSE;
    }
  } music_styles_re_active_iterate_end;
 
  terrain_re_active_iterate(pterr) {
    terrain_animals_iterate(pterr, panimal) {
      if (!is_native_to_class(utype_class(panimal), pterr, nullptr)) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s has %s as animal to appear, but it's not native to the terrain."),
                      terrain_rule_name(pterr), utype_rule_name(panimal));
        ok = FALSE;
        break;
      }
    } terrain_animals_iterate_end
 
    terrain_resources_iterate(pterr, pres, freq) {
      (void) freq;
      if (!is_extra_caused_by(pres, EC_RESOURCE)) {
        ruleset_error(logger, LOG_ERROR,
                      _("%s has %s as a resource, but it's not a resource extra."),
                      terrain_rule_name(pterr), extra_rule_name(pres));
        ok = FALSE;
      }
    } terrain_resources_iterate_end;
  } terrain_re_active_iterate_end;
 
  /* Check that all unit classes can exist somewhere */
  unit_class_re_active_iterate(pclass) {
    if (!uclass_has_flag(pclass, UCF_BUILD_ANYWHERE)) {
      bool can_exist = FALSE;
 
      terrain_re_active_iterate(pterr) {
        if (BV_ISSET(pterr->native_to, uclass_index(pclass))) {
          can_exist = TRUE;
          break;
        }
      } terrain_re_active_iterate_end;
 
      if (!can_exist) {
        extra_type_re_active_iterate(pextra) {
          if (BV_ISSET(pextra->native_to, uclass_index(pclass))
              && extra_has_flag(pextra, EF_NATIVE_TILE)) {
            can_exist = TRUE;
            break;
          }
        } extra_type_re_active_iterate_end;
      }
 
      if (!can_exist) {
        ruleset_error(logger, LOG_ERROR,
                      _("Unit class %s cannot exist anywhere."),
                      uclass_rule_name(pclass));
        ok = FALSE;
      }
    }
  } unit_class_re_active_iterate_end;
 
  achievements_re_active_iterate(pach) {
    if (!pach->unique && pach->cons_msg == NULL) {
      ruleset_error(logger, LOG_ERROR,
                    _("Achievement %s has no message for consecutive gainers though "
                      "it's possible to be gained by multiple players"),
                    achievement_rule_name(pach));
      ok = FALSE;
    }
  } achievements_re_active_iterate_end;
 
  if (game.server.ruledit.embedded_nations != NULL) {
    int nati;
 
    for (nati = 0; nati < game.server.ruledit.embedded_nations_count; nati++) {
      struct nation_type *pnat
        = nation_by_rule_name(game.server.ruledit.embedded_nations[nati]);
 
      if (pnat == NULL) {
        ruleset_error(logger, LOG_ERROR,
                      _("There's nation %s listed in embedded nations, but there's "
                        "no such nation."),
                      game.server.ruledit.embedded_nations[nati]);
        ok = FALSE;
      }
    }
  }
 
  if (ok) {
    ok = rs_common_units(logger);
  }
  if (ok) {
    ok = rs_barbarian_units(logger);
  }
  if (ok) {
    ok = rs_buildings(logger);
  }
 
  return ok;
}
 
/**********************************************************************/
bool autoadjust_ruleset_data(void)
{
  bool ok = TRUE;
 
  extra_type_by_cause_iterate(EC_RESOURCE, pextra) {
    extra_type_by_cause_iterate(EC_RESOURCE, pextra2) {
      if (pextra != pextra2) {
        int idx = extra_index(pextra2);
 
        if (!BV_ISSET(pextra->conflicts, idx)) {
          log_debug("Autoconflicting resource %s with %s",
                    extra_rule_name(pextra), extra_rule_name(pextra2));
          BV_SET(pextra->conflicts, extra_index(pextra2));
        }
      }
    } extra_type_by_cause_iterate_end;
  } extra_type_by_cause_iterate_end;
 
  /* Hard coded action blocking. */
  {
    const struct {
      const enum action_result blocked;
      const enum action_result blocker;
    } must_block[] = {
      /* Hard code that Help Wonder blocks Disband Unit Recover. This must be done
       * because caravan_shields makes it possible to avoid the
       * consequences of choosing to do Disband Unit Recover rather than having it
       * do Help Wonder.
       *
       * Explanation: Disband Unit Recover adds 50% of the shields used to produce
       * the unit to the production of the city where it is located. Help
       * Wonder adds 100%. If a unit that can do Help Wonder is disbanded with
       * production recovery in a city and the production later is changed
       * to something that can receive help from Help Wonder the remaining 50%
       * of the shields are added. This can be done because the city remembers
       * them in caravan_shields.
       *
       * If a unit that can do Help Wonder intentionally is disbanded with recovery
       * rather than making it do Help Wonder its shields will still be
       * remembered. The target city that got 50% of the shields can
       * therefore get 100% of them by changing its production. This trick
       * makes the ability to select Disband Unit Recover when Help Wonder is legal
       * pointless. */
      { ACTRES_DISBAND_UNIT_RECOVER, ACTRES_HELP_WONDER },
 
      /* Allowing regular disband when ACTION_HELP_WONDER or
       * ACTION_DISBAND_UNIT_RECOVER is legal while ACTION_HELP_WONDER always
       * blocks ACTION_DISBAND_UNIT_RECOVER doesn't work well with the force_*
       * semantics. Should move to the ruleset once it has blocked_by
       * semantics. */
      { ACTRES_DISBAND_UNIT, ACTRES_HELP_WONDER },
      { ACTRES_DISBAND_UNIT, ACTRES_DISBAND_UNIT_RECOVER },
 
      /* Hard code that the ability to perform a regular attack blocks city
       * conquest. Is redundant as long as the requirement that the target
       * tile has no units remains hard coded. Kept "just in case" that
       * changes. */
      { ACTRES_CONQUER_CITY, ACTRES_ATTACK },
 
      /* Hard code that the ability to perform a regular attack blocks
       * extras conquest. Is redundant as long as the requirement that the
       * target tile has no non-allied units remains hard coded. Kept "just
       * in case" that changes. */
      { ACTRES_CONQUER_EXTRAS, ACTRES_ATTACK },
 
      /* Hard code that the ability to enter or frighten a hut blocks
       * regular disembarking. */
      { ACTRES_TRANSPORT_DISEMBARK, ACTRES_CONQUER_EXTRAS },
      { ACTRES_TRANSPORT_DISEMBARK, ACTRES_HUT_ENTER },
      { ACTRES_TRANSPORT_DISEMBARK, ACTRES_HUT_FRIGHTEN },
    };
 
    int i;
 
    for (i = 0; i < ARRAY_SIZE(must_block); i++) {
      enum action_result blocked_result = must_block[i].blocked;
      enum action_result blocker_result = must_block[i].blocker;
 
      action_by_result_iterate(blocked, blocked_result) {
        action_by_result_iterate(blocker, blocker_result) {
          if (!action_would_be_blocked_by(blocked, blocker)) {
            log_verbose("Autoblocking %s with %s",
                        action_rule_name(blocked),
                        action_rule_name(blocker));
            BV_SET(blocked->blocked_by, action_id(blocker));
          }
        } action_by_result_iterate_end;
      } action_by_result_iterate_end;
    }
  }
 
  return ok;
}
 
/**********************************************************************/
bool autolock_settings(void)
{
  bool ok = TRUE;
 
  if (num_role_units(L_BARBARIAN) == 0) {
    struct setting *pset = setting_by_name("barbarians");
 
    log_normal(_("Disabling 'barbarians' setting for lack of suitable "
                 "unit types."));
    setting_ruleset_lock_clear(pset);
    if (!setting_enum_set(pset, "DISABLED", NULL, NULL, 0)) {
      ok = FALSE;
    }
    setting_ruleset_lock_set(pset);
  }
 
  return ok;
}