living.c

Go to the documentation of this file.

/*
 * Crossfire -- cooperative multi-player graphical RPG and adventure game
 *
 * Copyright (c) 1999-2014 Mark Wedel and the Crossfire Development Team
 * Copyright (c) 1992 Frank Tore Johansen
 *
 * Crossfire is free software and comes with ABSOLUTELY NO WARRANTY. You are
 * welcome to redistribute it under certain conditions. For details, please
 * see COPYING and LICENSE.
 *
 * The authors can be reached via e-mail at <crossfire@metalforge.org>.
 */

#include "global.h"

#include <assert.h>
#include <ctype.h>
#include <stdlib.h>
#include <string.h>

#include "sproto.h"
#include "living.h"

static float get_con_bonus(int stat);
static float get_sp_bonus(int stat);
static float get_grace_bonus(int stat);
static size_t get_index(int stat, size_t max_index);

#define ADD_EXP(exptotal, exp) { exptotal += exp; if (exptotal > MAX_EXPERIENCE) exptotal = MAX_EXPERIENCE; }

#define INT_FEAR_BONUS      0
#define INT_TURN_BONUS      1
#define INT_CLERIC_CHANCE   2
#define INT_LEARN_SPELL     3
#define INT_CHA_BONUS       4
#define INT_DEX_BONUS       5
#define INT_DAM_BONUS       6
#define INT_THAC0_BONUS     7
#define INT_WEIGHT_LIMIT    8
#define NUM_INT_BONUSES     9

static int *int_bonuses[NUM_INT_BONUSES];

static const char *int_bonus_names[NUM_INT_BONUSES] = {
    "cha_fear_bonus", "wis_turn_bonus", "wis_cleric_chance", "int_wis_learn_spell",
    "cha_shop_bonus", "dex_bonus", "str_damage_bonus", "str_hit_bonus",
    "str_weight_limit",
};

#define FLOAT_CON_BONUS     0
#define FLOAT_DEX_BONUS     1
#define FLOAT_SP_BONUS      2
#define FLOAT_GRACE_BONUS   3
#define NUM_FLOAT_BONUSES   4
static float *float_bonuses[NUM_FLOAT_BONUSES];
static const char *float_bonus_names[NUM_FLOAT_BONUSES] = {
    "con_hp_bonus", "dex_speed_bonus", "pow_int_sp_bonus", "wis_pow_grace_bonus"
};

/*
 * Since this is nowhere defined ...
 * Both come in handy at least in function add_exp()
 */
#define MAX_EXPERIENCE levels[settings.max_level]

extern int64_t *levels;

#define MAX_SAVE_LEVEL 110

static const int savethrow[MAX_SAVE_LEVEL+1] = {
    18,
    18, 17, 16, 15, 14, 14, 13, 13, 12, 12,
    12, 11, 11, 11, 11, 10, 10, 10, 10, 9,
    9, 9, 9, 9, 8, 8, 8, 8, 8, 8,
    7, 7, 7, 7, 7, 7, 7, 6, 6, 6,
    6, 6, 6, 6, 6, 5, 5, 5, 5, 5,
    5, 5, 5, 5, 4, 4, 4, 4, 4, 4,
    4, 4, 4, 4, 3, 3, 3, 3, 3, 3,
    3, 3, 3, 3, 3, 2, 2, 2, 2, 2,
    2, 2, 2, 2, 2, 2, 2, 2, 2, 1,
    1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
    1, 1, 1, 1, 1, 1, 1, 1, 1, 1
};

const char *const attacks[NROFATTACKS] = {
    "physical", "magical", "fire", "electricity", "cold",
    "confusion", "acid", "drain", "weaponmagic", "ghosthit",
    "poison", "slow", "paralyze", "turn undead", "fear",
    "cancellation", "depletion", "death", "chaos", "counterspell",
    "god power", "holy power", "blinding", "", "life stealing",
    "disease"
};

static const char *const drain_msg[NUM_STATS] = {
    "Oh no! You are weakened!",
    "You're feeling clumsy!",
    "Your health deteriorates!",
    "You suddenly begin to lose your memory!",
    "Your face gets distorted!",
    "Watch out, your mind is going!",
    "Your spirit feels drained!"
};

const char *const restore_msg[NUM_STATS] = {
    "You feel your strength return.",
    "You feel your agility return.",
    "You feel your health return.",
    "You feel your wisdom return.",
    "You feel your charisma return.",
    "You feel your memory return.",
    "You feel your spirits return."
};

const char *const gain_msg[NUM_STATS] = {
    "You feel stronger.",
    "You feel more agile.",
    "You feel healthy.",
    "You feel wiser.",
    "You seem to look better.",
    "You feel smarter.",
    "You feel more potent."
};

const char *const lose_msg[NUM_STATS] = {
    "You feel weaker!",
    "You feel clumsy!",
    "You feel less healthy!",
    "You lose some of your memory!",
    "You look ugly!",
    "You feel stupid!",
    "You feel less potent!"
};

const char *const statname[NUM_STATS] = {
    "strength",
    "dexterity",
    "constitution",
    "wisdom",
    "charisma",
    "intelligence",
    "power"
};

const char *const short_stat_name[NUM_STATS] = {
    "Str",
    "Dex",
    "Con",
    "Wis",
    "Cha",
    "Int",
    "Pow"
};

void set_attr_value(living *stats, int attr, int8_t value) {
    switch (attr) {
    case STRENGTH:
        stats->Str = value;
        break;

    case DEXTERITY:
        stats->Dex = value;
        break;

    case CONSTITUTION:
        stats->Con = value;
        break;

    case WISDOM:
        stats->Wis = value;
        break;

    case POWER:
        stats->Pow = value;
        break;

    case CHARISMA:
        stats->Cha = value;
        break;

    case INTELLIGENCE:
        stats->Int = value;
        break;
    }
}

void change_attr_value(living *stats, int attr, int8_t value) {
    if (value == 0)
        return;
    switch (attr) {
    case STRENGTH:
        stats->Str += value;
        break;

    case DEXTERITY:
        stats->Dex += value;
        break;

    case CONSTITUTION:
        stats->Con += value;
        break;

    case WISDOM:
        stats->Wis += value;
        break;

    case POWER:
        stats->Pow += value;
        break;

    case CHARISMA:
        stats->Cha += value;
            break;

    case INTELLIGENCE:
        stats->Int += value;
        break;

    default:
        LOG(llevError, "Invalid attribute in change_attr_value: %d\n", attr);
    }
}

int8_t get_attr_value(const living *stats, int attr) {
    switch (attr) {
    case STRENGTH:
        return(stats->Str);

    case DEXTERITY:
        return(stats->Dex);

    case CONSTITUTION:
        return(stats->Con);

    case WISDOM:
        return(stats->Wis);

    case CHARISMA:
        return(stats->Cha);

    case INTELLIGENCE:
        return(stats->Int);

    case POWER:
        return(stats->Pow);
    }
    return 0;
}

void check_stat_bounds(living *stats, int8_t min_stat, int8_t max_stat) {
    int i, v;
    for (i = 0; i < NUM_STATS; i++)
        if ((v = get_attr_value(stats, i)) > max_stat)
            set_attr_value(stats, i, max_stat);
        else if (v < min_stat)
            set_attr_value(stats, i, min_stat);
}


#define DIFF_MSG(flag, subtype1, subtype2, msg1, msg2) \
    draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, (flag > 0) ? subtype1 : subtype2, (flag > 0) ? msg1 : msg2);

