Skip to content

Commit ed539cc

Browse files
committed
minor fixes and clarity
1 parent 97c2077 commit ed539cc

2 files changed

Lines changed: 9 additions & 5 deletions

File tree

code/mission/missionparse.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1291,7 +1291,9 @@ void parse_player_info2(mission *pm)
12911291
support_rearm_list.clear();
12921292
stuff_loadout_list(support_rearm_list, ParseLookupType::MISSION_LOADOUT_WEAPON_LIST);
12931293

1294-
if (!pm->support_ships.rearm_pool_from_loadout) {
1294+
if (pm->support_ships.rearm_pool_from_loadout) {
1295+
WarningEx(LOCATION, "+Support Rearm Pool is set but +Support Rearm Pool From Loadout is also enabled! The explicit pool will be ignored.\n");
1296+
} else {
12951297
for (auto& wc : support_rearm_list) {
12961298
if (wc.index < 0 || wc.index >= weapon_info_size()) {
12971299
continue;
@@ -1308,6 +1310,8 @@ void parse_player_info2(mission *pm)
13081310
pm->support_ships.rearm_weapon_pool[nt][wc.index] = 0;
13091311
} else if (wc.count < 0) {
13101312
pm->support_ships.rearm_weapon_pool[nt][wc.index] = -1;
1313+
} else if (wc.count == 0) {
1314+
pm->support_ships.rearm_weapon_pool[nt][wc.index] = 0;
13111315
} else if (wc.count > 0 && pm->support_ships.rearm_weapon_pool[nt][wc.index] >= 0) {
13121316
pm->support_ships.rearm_weapon_pool[nt][wc.index] += wc.count;
13131317
}

code/ship/ship.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16427,13 +16427,12 @@ int ship_do_rearm_frame( object *objp, float frametime )
1642716427
if (objp == Player_obj)
1642816428
joy_ff_play_reload_effect();
1642916429

16430+
const int prev_ammo = swp->secondary_bank_ammo[i];
1643016431
swp->secondary_bank_ammo[i] += reload_amount;
16431-
use_mission_rearm_pool_for_weapon(weapon_class, reload_amount, shipp->team);
1643216432
if (swp->secondary_bank_ammo[i] > swp->secondary_bank_start_ammo[i]) {
1643316433
swp->secondary_bank_ammo[i] = swp->secondary_bank_start_ammo[i];
1643416434
}
16435-
} else {
16436-
}
16435+
use_mission_rearm_pool_for_weapon(weapon_class, swp->secondary_bank_ammo[i] - prev_ammo, shipp->team);
1643716436
} else {
1643816437
banks_full++;
1643916438
}
@@ -16518,11 +16517,12 @@ int ship_do_rearm_frame( object *objp, float frametime )
1651816517
if (sound_index.isValid())
1651916518
snd_play_3d(gamesnd_get_game_sound(sound_index), &objp->pos, &View_position);
1652016519

16520+
const int prev_primary_ammo = swp->primary_bank_ammo[i];
1652116521
swp->primary_bank_ammo[i] += reload_amount;
16522-
use_mission_rearm_pool_for_weapon(weapon_class, reload_amount, shipp->team);
1652316522
if (swp->primary_bank_ammo[i] > swp->primary_bank_start_ammo[i]) {
1652416523
swp->primary_bank_ammo[i] = swp->primary_bank_start_ammo[i];
1652516524
}
16525+
use_mission_rearm_pool_for_weapon(weapon_class, swp->primary_bank_ammo[i] - prev_primary_ammo, shipp->team);
1652616526
}
1652716527
} else {
1652816528
primary_banks_full++;

0 commit comments

Comments
 (0)