int change_abil(object *op, object *tmp) {
    int flag = QUERY_FLAG(tmp, FLAG_APPLIED) ? 1 : -1, i, j, success = 0;
    object refop;
    int potion_max = 0;

    /* remember what object was like before it was changed.  note that
     * refop is a local copy of op only to be used for detecting changes
    * found by fix_object.  refop is not a real object
     */
    memcpy(&refop, op, sizeof(object));

    if (op->type == PLAYER) {
        if (tmp->type == POTION) {
            potion_max = 1;
            for (j = 0; j < NUM_STATS; j++) {
                int nstat, ostat;

                ostat = get_attr_value(&(op->contr->orig_stats), j);
                i = get_attr_value(&(tmp->stats), j);

                /* nstat is what the stat will be after use of the potion */
                nstat = flag*i+ostat;

                /* Do some bounds checking.  While I don't think any
                 * potions do so right now, there is the potential for potions
                 * that adjust that stat by more than one point, so we need
                 * to allow for that.
                 */
                if (nstat < 1 && i*flag < 0)
                    nstat = 1;
                else if (nstat > 20+get_attr_value(&(op->arch->clone.stats), j)) {
                    nstat = 20+get_attr_value(&(op->arch->clone.stats), j);
                }
                if (nstat != ostat) {
                    set_attr_value(&(op->contr->orig_stats), j, nstat);
                    potion_max = 0;
                } else if (i) {
                    /* potion is useless - player has already hit the natural maximum */
                    potion_max = 1;
                }
            }
            /* This section of code ups the characters normal stats also.  I am not
             * sure if this is strictly necessary, being that fix_object probably
             * recalculates this anyway.
             */
            for (j = 0; j < NUM_STATS; j++)
                change_attr_value(&(op->stats), j, flag*get_attr_value(&(tmp->stats), j));
            check_stat_bounds(&(op->stats), MIN_STAT, settings.max_stat);
        } /* end of potion handling code */
    }

    /* reset attributes that fix_object doesn't reset since it doesn't search
     * everything to set
     */
    if (flag == -1) {
        op->attacktype &= ~tmp->attacktype;
        op->path_attuned &= ~tmp->path_attuned;
        op->path_repelled &= ~tmp->path_repelled;
        op->path_denied &= ~tmp->path_denied;
        /* Presuming here that creatures only have move_type,
         * and not the other move_ fields.
         */
        op->move_type &= ~tmp->move_type;
    }

    /* call fix_object since op object could have whatever attribute due
     * to multiple items.  if fix_object always has to be called after
     * change_ability then might as well call it from here
     */
    fix_object(op);

    /* Fix player won't add the bows ability to the player, so don't
     * print out message if this is a bow.
     */
    if (tmp->attacktype&AT_CONFUSION && tmp->type != BOW) {
        success = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_ATTACKTYPE_GAIN, MSG_TYPE_ATTRIBUTE_ATTACKTYPE_LOSS,
                 "Your hands begin to glow red.",
                 "Your hands stop glowing red.");
    }
    if (QUERY_FLAG(op, FLAG_LIFESAVE) != QUERY_FLAG(&refop, FLAG_LIFESAVE)) {
        success = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_PROTECTION_GAIN, MSG_TYPE_ATTRIBUTE_PROTECTION_LOSS,
                 "You feel very protected.",
                 "You don't feel protected anymore.");
    }
    if (QUERY_FLAG(op, FLAG_REFL_MISSILE) != QUERY_FLAG(&refop, FLAG_REFL_MISSILE)) {
        success = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_PROTECTION_GAIN, MSG_TYPE_ATTRIBUTE_PROTECTION_LOSS,
                 "A magic force shimmers around you.",
                 "The magic force fades away.");
    }
    if (QUERY_FLAG(op, FLAG_REFL_SPELL) != QUERY_FLAG(&refop, FLAG_REFL_SPELL)) {
        success = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_PROTECTION_GAIN, MSG_TYPE_ATTRIBUTE_PROTECTION_LOSS,
                 "You feel more safe now, somehow.",
                 "Suddenly you feel less safe, somehow.");
    }
    /* movement type has changed.  We don't care about cases where
     * user has multiple items giving the same type appled like we
     * used to - that is more work than what we gain, plus messages
     * can be misleading (a little higher could be miscontrued from
     * from fly high)
     */
    if (tmp->move_type && op->move_type != refop.move_type) {
        success = 1;

        /* MOVE_FLY_HIGH trumps MOVE_FLY_LOW - changing your move_fly_low
         * status doesn't make a difference if you are flying high
         */
        if (tmp->move_type&MOVE_FLY_LOW && !(op->move_type&MOVE_FLY_HIGH)) {
            DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_MOVE, MSG_TYPE_ATTRIBUTE_MOVE,
                     "You start to float in the air!",
                     "You float down to the ground.");
        }

        if (tmp->move_type&MOVE_FLY_HIGH) {
            /* double conditional - second case covers if you have move_fly_low -
             * in that case, you don't actually land
             */
            DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_MOVE, MSG_TYPE_ATTRIBUTE_MOVE,
                     "You soar into the air!.",
                     (op->move_type&MOVE_FLY_LOW ? "You glide closer to the ground.":
                      "You float down to the ground."));
        }
        if (tmp->move_type&MOVE_SWIM)
            DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_MOVE, MSG_TYPE_ATTRIBUTE_MOVE,
                     "You feel ready for a swim",
                     "You no longer feel like swimming");

        /* Changing move status may mean you are affected by things you weren't before */
        object_check_move_on(op, op);
    }

    /* becoming UNDEAD... a special treatment for this flag. Only those not
     * originally undead may change their status
     */
    if (!QUERY_FLAG(&op->arch->clone, FLAG_UNDEAD))
        if (QUERY_FLAG(op, FLAG_UNDEAD) != QUERY_FLAG(&refop, FLAG_UNDEAD)) {
            success = 1;
            if (flag > 0) {
                if (op->race)
                    free_string(op->race);
                op->race = add_string("undead");
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_RACE, "Your lifeforce drains away!");
            } else {
                if (op->race)
                    free_string(op->race);
                if (op->arch->clone.race)
                    op->race = add_string(op->arch->clone.race);
                else
                    op->race = NULL;
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_RACE, "Your lifeforce returns!");
            }
        }

    if (QUERY_FLAG(op, FLAG_STEALTH) != QUERY_FLAG(&refop, FLAG_STEALTH)) {
        success = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_START, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_END,
                 "You walk more quietly.",
                 "You walk more noisily.");
    }
    if (QUERY_FLAG(op, FLAG_MAKE_INVIS) != QUERY_FLAG(&refop, FLAG_MAKE_INVIS)) {
        success = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_START, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_END,
                 "You become transparent.",
                 "You can see yourself.");
    }
    /* blinded you can tell if more blinded since blinded player has minimal
     * vision
     */
    if (QUERY_FLAG(tmp, FLAG_BLIND)) {
        success = 1;
        if (flag > 0) {
            if (QUERY_FLAG(op, FLAG_WIZ))
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_BAD_EFFECT_START, "Your mortal self is blinded.");
            else {
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_BAD_EFFECT_START, "You are blinded.");
                SET_FLAG(op, FLAG_BLIND);
                if (op->type == PLAYER)
                    op->contr->do_los = 1;
            }
        } else {
            if (QUERY_FLAG(op, FLAG_WIZ))
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_BAD_EFFECT_END, "Your mortal self can now see again.");
            else {
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_BAD_EFFECT_END, "Your vision returns.");
                CLEAR_FLAG(op, FLAG_BLIND);
                if (op->type == PLAYER)
                    op->contr->do_los = 1;
            }
        }
    }

    if (QUERY_FLAG(op, FLAG_SEE_IN_DARK) != QUERY_FLAG(&refop, FLAG_SEE_IN_DARK)) {
        success = 1;
        if (op->type == PLAYER)
            op->contr->do_los = 1;
        DIFF_MSG(flag, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_START, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_END,
                 "Your vision is better in the dark.",
                 "You see less well in the dark.");
    }

    if (QUERY_FLAG(op, FLAG_XRAYS) != QUERY_FLAG(&refop, FLAG_XRAYS)) {
        success = 1;
        if (flag > 0) {
            if (QUERY_FLAG(op, FLAG_WIZ))
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_START, "Your vision becomes a little clearer.");
            else {
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_START, "Everything becomes transparent.");
                if (op->type == PLAYER)
                    op->contr->do_los = 1;
            }
        } else {
            if (QUERY_FLAG(op, FLAG_WIZ))
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_END, "Your vision becomes a bit out of focus.");
            else {
                draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_GOOD_EFFECT_END, "Everything suddenly looks very solid.");
                if (op->type == PLAYER)
                    op->contr->do_los = 1;
            }
        }
    }

    if (tmp->stats.luck) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.luck, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel more lucky.",
                 "You feel less lucky.");
    }

    if (tmp->stats.hp && op->type == PLAYER) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.hp, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel much more healthy!",
                 "You feel much less healthy!");
    }

    if (tmp->stats.sp && op->type == PLAYER && tmp->type != SKILL) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.sp, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel one with the powers of magic!",
                 "You suddenly feel very mundane.");
    }

    /* for the future when artifacts set this -b.t. */
    if (tmp->stats.grace && op->type == PLAYER) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.grace, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel closer to your god!",
                 "You suddenly feel less holy.");
    }

    if (tmp->stats.wc && op->type == PLAYER) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.wc, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel more confident in combat.",
                 "You feel less confident in combat.");
    }

    if (tmp->stats.ac && op->type == PLAYER) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.ac, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel more confident in your dodging skills.",
                 "You feel less confident in your dodging skills.");
    }

    if (tmp->stats.exp && tmp->type != SKILL && op->type == PLAYER) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.exp, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel like you're moving faster.",
                 "You feel like you're moving more slowly.");
    }

    if (tmp->stats.food && op->type == PLAYER) {
        success = 1;
        DIFF_MSG(flag*tmp->stats.food, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS,
                 "You feel your digestion slowing down.",
                 "You feel your digestion speeding up.");
    }

    /* Messages for changed resistance */
    for (i = 0; i < NROFATTACKS; i++) {
        if (i == ATNR_PHYSICAL)
            continue; /* Don't display about armour */

        if (op->resist[i] != refop.resist[i]) {
            success = 1;
            if (op->resist[i] > refop.resist[i])
                draw_ext_info_format(NDI_UNIQUE|NDI_BLUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_PROTECTION_GAIN,
                                     "Your resistance to %s rises to %d%%.",
                                     change_resist_msg[i], op->resist[i]);
            else
                draw_ext_info_format(NDI_UNIQUE|NDI_BLUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_PROTECTION_LOSS,
                                     "Your resistance to %s drops to %d%%.",
                                     change_resist_msg[i], op->resist[i]);
        }
    }

    if (!potion_max) {
        for (j = 0; j < NUM_STATS; j++) {
            if ((i = get_attr_value(&(tmp->stats), j)) != 0) {
                success = 1;
                DIFF_MSG(i*flag, MSG_TYPE_ATTRIBUTE_STAT_GAIN, MSG_TYPE_ATTRIBUTE_STAT_LOSS, gain_msg[j], lose_msg[j]);
            }
        }
    }
    return success;
}

void drain_stat(object *op) {
    drain_specific_stat(op, RANDOM()%NUM_STATS);
}

void drain_specific_stat(object *op, int deplete_stats) {
    object *tmp;
    archetype *at;

    at = find_archetype(ARCH_DEPLETION);
    if (!at) {
        return;
    } else {
        tmp = arch_present_in_ob(at, op);
        if (!tmp) {
            tmp = arch_to_object(at);
            tmp = object_insert_in_ob(tmp, op);
            SET_FLAG(tmp, FLAG_APPLIED);
        }
    }

    draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_STAT_LOSS, drain_msg[deplete_stats]);
    change_attr_value(&tmp->stats, deplete_stats, -1);
    fix_object(op);
}

int remove_depletion(object *op, int level) {
    object *depl;
    archetype *at;
    int i, count = 0;

    if ((at = find_archetype(ARCH_DEPLETION)) == NULL) {
        return 0;
    }

    depl = arch_present_in_ob(at, op);

    if (depl == NULL)
        return 0;

    if (level != -1 && level < op->level)
        return 0;

    for (i = 0; i < NUM_STATS; i++) {
        if (get_attr_value(&depl->stats, i)) {
            count++;
            draw_ext_info(NDI_UNIQUE, 0, op, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_STAT_GAIN, restore_msg[i]);
        }
    }

    object_remove(depl);
    object_free_drop_inventory(depl);
    fix_object(op);

    return (count == 0) ? 0 : 1;
}

void change_luck(object *op, int value) {
    object *tmp;
    archetype *at;
    int new_luck;

    at = find_archetype("luck");
    if (!at)
        ;
    else {
        tmp = arch_present_in_ob(at, op);
        if (!tmp) {
            if (!value)
                return;
            tmp = arch_to_object(at);
            tmp = object_insert_in_ob(tmp, op);
            SET_FLAG(tmp, FLAG_APPLIED);
        }
        if (value) {
            /* Limit the luck value of the bad luck object to +/-100. This
            * (arbitrary) value prevents overflows (both in the bad luck object and
            * in op itself).
            */
            new_luck = tmp->stats.luck+value;
            if (new_luck >= -100 && new_luck <= 100) {
                op->stats.luck += value;
                tmp->stats.luck = new_luck;
            }
        } else {
            if (!tmp->stats.luck) {
                return;
            }
            /* Randomly change the players luck.  Basically, we move it
            * back neutral (if greater>0, subtract, otherwise add)
            */
            if (RANDOM()%(FABS(tmp->stats.luck)) >= RANDOM()%30) {
                int diff = tmp->stats.luck > 0 ? -1 : 1;
                op->stats.luck += diff;
                tmp->stats.luck += diff;
            }
        }
    }
}

void remove_statbonus(object *op) {
    op->stats.Str -= op->arch->clone.stats.Str;
    op->stats.Dex -= op->arch->clone.stats.Dex;
    op->stats.Con -= op->arch->clone.stats.Con;
    op->stats.Wis -= op->arch->clone.stats.Wis;
    op->stats.Pow -= op->arch->clone.stats.Pow;
    op->stats.Cha -= op->arch->clone.stats.Cha;
    op->stats.Int -= op->arch->clone.stats.Int;
    op->contr->orig_stats.Str -= op->arch->clone.stats.Str;
    op->contr->orig_stats.Dex -= op->arch->clone.stats.Dex;
    op->contr->orig_stats.Con -= op->arch->clone.stats.Con;
    op->contr->orig_stats.Wis -= op->arch->clone.stats.Wis;
    op->contr->orig_stats.Pow -= op->arch->clone.stats.Pow;
    op->contr->orig_stats.Cha -= op->arch->clone.stats.Cha;
    op->contr->orig_stats.Int -= op->arch->clone.stats.Int;
}

void add_statbonus(object *op) {
    op->stats.Str += op->arch->clone.stats.Str;
    op->stats.Dex += op->arch->clone.stats.Dex;
    op->stats.Con += op->arch->clone.stats.Con;
    op->stats.Wis += op->arch->clone.stats.Wis;
    op->stats.Pow += op->arch->clone.stats.Pow;
    op->stats.Cha += op->arch->clone.stats.Cha;
    op->stats.Int += op->arch->clone.stats.Int;
    op->contr->orig_stats.Str += op->arch->clone.stats.Str;
    op->contr->orig_stats.Dex += op->arch->clone.stats.Dex;
    op->contr->orig_stats.Con += op->arch->clone.stats.Con;
    op->contr->orig_stats.Wis += op->arch->clone.stats.Wis;
    op->contr->orig_stats.Pow += op->arch->clone.stats.Pow;
    op->contr->orig_stats.Cha += op->arch->clone.stats.Cha;
    op->contr->orig_stats.Int += op->arch->clone.stats.Int;
}

static void fix_player(object *op, int *ac, int *wc, const object *grace_obj, const object *mana_obj, const object *wc_obj, int weapon_speed, float added_speed)
{
    int pl_level, i;
    float character_load = 0.0, maxhp, tmpf;

    if (op->type != PLAYER)
        return;

    check_stat_bounds(&(op->stats), MIN_STAT, settings.max_stat);
    pl_level = op->level;

    if (pl_level < 1)
        pl_level = 1; /* safety, we should always get 1 levels worth of hp! */

    /*
     * We store maxhp as a float to hold any fractional hp bonuses,
     * (eg, 2.5 con bonus).  While it may seem simpler to just
     * do a get_con_bonus() * min(level,10), there is also a 1 hp/level
     * minimum (including bonus), se we have to do the logic on a
     * level basis.
     */
    maxhp = 0.0;
    for (i = 1, op->stats.maxhp = 0; i <= pl_level && i <= 10; i++) {

        tmpf = op->contr->levhp[i]+get_con_bonus(op->stats.Con);

        /* always get at least 1 hp/level */
        if (tmpf < 1.0) tmpf = 1.0;

        maxhp += tmpf;
    }

    /* Add 0.5 so that this rounds normally - the cast just drops the
     * fraction, so 1.5 becomes 1.
     */
    op->stats.maxhp = (int)(maxhp + 0.5);
    if (op->level > 10)
        op->stats.maxhp += 2 * (op->level - 10);

    op->stats.maxhp += op->arch->clone.stats.maxhp;

    if (op->stats.hp > op->stats.maxhp)
        op->stats.hp = op->stats.maxhp;

    /* Sp gain is controlled by the level of the player's
     * relevant experience object (mana_obj, see above) */

    /* set maxsp */
    if (!mana_obj || !mana_obj->level) {
        op->stats.maxsp = 1;
    } else {
        float sp_tmp = 0.0, mana_bonus;
        int mana_lvl_max;

        mana_lvl_max = (mana_obj->level >10 ? 10: mana_obj->level);
        mana_bonus = (2.0*get_sp_bonus(op->stats.Pow)+get_sp_bonus(op->stats.Int)) / 3.0;

        for (i = 1; i <= mana_lvl_max; i++) {
            float stmp;

            stmp =  op->contr->levsp[i] + mana_bonus;

            /* Got some extra bonus at first level */
            if (i == 1) stmp += mana_bonus;

            if (stmp < 1.0)
                stmp = 1.0;

            sp_tmp += stmp;
        }

        op->stats.maxsp = (int)sp_tmp+op->arch->clone.stats.maxsp;

        if (mana_obj->level > 10)
            op->stats.maxsp += 2 * (mana_obj->level - 10);
    }

    /* Characters can get their sp supercharged via rune of transferrance */
    if (op->stats.sp > op->stats.maxsp*2)
        op->stats.sp = op->stats.maxsp*2;

    /* set maxgrace, notice 3-4 lines below it depends on both Wis and Pow */
    if (!grace_obj || !grace_obj->level) {
        op->stats.maxgrace = 1;
    } else {
        /* store grace in a float - this way, the divisions below don't create
         * big jumps when you go from level to level - with int's, it then
         * becomes big jumps when the sums of the bonuses jump to the next
         * step of 8 - with floats, even fractional ones are useful.
         */
        float sp_tmp = 0.0, grace_bonus;

        grace_bonus = (get_grace_bonus(op->stats.Pow)+2.0*get_grace_bonus(op->stats.Wis)) / 3.0;

        for (i = 1; i <= grace_obj->level && i <= 10; i++) {
            float grace_tmp = op->contr->levgrace[i] + grace_bonus;

            /* Got some extra bonus at first level */
            if (i == 1)
                grace_tmp += grace_bonus;

            if (grace_tmp < 1.0)
                grace_tmp = 1.0;
            sp_tmp += grace_tmp;
        }
        op->stats.maxgrace = (int)sp_tmp+op->arch->clone.stats.maxgrace;

        /* two grace points per level after 11 */
        if (grace_obj->level > 10)
            op->stats.maxgrace += 2 * (grace_obj->level - 10);
    }
    /* No limit on grace vs maxgrace */

    if (op->contr->braced) {
        (*ac) += 2;
        (*wc) += 4;
    } else
        (*ac) -= get_dex_bonus(op->stats.Dex);

    /* In new exp/skills system, wc bonuses are related to
     * the players level in a relevant exp object (wc_obj)
     * not the general player level -b.t.
     * I changed this slightly so that wc bonuses are better
     * than before. This is to balance out the fact that
     * the player no longer gets a personal weapon w/ 1
     * improvement every level, now its fighterlevel/5. So
     * we give the player a bonus here in wc and dam
     * to make up for the change. Note that I left the
     * monster bonus the same as before. -b.t.
     */

    if (wc_obj && wc_obj->level >= 1) {
        const  char *wc_in = object_get_value(wc_obj, "wc_increase_rate");
        int wc_increase_rate;

        wc_increase_rate = wc_in?atoi(wc_in):5;
        assert(wc_increase_rate != 0);

        (*wc) -= get_thaco_bonus(op->stats.Str);
        (*wc) -= (wc_obj->level-1)/wc_increase_rate;
        op->stats.dam += (wc_obj->level-1)/4;
    } else {
        (*wc) -= (((op->level-1)/5)+get_thaco_bonus(op->stats.Str));
    }
    op->stats.dam += get_dam_bonus(op->stats.Str);

    if (op->stats.dam < 1)
        op->stats.dam = 1;

    op->speed = MAX_PLAYER_SPEED+get_speed_bonus(op->stats.Dex);

    if (settings.search_items && op->contr->search_str[0])
        op->speed -= 0.25;

    if (op->attacktype == 0)
        op->attacktype = op->arch->clone.attacktype;


    /* First block is for encumbrance of the player */

    /* The check for FREE_PLAYER_LOAD_PERCENT < 1.0 is really a safety.  One would
     * think that it should never be the case if that is set to 1.0, that carrying
     * would be above the limit.  But it could be if character is weakened and
     * was otherwise at limit.  Without that safety, could get divide by zeros.
     */
    if (op->carrying > (get_weight_limit(op->stats.Str)*FREE_PLAYER_LOAD_PERCENT)
    && (FREE_PLAYER_LOAD_PERCENT < 1.0)) {
        int extra_weight = op->carrying-get_weight_limit(op->stats.Str)*FREE_PLAYER_LOAD_PERCENT;

        character_load = (float)extra_weight/(float)(get_weight_limit(op->stats.Str)*(1.0-FREE_PLAYER_LOAD_PERCENT));

        /* character_load is used for weapon speed below, so sanitize value */
        if (character_load >= 1.0)
            character_load = 1.0;

        /* If a character is fully loaded, they will always get cut down to min
         * speed no matter what their dex.  Note that magic is below, so
         * still helps out.
         */
        if (op->speed > MAX_PLAYER_SPEED)
            op->speed -= character_load*(op->speed-MIN_PLAYER_SPEED);
        else
            op->speed -= character_load*(MAX_PLAYER_SPEED-MIN_PLAYER_SPEED);
    }

    /* This block is for weapon speed */
    op->weapon_speed = BASE_WEAPON_SPEED+get_speed_bonus(op->stats.Dex)-weapon_speed/20.0+added_speed/10.0;
    if (wc_obj) {
        op->weapon_speed += 0.005*wc_obj->level;
    } else
        op->weapon_speed += 0.005*op->level;

    /* character_load=1.0 means character is fully loaded, 0.0 is unloaded.  Multiplying
     * by 0.2 for penalty is purely arbitrary, but slows things down without completely
     * stopping them.
     */
    op->weapon_speed -= character_load*0.2;

    if (op->weapon_speed < 0.05)
        op->weapon_speed = 0.05;

    /* It is quite possible that a player's spell costing might have changed,
     * so we will check that now.
     */
    esrv_update_spells(op->contr);
}
void fix_object(object *op) {
    int i;
    float max = 9, added_speed = 0, speed_reduce_from_disease = 1;
    int weapon_speed = 0;
    int best_wc = 0, best_ac = 0, wc = 0, ac = 0;
    int prot[NROFATTACKS], vuln[NROFATTACKS], potion_resist[NROFATTACKS];
    const object *grace_obj = NULL, *mana_obj = NULL, *wc_obj = NULL;

    /* First task is to clear all the values back to their original values */
    if (op->type == PLAYER) {
        for (i = 0; i < NUM_STATS; i++) {
            set_attr_value(&(op->stats), i, get_attr_value(&(op->contr->orig_stats), i));
            set_attr_value(&(op->contr->applied_stats), i, 0);
        }
        if (settings.spell_encumbrance == TRUE)
            op->contr->encumbrance = 0;

        op->attacktype = 0;
        op->contr->digestion = 0;
        op->contr->gen_hp = 0;
        op->contr->gen_sp = 0;
        op->contr->gen_grace = 0;
        op->contr->gen_sp_armour = 10;
        op->contr->item_power = 0;

        /* Don't clobber all the range_ values.  range_golem otherwise
         * gets reset for no good reason, and we don't want to reset
         * range_magic (what spell is readied).  These three below
         * well get filled in based on what the player has equipped.
         */
        op->contr->ranges[range_bow] = NULL;
        op->contr->ranges[range_misc] = NULL;
        op->contr->ranges[range_skill] = NULL;
    } /* If player */
    memcpy(op->body_used, op->body_info, sizeof(op->body_info));

    if (op->slaying != NULL) {
        free_string(op->slaying);
        op->slaying = NULL;
    }
    if (!QUERY_FLAG(op, FLAG_WIZ)) {
        CLEAR_FLAG(op, FLAG_XRAYS);
        CLEAR_FLAG(op, FLAG_MAKE_INVIS);
    }

    CLEAR_FLAG(op, FLAG_LIFESAVE);
    CLEAR_FLAG(op, FLAG_STEALTH);
    CLEAR_FLAG(op, FLAG_BLIND);
    if (!QUERY_FLAG(&op->arch->clone, FLAG_REFL_SPELL))
        CLEAR_FLAG(op, FLAG_REFL_SPELL);
    if (!QUERY_FLAG(&op->arch->clone, FLAG_REFL_MISSILE))
        CLEAR_FLAG(op, FLAG_REFL_MISSILE);
    if (!QUERY_FLAG(&op->arch->clone, FLAG_UNDEAD))
        CLEAR_FLAG(op, FLAG_UNDEAD);
    if (!QUERY_FLAG(&op->arch->clone, FLAG_SEE_IN_DARK))
        CLEAR_FLAG(op, FLAG_SEE_IN_DARK);
    CLEAR_FLAG(op, FLAG_PROBE);

    op->path_attuned = op->arch->clone.path_attuned;
    op->path_repelled = op->arch->clone.path_repelled;
    op->path_denied = op->arch->clone.path_denied;
    op->glow_radius = op->arch->clone.glow_radius;
    op->move_type = op->arch->clone.move_type;
    op->chosen_skill = NULL;

    /* initializing resistances from the values in player/monster's
     * archetype clone
     */
    memcpy(&op->resist, &op->arch->clone.resist, sizeof(op->resist));

    for (i = 0; i < NROFATTACKS; i++) {
        if (op->resist[i] > 0)
            prot[i] = op->resist[i], vuln[i] = 0;
        else
            vuln[i] = -(op->resist[i]), prot[i] = 0;
        potion_resist[i] = 0;
    }

    wc = op->arch->clone.stats.wc;
    op->stats.dam = op->arch->clone.stats.dam;

    /* for players which cannot use armour, they gain AC -1 per 3 levels,
     * plus a small amount of physical resist, those poor suckers. ;)
     * the fact that maxlevel is factored in could be considered sort of bogus -
     * we should probably give them some bonus and cap it off - otherwise,
     * basically, if a server updates its max level, these playes may find
     * that their protection from physical goes down
     */
    if (!QUERY_FLAG(op, FLAG_USE_ARMOUR) && op->type == PLAYER) {
        ac = MAX(-10, op->arch->clone.stats.ac-op->level/3);
        prot[ATNR_PHYSICAL] += ((100-prot[AT_PHYSICAL])*(80*op->level/settings.max_level))/100;
    } else
        ac = op->arch->clone.stats.ac;

    op->stats.luck = op->arch->clone.stats.luck;
    op->speed = op->arch->clone.speed;

    /* OK - we've reset most all the objects attributes to sane values.
     * now go through and make adjustments for what the player has equipped.
     */

    FOR_INV_PREPARE(op, tmp) {
        /* See note in map.c:update_position about making this additive
         * since light sources are never applied, need to put check here.
         */
        if (tmp->glow_radius > op->glow_radius)
            op->glow_radius = tmp->glow_radius;

        /* This happens because apply_potion calls change_abil with the potion
         * applied so we can tell the player what chagned.  But change_abil
         * then calls this function.
         */
        if (QUERY_FLAG(tmp, FLAG_APPLIED) && tmp->type == POTION)
            continue;

        /* For some things, we don't care what is equipped */
        if (tmp->type == SKILL) {
            /* Want to take the highest skill here. */
            if (IS_MANA_SKILL(tmp->subtype)) {
                if (!mana_obj)
                    mana_obj = tmp;
                else if (tmp->level > mana_obj->level)
                    mana_obj = tmp;
            }
            if (IS_GRACE_SKILL(tmp->subtype)) {
                if (!grace_obj)
                    grace_obj = tmp;
                else if (tmp->level > grace_obj->level)
                    grace_obj = tmp;
            }
        }

        /* Container objects are not meant to adjust a players, but other applied
         * objects need to make adjustments.
         * This block should handle all player specific changes
         * The check for Praying is a bit of a hack - god given bonuses are put
         * in the praying skill, and the player should always get those.
         * It also means we need to put in additional checks for applied below,
         * because the skill shouldn't count against body positions being used
         * up, etc.
         */
        if ((QUERY_FLAG(tmp, FLAG_APPLIED) && tmp->type != CONTAINER && tmp->type != CLOSE_CON)
        || (tmp->type == SKILL && tmp->subtype == SK_PRAYING)) {
            if (op->type == PLAYER) {
                if (tmp->type == BOW)
                    op->contr->ranges[range_bow] = tmp;

                if (tmp->type == WAND || tmp->type == ROD)
                    op->contr->ranges[range_misc] = tmp;

                for (i = 0; i < NUM_STATS; i++) {
                    int8_t value;

                    value = get_attr_value(&(tmp->stats), i);
                    change_attr_value(&(op->stats), i, value);
                    if (strcmp(tmp->arch->clone.name, ARCH_DEPLETION) != 0)
                        change_attr_value(&(op->contr->applied_stats), i, value);
                }

                /* For this temporary calculation, allow wider range of stat - if we have
                 * having that gives +5 and different object that gives -5 and stat
                 * is maxed, we don't want different results based on order of
                 * inventory.
                 */
                check_stat_bounds(&(tmp->stats), -settings.max_stat, 2*settings.max_stat);

                /* these are the items that currently can change digestion, regeneration,
                 * spell point recovery and mana point recovery.  Seems sort of an arbitary
                 * list, but other items store other info into stats array.
                 */
                switch (tmp->type)
                {
                    case WEAPON:
                    case ARMOUR:
                    case HELMET:
                    case SHIELD:
                    case RING:
                    case BOOTS:
                    case GLOVES:
                    case AMULET:
                    case GIRDLE:
                    case BRACERS:
                    case CLOAK:
                    case DISEASE:
                    case FORCE:
                    case SKILL:
                        op->contr->digestion += tmp->stats.food;
                        op->contr->gen_hp += tmp->stats.hp;
                        op->contr->gen_sp += tmp->stats.sp;
                        op->contr->gen_grace += tmp->stats.grace;
                        op->contr->gen_sp_armour += tmp->gen_sp_armour;
                        /*FALLTHROUGH*/

                    /* Bow and skill utils need to update item_power specifically.
                     * This should fix bug #648
                     * Daniel Hawkins 2017-08-09
                     */
                    case BOW:
                    case SKILL_TOOL:
                        op->contr->item_power += tmp->item_power;
                        break;
                }
            } /* if this is a player */

            /* Update slots used for items */
            if (QUERY_FLAG(tmp, FLAG_APPLIED)) {
                for (i = 0; i < NUM_BODY_LOCATIONS; i++)
                    op->body_used[i] += tmp->body_info[i];
            }

            if (tmp->type == SYMPTOM && tmp->last_sp) {
                /* Should take the worst disease of the bunch */
                if (((float)tmp->last_sp/100.0) < speed_reduce_from_disease)
                    speed_reduce_from_disease = (float)tmp->last_sp/100.0;
            }

            /* Pos. and neg. protections are counted seperate (-> pro/vuln).
             * (Negative protections are calculated extactly like positive.)
             * Resistance from potions are treated special as well. If there's
             * more than one potion-effect, the bigger prot.-value is taken.
             */
            if (tmp->type != POTION) {
                for (i = 0; i < NROFATTACKS; i++) {
                    /* Potential for cursed potions, in which case we just can use
                     * a straight MAX, as potion_resist is initialized to zero.
                     */
                    if (tmp->type == POTION_RESIST_EFFECT) {
                        if (potion_resist[i])
                            potion_resist[i] = MAX(potion_resist[i], tmp->resist[i]);
                        else
                            potion_resist[i] = tmp->resist[i];
                    } else if (tmp->resist[i] > 0)
                        prot[i] += ((100-prot[i])*tmp->resist[i])/100;
                    else if (tmp->resist[i] < 0)
                        vuln[i] += ((100-vuln[i])*(-tmp->resist[i]))/100;
                }
            }

            /* There may be other things that should not adjust the attacktype */
            if (tmp->type != BOW && tmp->type != SYMPTOM)
                op->attacktype |= tmp->attacktype;

            op->path_attuned |= tmp->path_attuned;
            op->path_repelled |= tmp->path_repelled;
            op->path_denied |= tmp->path_denied;
            op->stats.luck += tmp->stats.luck;
            op->move_type |= tmp->move_type;

            if (QUERY_FLAG(tmp, FLAG_LIFESAVE))
                SET_FLAG(op, FLAG_LIFESAVE);
            if (QUERY_FLAG(tmp, FLAG_REFL_SPELL))
                SET_FLAG(op, FLAG_REFL_SPELL);
            if (QUERY_FLAG(tmp, FLAG_REFL_MISSILE))
                SET_FLAG(op, FLAG_REFL_MISSILE);
            if (QUERY_FLAG(tmp, FLAG_STEALTH))
                SET_FLAG(op, FLAG_STEALTH);
            if (QUERY_FLAG(tmp, FLAG_XRAYS))
                SET_FLAG(op, FLAG_XRAYS);
            if (QUERY_FLAG(tmp, FLAG_BLIND))
                SET_FLAG(op, FLAG_BLIND);
            if (QUERY_FLAG(tmp, FLAG_SEE_IN_DARK))
                SET_FLAG(op, FLAG_SEE_IN_DARK);
            if (QUERY_FLAG(tmp, FLAG_PROBE))
                SET_FLAG(op, FLAG_PROBE);
                
            // Items can make the wielder confused.
            if (QUERY_FLAG(tmp, FLAG_CONFUSED))
                SET_FLAG(op, FLAG_CONFUSED);

            if (QUERY_FLAG(tmp, FLAG_UNDEAD) && !QUERY_FLAG(&op->arch->clone, FLAG_UNDEAD))
                SET_FLAG(op, FLAG_UNDEAD);

            if (QUERY_FLAG(tmp, FLAG_MAKE_INVIS)) {
                SET_FLAG(op, FLAG_MAKE_INVIS);
                op->invisible = 1;
            }

            if (tmp->stats.exp && tmp->type != SKILL) {
                added_speed += (float)tmp->stats.exp/3.0;
            }

            switch (tmp->type) {
                /* skills modifying the character -b.t. */
                /* for all skills and skill granting objects */
            case SKILL:
                if (!QUERY_FLAG(tmp, FLAG_APPLIED))
                    break;

                if (IS_COMBAT_SKILL(tmp->subtype))
                    wc_obj = tmp;

                if (op->chosen_skill) {
                    LOG(llevDebug, "fix_object, op %s has multiple skills applied\n", op->name);
                }
                op->chosen_skill = tmp;
                if (tmp->stats.dam > 0) { /* skill is a 'weapon' */
                    if (!QUERY_FLAG(op, FLAG_READY_WEAPON))
                        weapon_speed = (int)WEAPON_SPEED(tmp);
                    if (weapon_speed < 0)
                        weapon_speed = 0;
                    op->stats.dam += tmp->stats.dam*(1+(op->chosen_skill->level/9));
                    op->stats.dam += tmp->magic;
                }
                if (tmp->stats.wc)
                    wc -= (tmp->stats.wc+tmp->magic);

                if (tmp->slaying != NULL) {
                    if (op->slaying != NULL)
                        free_string(op->slaying);
                    add_refcount(op->slaying = tmp->slaying);
                }

                if (tmp->stats.ac)
                    ac -= (tmp->stats.ac+tmp->magic);
                if (settings.spell_encumbrance == TRUE && op->type == PLAYER)
                    op->contr->encumbrance += (int)3*tmp->weight/1000;
                if (op->type == PLAYER)
                    op->contr->ranges[range_skill] = op;
                break;

            case SKILL_TOOL:
                if (op->chosen_skill) {
                    LOG(llevDebug, "fix_object, op %s has multiple skills applied\n", op->name);
                }
                op->chosen_skill = tmp;
                if (op->type == PLAYER)
                    op->contr->ranges[range_skill] = op;
                break;

            case SHIELD:
                if (settings.spell_encumbrance == TRUE && op->type == PLAYER)
                    op->contr->encumbrance += (int)tmp->weight/2000;
            case RING:
            case AMULET:
            case GIRDLE:
            case HELMET:
            case BOOTS:
            case GLOVES:
            case CLOAK:
                if (tmp->stats.wc)
                    wc -= tmp->stats.wc;
                if (tmp->stats.dam)
                    op->stats.dam += (tmp->stats.dam+tmp->magic);
                if (tmp->stats.ac)
                    ac -= (tmp->stats.ac+tmp->magic);
                break;

            case WEAPON:
                wc -= tmp->stats.wc;
                if (tmp->stats.ac && tmp->stats.ac+tmp->magic > 0)
                    ac -= tmp->stats.ac+tmp->magic;
                op->stats.dam += (tmp->stats.dam+tmp->magic);
                weapon_speed = ((int)WEAPON_SPEED(tmp)*2-tmp->magic)/2;
                if (weapon_speed < 0)
                    weapon_speed = 0;
                if (tmp->slaying != NULL) {
                    if (op->slaying != NULL)
                        free_string(op->slaying);
                    add_refcount(op->slaying = tmp->slaying);
                }
                /* If there is desire that two handed weapons should do
                 * extra strength damage, this is where the code should
                 * go.
                 */
                op->current_weapon = tmp;
                if (settings.spell_encumbrance == TRUE && op->type == PLAYER)
                    op->contr->encumbrance += (int)3*tmp->weight/1000;
                break;

            case ARMOUR: /* Only the best of these three are used: */
                if (settings.spell_encumbrance == TRUE && op->type == PLAYER)
                    op->contr->encumbrance += (int)tmp->weight/1000;

                /* ARMOUR falls through to here */

            case BRACERS:
            case FORCE:
                // Code simplification to reduce branching -- we don't need to sub/add ac and wc all the time.
                // Daniel Hawkins 2018-05-28
                if (tmp->stats.wc) {
                    // Since we are alreay here, make sure wc stacking also occurs for serpentman players.
                    if (tmp->type == BRACERS && op->type == PLAYER && op->arch->name && strcmp(op->arch->name, "serpentman_player") == 0)
                    {
                        // Apply ac from bracers directly.
                        // This also grants the side effect that armor and bracer ac are separate for serpentmen,
                        // But that should be better than the extra bracers being mostly useless.
                        wc -= tmp->stats.wc;
                    }
                    else if (best_wc < tmp->stats.wc) {
                        wc += best_wc;
                        best_wc = tmp->stats.wc;
                        wc -= tmp->stats.wc;
                    }
                }
                if (tmp->stats.ac) {
                    /*
                     * If we have a serpentman player, then we do bracers differently
                     * to allow for both bracers they equip to apply to ac, instead of only the best.
                     *
                     * Daniel Hawkins 2018-05-28
                     */
                    if (tmp->type == BRACERS && op->type == PLAYER && op->arch->name && strcmp(op->arch->name, "serpentman_player") == 0)
                    {
                        // Apply ac from bracers directly.
                        // This also grants the side effect that armor and bracer ac are separate for serpentmen,
                        // But that should be better than the extra bracers being mostly useless.
                        ac -= tmp->stats.ac+tmp->magic;
                    }
                    else if (best_ac < tmp->stats.ac+tmp->magic) {
                        ac += best_ac; /* Remove last bonus */
                        best_ac = tmp->stats.ac+tmp->magic;
                        ac -= (tmp->stats.ac+tmp->magic);
                    }
                }
                if (tmp->stats.dam && tmp->type == BRACERS)
                    op->stats.dam += (tmp->stats.dam+tmp->magic);
                if (ARMOUR_SPEED(tmp) && ARMOUR_SPEED(tmp)/10.0 < max)
                    max = ARMOUR_SPEED(tmp)/10.0;
                break;
            } /* switch tmp->type */
        } /* item is equipped */
    } FOR_INV_FINISH(); /* for loop of items */

    /* We've gone through all the objects the player has equipped.  For many things, we
     * have generated intermediate values which we now need to assign.
     */

    /* 'total resistance = total protections - total vulnerabilities'.
     * If there is an uncursed potion in effect, granting more protection
     * than that, we take: 'total resistance = resistance from potion'.
     * If there is a cursed (and no uncursed) potion in effect, we take
     * 'total resistance = vulnerability from cursed potion'.
     */
    for (i = 0; i < NROFATTACKS; i++) {
        op->resist[i] = prot[i]-vuln[i];
        if (potion_resist[i]
        && ((potion_resist[i] > op->resist[i]) || (potion_resist[i] < 0)))
            op->resist[i] = potion_resist[i];
    }

    fix_player(op, &ac, &wc, grace_obj, mana_obj, wc_obj, weapon_speed, added_speed);

    op->speed = op->speed*speed_reduce_from_disease;

    /* Max is determined by armour */
    if (op->speed > max)
        op->speed = max;

    op->speed += added_speed/10.0;


    /* Put a lower limit on speed.  Note with this speed, you move once every
     * 20 ticks or so.  This amounts to once every 3 seconds of realtime.
     */
    if (op->speed < 0.05 && op->type == PLAYER)
        op->speed = 0.05;

    /* I want to limit the power of small monsters with big weapons: */
    if (op->type != PLAYER
    && op->arch != NULL
    && op->stats.dam > op->arch->clone.stats.dam*3)
        op->stats.dam = op->arch->clone.stats.dam*3;

    /* Prevent overflows of wc - best you can get is ABS(120) - this
     * should be more than enough - remember, AC is also in 8 bits,
     * so its value is the same.
     */
    if (wc > 120)
        wc = 120;
    else if (wc < -120)
        wc = -120;
    op->stats.wc = wc;

    if (ac > 120)
        ac = 120;
    else if (ac < -120)
        ac = -120;
    op->stats.ac = ac;

    /* if for some reason the creature doesn't have any move type,
     * give them walking as a default.
     * The second case is a special case - to more closely mimic the
     * old behaviour - if your flying, your not walking - just
     * one or the other.
     */
    if (op->move_type == 0)
        op->move_type = MOVE_WALK;
    else if (op->move_type&(MOVE_FLY_LOW|MOVE_FLY_HIGH))
        op->move_type &= ~MOVE_WALK;

    object_update_speed(op);
}

int allowed_class(const object *op) {
    return op->stats.Dex > 0
        && op->stats.Str > 0
        && op->stats.Con > 0
        && op->stats.Int > 0
        && op->stats.Wis > 0
        && op->stats.Pow > 0
        && op->stats.Cha > 0;
}

void set_dragon_name(object *pl, const object *abil, const object *skin) {
    int atnr = -1;  /* attacknumber of highest level */
    int level = 0;  /* highest level */
    int i;

    /* Perhaps do something more clever? */
    if (!abil || !skin)
        return;

    /* first, look for the highest level */
    for (i = 0; i < NROFATTACKS; i++) {
        if (atnr_is_dragon_enabled(i)
        && (atnr == -1 || abil->resist[i] > abil->resist[atnr])) {
            level = abil->resist[i];
            atnr = i;
        }
    }

    /* now if there are equals at highest level, pick the one with focus,
        or else at random */
    if (atnr_is_dragon_enabled(abil->stats.exp)
    && abil->resist[abil->stats.exp] >= level)
        atnr = abil->stats.exp;

    level = (int)(level/5.);

    /* now set the new title */
    if (pl->contr != NULL) {
        player_set_dragon_title(pl->contr, level, attacks[atnr], skin->resist[atnr]);
    }
}

static void dragon_level_gain(object *who) {
    object *abil = NULL;    /* pointer to dragon ability force*/
    object *skin = NULL;    /* pointer to dragon skin force*/

    /* now grab the 'dragon_ability'-forces from the player's inventory */
    abil = object_find_by_type_and_arch_name(who, FORCE, "dragon_ability_force");
    skin = object_find_by_type_and_arch_name(who, FORCE, "dragon_skin_force");
    /* if the force is missing -> bail out */
    if (abil == NULL)
        return;

    /* The ability_force keeps track of maximum level ever achieved.
     * New abilties can only be gained by surpassing this max level
     */
    if (who->level > abil->level) {
        /* increase our focused ability */
        int lev = ++(abil->resist[abil->stats.exp]);

        if (lev > 0) {
            /* try to hand out a new ability-gift
             * if not the right level, this will handout nothing.
             */
            dragon_ability_gain(who, (int)abil->stats.exp, lev);
        }

        if (abil->last_eat > 0 && atnr_is_dragon_enabled(abil->last_eat)) {
            /* apply new ability focus */
            draw_ext_info_format(NDI_UNIQUE|NDI_BLUE, 0, who, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_RACE,
                                 "Your metabolism now focuses on %s!",
                                 change_resist_msg[abil->last_eat]);

            abil->stats.exp = abil->last_eat;
            abil->last_eat = 0;
        }

        abil->level = who->level;
    }

    /* last but not least, set the new title for the dragon */
    set_dragon_name(who, abil, skin);
}

object *give_skill_by_name(object *op, const char *skill_name) {
    object *skill_obj;
    archetype *skill_arch;

    skill_arch = get_archetype_by_skill_name(skill_name, SKILL);
    if (!skill_arch) {
        LOG(llevError, "add_player_exp: couldn't find skill %s\n", skill_name);
        return NULL;
    }
    skill_obj = arch_to_object(skill_arch); /* never returns NULL. */

    /* clear the flag - exp goes into this bucket, but player
     * still doesn't know it.
     */
    CLEAR_FLAG(skill_obj, FLAG_CAN_USE_SKILL);
    skill_obj->stats.exp = 0;
    skill_obj->level = 1;
    object_insert_in_ob(skill_obj, op);
    if (op->contr) {
        op->contr->last_skill_ob[skill_obj->subtype] = skill_obj;
        op->contr->last_skill_exp[skill_obj->subtype] = -1;
    }
    return skill_obj;
}

void player_lvl_adj(object *who, object *op) {
    char buf[MAX_BUF];

    assert(who);

    if (!op)       /* when rolling stats */
        op = who;

    if (op->level < settings.max_level && op->stats.exp >= level_exp(op->level+1, who->expmul)) {
        do{
            op->level++;

            if (op != NULL && op == who && op->stats.exp > 1 && is_dragon_pl(who))
                dragon_level_gain(who);

            /* Only roll these if it is the player (who) that gained the level */
            if (op == who && (who->level < 11) && who->type == PLAYER) {
                who->contr->levhp[who->level] = die_roll(2, 4, who, PREFER_HIGH)+1;
                who->contr->levsp[who->level] = die_roll(2, 3, who, PREFER_HIGH);
                who->contr->levgrace[who->level] = die_roll(2, 2, who, PREFER_HIGH)-1;
            }

            fix_object(who); // TODO: Call this only once per function call?
            if (op->level > 1) {
                if (op->type != PLAYER)
                    snprintf(buf, sizeof(buf), "You are now level %d in the %s skill.", op->level, op->name);
                else
                    snprintf(buf, sizeof(buf), "You are now level %d.", op->level);

                draw_ext_info(NDI_UNIQUE|NDI_RED, 0, who, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_LEVEL_GAIN, buf);
            }
        } while (op->level < settings.max_level && op->stats.exp >= level_exp(op->level+1, who->expmul)); /* To increase more levels */
    } else if (op->level > 1 && op->stats.exp < level_exp(op->level, who->expmul)) {
        do{
            op->level--;
            fix_object(who); // TODO: Call this only once per function call?
            if (op->type != PLAYER)
                snprintf(buf, sizeof(buf), "You are now level %d in the %s skill.", op->level, op->name);
            else
                snprintf(buf, sizeof(buf), "You are now level %d.", op->level);

            draw_ext_info(NDI_UNIQUE|NDI_RED, 0, who, MSG_TYPE_ATTRIBUTE, MSG_TYPE_ATTRIBUTE_LEVEL_LOSS, buf);
        } while (op->level > 1 && op->stats.exp < level_exp(op->level, who->expmul)); /* To decrease more levels */
    }
}

int64_t level_exp(int level, double expmul) {
    if (level > settings.max_level)
        return expmul*levels[settings.max_level];
    return expmul*levels[level];
}

void calc_perm_exp(object *op) {
    int64_t p_exp_min;

    /* Ensure that our permanent experience minimum is met.
     * permenent_exp_ratio is an integer percentage, we divide by 100
     * to get the fraction   */
    p_exp_min = settings.permanent_exp_ratio*op->stats.exp/100;

    if (op->perm_exp < p_exp_min)
        op->perm_exp = p_exp_min;

    /* Cap permanent experience. */
    if (op->perm_exp < 0)
        op->perm_exp = 0;
    else if (op->perm_exp > MAX_EXPERIENCE)
        op->perm_exp = MAX_EXPERIENCE;
}


static void add_player_exp(object *op, int64_t exp, const char *skill_name, int flag) {
    object *skill_obj = NULL;
    int64_t limit, exp_to_add;
    int i;

    /* prevents some forms of abuse. */
    if (op->contr->braced)
        exp = exp/5;

    /* Try to find the matching skill.
     * We do a shortcut/time saving mechanism first - see if it matches
     * chosen_skill.  This means we don't need to search through
     * the players inventory.
     */
    if (skill_name) {
        if (op->chosen_skill
        && op->chosen_skill->type == SKILL
        && !strcmp(skill_name, op->chosen_skill->skill))
            skill_obj = op->chosen_skill;
        else {
            for (i = 0; i < NUM_SKILLS; i++)
                if (op->contr->last_skill_ob[i]
                && !strcmp(op->contr->last_skill_ob[i]->skill, skill_name)) {
                    skill_obj = op->contr->last_skill_ob[i];
                    break;
                }

            /* Player doesn't have the skill.  Check to see what to do, and give
                * it to the player if necessary
                */
            if (!skill_obj) {
                if (flag == SK_EXP_NONE)
                    return;
                else if (flag == SK_EXP_ADD_SKILL)
                    skill_obj = give_skill_by_name(op, skill_name);
            }
        }
    }

    /* Basically, you can never gain more experience in one shot
     * than half what you need to gain for next level.
     */
    exp_to_add = exp;
    /*
     * Make sure we aren't trying to go backwards when we hit maximum level,
     * but make sure we can still add to our permanent experience.
     * 
     * -- Daniel Hawkins 2015-05-24
     */
    if (op->level == settings.max_level)
        limit = levels[op->level] / 2;
    else
        limit = (levels[op->level+1]-levels[op->level])/2;
        
    if (exp_to_add > limit)
        exp_to_add = limit;

    ADD_EXP(op->stats.exp, (float)exp_to_add*(skill_obj ? skill_obj->expmul : 1));
    if (settings.permanent_exp_ratio) {
        ADD_EXP(op->perm_exp, (float)exp_to_add*PERM_EXP_GAIN_RATIO*(skill_obj ? skill_obj->expmul : 1));
        calc_perm_exp(op);
    }

    player_lvl_adj(op, NULL);
    if (skill_obj) {
        exp_to_add = exp;
        /*
         * Make sure we aren't trying to go backwards when we hit maximum level,
         * but make sure we can still add to our permanent experience.
         * 
         * -- Daniel Hawkins 2015-05-24
         */
        if (skill_obj->level == settings.max_level)
            limit = levels[skill_obj->level] / 2;
        else
            limit = (levels[skill_obj->level+1]-levels[skill_obj->level])/2;
            
        if (exp_to_add > limit)
            exp_to_add = limit;
        ADD_EXP(skill_obj->stats.exp, exp_to_add);
        if (settings.permanent_exp_ratio) {
            skill_obj->perm_exp += exp_to_add*PERM_EXP_GAIN_RATIO;
            calc_perm_exp(skill_obj);
        }
        player_lvl_adj(op, skill_obj);
    }
}

int64_t check_exp_loss(const object *op, int64_t exp) {
    int64_t del_exp;

    if (exp > op->stats.exp)
        exp = op->stats.exp;
    if (settings.permanent_exp_ratio) {
        del_exp = (op->stats.exp-op->perm_exp)*PERM_EXP_MAX_LOSS_RATIO;
        if (del_exp < 0)
            del_exp = 0;
        if (exp > del_exp)
            exp = del_exp;
    }
    return exp;
}

int64_t check_exp_adjust(const object *op, int64_t exp) {
    if (exp < 0)
        return check_exp_loss(op, exp);
    else
        return MIN(exp, MAX_EXPERIENCE-op->stats.exp);
}

static void subtract_player_exp(object *op, int64_t exp, const char *skill, int flag) {
    float fraction = (float)exp/(float)op->stats.exp;
    int64_t del_exp;

    FOR_INV_PREPARE(op, tmp)
        if (tmp->type == SKILL && tmp->stats.exp) {
            if (flag == SK_SUBTRACT_SKILL_EXP && skill && !strcmp(tmp->skill, skill)) {
                del_exp = check_exp_loss(tmp, exp);
                tmp->stats.exp -= del_exp;
                player_lvl_adj(op, tmp);
            } else if (flag != SK_SUBTRACT_SKILL_EXP) {
                /* only want to process other skills if we are not trying
                    * to match a specific skill.
                    */
                del_exp = check_exp_loss(tmp, tmp->stats.exp*fraction);
                tmp->stats.exp -= del_exp;
                player_lvl_adj(op, tmp);
            }
        }
    FOR_INV_FINISH();
    if (flag != SK_SUBTRACT_SKILL_EXP) {
        del_exp = check_exp_loss(op, exp);
        op->stats.exp -= del_exp;
        player_lvl_adj(op, NULL);
    }
}

void change_exp(object *op, int64_t exp, const char *skill_name, int flag) {
    /* safety */
    if (!op) {
        LOG(llevError, "change_exp() called for null object!\n");
        return;
    }

    /* if no change in exp, just return - most of the below code
     * won't do anything if the value is 0 anyways.
     */
    if (exp == 0)
        return;

    /* Monsters are easy - we just adjust their exp - we
     * don't adjust level, since in most cases it is unrelated to
     * the exp they have - the monsters exp represents what its
     * worth.
     */
    if (op->type != PLAYER) {
        /* Sanity check */
        if (!QUERY_FLAG(op, FLAG_ALIVE))
            return;

        /* reset exp to max allowed value.  We subtract from
        * MAX_EXPERIENCE to prevent overflows.  If the player somehow has
        * more than max exp, just return.
        */
        if (exp > 0 && (op->stats.exp > (MAX_EXPERIENCE-exp))) {
            exp = MAX_EXPERIENCE-op->stats.exp;
            if (exp < 0)
                return;
        }

        op->stats.exp += exp;
    } else {   /* Players only */
        if (exp > 0)
            add_player_exp(op, exp, skill_name, flag);
        else
            subtract_player_exp(op, FABS(exp), skill_name, flag);
    }
}

void apply_death_exp_penalty(object *op) {
    int64_t loss;
    int64_t percentage_loss;  /* defined by the setting 'death_penalty_percent' */
    int64_t level_loss;   /* defined by the setting 'death_penalty_levels */

    FOR_INV_PREPARE(op, tmp)
        if (tmp->type == SKILL && tmp->stats.exp) {
            percentage_loss = tmp->stats.exp*settings.death_penalty_ratio/100;
            level_loss = tmp->stats.exp-levels[MAX(0, tmp->level-settings.death_penalty_level)];

            /* With the revised exp system, you can get cases where
            * losing several levels would still require that you have more
            * exp than you currently have - this is true if the levels
            * tables is a lot harder.
            */
            if (level_loss < 0)
                level_loss = 0;

            loss = check_exp_loss(tmp, MIN(level_loss, percentage_loss));

            tmp->stats.exp -= loss;
            player_lvl_adj(op, tmp);
        }
    FOR_INV_FINISH();

    percentage_loss = op->stats.exp*settings.death_penalty_ratio/100;
    level_loss = op->stats.exp-levels[MAX(0, op->level-settings.death_penalty_level)];
    if (level_loss < 0)
        level_loss = 0;
    loss = check_exp_loss(op, MIN(level_loss, percentage_loss));

    op->stats.exp -= loss;
    player_lvl_adj(op, NULL);
}

int did_make_save(const object *op, int level, int bonus) {
    if (level > MAX_SAVE_LEVEL)
        level = MAX_SAVE_LEVEL;

    if ((random_roll(1, 20, op, PREFER_HIGH)+bonus) < savethrow[level])
        return 0;
    return 1;
}

void share_exp(object *op, int64_t exp, const char *skill, int flag) {
    int shares = 0, count = 0;
    player *pl;
    partylist *party;

    if (op->type != PLAYER || op->contr->party == NULL) {
        change_exp(op, exp, skill, 0);
        return;
    }

    party = op->contr->party;

    for (pl = first_player; pl != NULL; pl = pl->next) {
        if (party && pl->ob->contr->party == party && on_same_map(pl->ob, op)) {
            count++;
            shares += (pl->ob->level+4);
        }
    }

    assert(shares > 0);

    if (count == 1 || shares > exp)
        change_exp(op, exp, skill, flag);
    else {
        int64_t share = exp/shares, given = 0, nexp;
        for (pl = first_player; pl != NULL; pl = pl->next) {
            if (party && pl->ob->contr->party == party && on_same_map(pl->ob, op)) {
                nexp = (pl->ob->level+4)*share;
                change_exp(pl->ob, nexp, skill, SK_EXP_TOTAL);
                given += nexp;
            }
        }
        exp -= given;
        /* give any remainder to the player */
        change_exp(op, exp, skill, flag);
    }
}

int get_cha_bonus(int stat) {
    return int_bonuses[INT_CHA_BONUS][get_index(stat, settings.max_stat)];
}

int get_dex_bonus(int stat) {
    return int_bonuses[INT_DEX_BONUS][get_index(stat, settings.max_stat)];
}

int get_thaco_bonus(int stat) {
    return int_bonuses[INT_THAC0_BONUS][get_index(stat, settings.max_stat)];
}

uint32_t get_weight_limit(int stat) {
    return int_bonuses[INT_WEIGHT_LIMIT][get_index(stat, settings.max_stat)];
}

int get_learn_spell(int stat) {
    return int_bonuses[INT_LEARN_SPELL][get_index(stat, settings.max_stat)];
}

int get_cleric_chance(int stat) {
    return int_bonuses[INT_CLERIC_CHANCE][get_index(stat, settings.max_stat)];
}

int get_turn_bonus(int stat) {
    return int_bonuses[INT_TURN_BONUS][get_index(stat, settings.max_stat)];
}

int get_dam_bonus(int stat) {
    return int_bonuses[INT_DAM_BONUS][get_index(stat, settings.max_stat)];
}

float get_speed_bonus(int stat) {
    return float_bonuses[FLOAT_DEX_BONUS][get_index(stat, settings.max_stat)];
}

int get_fear_bonus(int stat) {
    return int_bonuses[INT_FEAR_BONUS][get_index(stat, settings.max_stat)];
}

static float get_con_bonus(int stat) {
    return float_bonuses[FLOAT_CON_BONUS][get_index(stat, settings.max_stat)];
}

static float get_sp_bonus(int stat) {
    return float_bonuses[FLOAT_SP_BONUS][get_index(stat, settings.max_stat)];
}

static float get_grace_bonus(int stat) {
    return float_bonuses[FLOAT_GRACE_BONUS][get_index(stat, settings.max_stat)];
}

static size_t get_index(int stat, size_t max_index) {
    size_t index;

    if (stat < 0) {
        return 0;
    }

    index = (size_t)stat;
    return MIN(index, max_index);
}

static int load_table_int(int **bonuses, FILE *fp, char *bonus_name)
{
    char buf[MAX_BUF], *cp;
    int on_stat = 0, tmp_bonus;

    *bonuses = calloc(settings.max_stat+1, sizeof(int));

    while (fgets(buf, MAX_BUF-1, fp) != NULL) {
        if (buf[0] == '#')
            continue;

        /* Skip over empty lines */
        if (buf[0] == '\n')
            continue;

        /* Do not care about opening brace */
        if (buf[0] == '{')
            continue;

        if (buf[0] == '}') {
            if ((on_stat-1) != settings.max_stat) {
                LOG(llevError,"Number of bonus does not match max stat (%d!=%d, bonus=%s)\n",
                    on_stat, settings.max_stat, bonus_name);
                return 1;
            }
            else return 0;
        }

        /* If not any of the above values, must be the stat table,
         * or so we hope.
         */
        cp = buf;
        while (*cp != 0) {
            /* Skip over any non numbers */
            while (!isdigit(*cp) && *cp!='.' && *cp!='-' && *cp!='+' && *cp != 0)
                cp++;

            if (*cp == 0) break;

            tmp_bonus = atoi(cp);

            if (on_stat > settings.max_stat) {
                LOG(llevError,"Number of bonus entries exceed max stat (line=%s, bonus=%s)\n",
                    buf, bonus_name);
                return 1;
            }
            (*bonuses)[on_stat] = tmp_bonus;
            on_stat++;

            /* Skip over any digits, as that is the number we just processed */
            while ((isdigit(*cp) || *cp=='-' || *cp=='+') && *cp != 0)
                cp++;
        }
    }
    /* This should never happen - we should always get the closing brace */
    LOG(llevError,"Reached end of file without getting close brace?  bonus=%s\n", bonus_name);
    return 1;
}

static int load_table_float(float **bonuses, FILE *fp, char *bonus_name)
{
    char buf[MAX_BUF], *cp;
    int on_stat = 0;
    float tmp_bonus;

    *bonuses = calloc(settings.max_stat+1, sizeof(float));

    while (fgets(buf, MAX_BUF-1, fp) != NULL) {
        if (buf[0] == '#')
            continue;

        /* Skip over empty lines */
        if (buf[0] == '\n')
            continue;

        /* Do not care about opening brace */
        if (buf[0] == '{')
            continue;

        if (buf[0] == '}') {
            if ((on_stat-1) != settings.max_stat) {
                LOG(llevError,"Number of bonus does not match max stat (%d!=%d, bonus=%s)\n",
                    on_stat, settings.max_stat, bonus_name);
                return 1;
            }
            else return 0;
        }

        /* If not any of the above values, must be the stat table,
         * or so we hope.
         */
        cp = buf;
        while (*cp != 0) {
            /* Skip over any non numbers */
            while (!isdigit(*cp) && *cp!='.' && *cp!='-' && *cp!='+' && *cp != 0)
                cp++;

            if (*cp == 0) break;

            tmp_bonus = atof(cp);

            if (on_stat > settings.max_stat) {
                LOG(llevError,"Number of bonus entries exceed max stat (line=%s, bonus=%s)\n",
                    buf, bonus_name);
                return 1;
            }
            (*bonuses)[on_stat] = tmp_bonus;
            on_stat++;

            /* Skip over any digits, as that is the number we just processed
             * since this is floats, also skip over any dots.
             */
            while ((isdigit(*cp) || *cp=='-' || *cp=='+' || *cp=='.') && *cp != 0)
                cp++;
        }
    }
    /* This should never happen - we should always get the closing brace */
    LOG(llevError,"Reached end of file without getting close brace?  bonus=%s\n", bonus_name);
    return 1;
}


void init_stats(int reload) {
    char buf[MAX_BUF], *cp;
    int error=0, i, oldmax = settings.max_stat;
    FILE *fp;
    float *new_float_bonuses[NUM_FLOAT_BONUSES];
    int *new_int_bonuses[NUM_INT_BONUSES];

    snprintf(buf, sizeof(buf), "%s/stat_bonus", settings.confdir);

    memset(new_int_bonuses, 0, NUM_INT_BONUSES * sizeof(int));
    memset(new_float_bonuses, 0, NUM_FLOAT_BONUSES * sizeof(float));

    if ((fp = fopen(buf, "r")) == NULL) {
        LOG(llevError, "Fatal error: could not open experience table (%s)\n", buf);
        if (reload) return;
        else exit(1);
    }
    while (fgets(buf, MAX_BUF-1, fp) != NULL) {
        if (buf[0] == '#')
            continue;

        /* eliminate newline */
        if ((cp = strrchr(buf, '\n')) != NULL)
            *cp = '\0';

        /* Skip over empty lines */
        if (buf[0] == 0)
            continue;

        /* Skip over leading spaces */
        cp = buf;
        while (isspace(*cp) && *cp != 0)
            cp++;

        if (!strncasecmp(cp, "max_stat", 8)) {
            int newmax = atoi(cp+8);

            /* newmax must be at least MIN_STAT and we do not currently
             * cupport decrease max stat on the fly - why this might be possible,
             * bounds checking for all objects would be needed, potentionally resetting
             * them.
             * If this is a reload, then on error, we just return without doing work.
             * If this is initial load, having an invalid stat range is an error, so
             * exit the program.
             */
            if (newmax < MIN_STAT || newmax < settings.max_stat) {
                LOG(llevError, "Got invalid max_stat (%d) from stat_bonus file\n", newmax);
                fclose(fp);
                if (reload) return;
                else exit(1);
            }
            settings.max_stat = newmax;
            continue;
        }
        /* max_stat needs to be set before any of the bonus values - we
         * need to know how large to make the array.
         */
        if (settings.max_stat == 0) {
            LOG(llevError, "Got bonus line or otherwise unknown value before max stat! (%s)\n",
                buf);
            fclose(fp);

            if (reload) {
                return;
            } else {
                exit(1);
            }
        }

        for (i=0; i<NUM_INT_BONUSES; i++) {
            if (!strncasecmp(cp, int_bonus_names[i], strlen(int_bonus_names[i]))) {
                error = load_table_int(&new_int_bonuses[i], fp, cp);
                break;
            }
        }
        /* If we did not find a match in the int bonuses, check the
         * float bonuses now.
         */
        if (i == NUM_INT_BONUSES) {
            for (i=0; i<NUM_FLOAT_BONUSES; i++) {
                if (!strncasecmp(cp, float_bonus_names[i], strlen(float_bonus_names[i]))) {
                    error = load_table_float(&new_float_bonuses[i], fp, cp);
                    break;
                }
            }
            /* This may not actually be a critical error */
            if (i == NUM_FLOAT_BONUSES) {
                LOG(llevError,"Unknown line in stat_bonus file: %s\n", buf);
            }
        }
        if (error) break;
    }
    fclose(fp);

    /* Make sure that we have load tables for all the bonuses.
     * This is critical on initial load, but on reloads, it enusres that
     * all the bonus data matches.
     */
    for (i=0; i<NUM_INT_BONUSES; i++) {
        if (!new_int_bonuses[i]) {
            LOG(llevError,"No bonus loaded for %s\n", int_bonus_names[i]);
            error=2;
        }
    }

    for (i=0; i<NUM_FLOAT_BONUSES; i++) {
        if (!new_float_bonuses[i]) {
            LOG(llevError,"No bonus loaded for %s\n", float_bonus_names[i]);
            error=2;
        }
    }

    /* If we got an error, we just free up the data we read in and return/exit.
     * if no error, we make the tables we just read in into the default
     * tables.
     */
    if (error) {
        if (error==1)
            LOG(llevError,"Got error reading stat_bonus: %s\n", buf);

        if (reload) {
            for (i=0; i<NUM_INT_BONUSES; i++)
                if (new_int_bonuses[i]) FREE_AND_CLEAR(new_int_bonuses[i]);
            for (i=0; i<NUM_FLOAT_BONUSES; i++)
                if (new_float_bonuses[i]) FREE_AND_CLEAR(new_float_bonuses[i]);
            settings.max_stat = oldmax;
        } else {
            exit(1);
        }
    } else {
        /* Everything check out - now copy the data into
         * the live arrays.
         */
        for (i=0; i<NUM_INT_BONUSES; i++) {
            free(int_bonuses[i]);
            int_bonuses[i] = new_int_bonuses[i];
            new_int_bonuses[i] = NULL;
        }

        for (i=0; i<NUM_FLOAT_BONUSES; i++) {
            free(float_bonuses[i]);
            float_bonuses[i] = new_float_bonuses[i];
            new_float_bonuses[i] = NULL;
        }
    }
}