"we're already holding a weapon we can reload" when we also have some ammo to reload it.

No need to check self.weapon here

Make a comment more correct

Bot skill level 2 for reloading in-hand weapon. Makes more sense

Only for bots skill 5 and up. Since only such both can reload weapons in their inventory idly, so not switching to unloaded weapons for lower skills would cause them to never reload their guns. This also makes bots more stupid at lower skills, causing them to switch to guns they'll then have to reload during fighting.

Only refuse reload scheduled weapons if we have something else to switch to. If we somehow have no other weapon we can use, switch to anything even during combat.

Simplify that last code a bit

Only avoid a reload scheduled weapon if our current one has ammo. If we're out of ammo, we must urgently switch to anything else

Write that last code in a better way

Bot AI: Don't switch to a weapon scheduled for reloading during combat. We spend time reloading it instead of fighting

Fix a small issue in reload code

Remove the first code. Now that bots know how to reload their weapons during brakes, it might be less intelligent to not switch while reloading during combat.

Better way of checking if we're already holding a weapon we can reload

Skill limits for each bot reloading decision. Also improve some comments

Improve that last code. When not attacking, a bot will switch to all weapons that are not fully loaded, and attempt to reload them, to keep them ready.

Make bots reload weapons when not attacking anyone. I'm not sure if the way I'm sending the impulse command is ok, however

Ok, let's do a few changes to the bot AI too. Bots should not switch weapons while reloading. Since not being very intelligent, they will end up with lots of unloaded weapons they'll keep switching between, not getting to shoot many of them at all.

Merge two voids back again. It makes more sense this way.

Better comments

No need to use floats when calling W_ReloadCheck, as we have the current globals to use directly.

Move all checks where they belong

Attempt to sort the code in a better way

Error case: Skip reloading and warn if attempting to reload a weapon that does not have WEP_FLAG_RELOADABLE. Useful for players that make new weapons, if they ever forget to add it.

Improve a comment

Skip melee attack for shotgun while reloading

Comment out that last change, rather

Do not set ATTACK_FINISHED in reload code any more. This causes annoying delays if eg: You start reloading a weapon, then quickly switch to another weapon and back to it. Reloading is then canceled, but the 2 second reload time is still there, so your weapon is delayed for 2 seconds without reason.

Fix that last change

A small change

Rename zero_ammo to ammo_none

Stop sending ammo type in W_Reload, now that we update self.current_ammo in WR_SETUP for each weapon.

Oops, forgot to update the uzi.

Set self.current_ammo in each weapon's WR_SETUP stage. That way, this float can be used for many things throughout the code, and is always updated. Not fully finished.

Remove parts of a forgotten feature

Use the new ammo function for laser and hook too (tested)

Phase 2. A global ammo decreasing function. Still not fully ready

Simplify some minstanex code

First phase, second part.

Attempt to further simplify the reload code, as requested. First part of the first phase.

After intensively testing the code today, and not receiving any complaints for weeks, I'm concluding this feature is finished. Revert reload settings to default balance, and mark the code as ready to merge on my end.

Oops, wrote wrong ammo in sniper rifle by mistake

Small fix for Electro ammo code

Merge branch 'master' into mirceakitsune/universal_reload_system

Merge branch 'master' of git://git.xonotic.org/xonotic/xonotic-data.pk3dir

more translation for French

fix typo

Merge branch 'master' of git://git.xonotic.org/xonotic/xonotic-data.pk3dir

add cl_allow_uidtracking

fix part of the last merge, because the "favorite" button is supposed to work even with an empty server list, if the server IP text box is not empty

Merge remote branch 'origin/terencehill/misc_bugfixes'

print a message when encountering "type"

Disable Bookmark button if there are no server

mark the Per weapon string translatable

Merge remote branch 'origin/akari/healthcross'

Conflicts:
qcsrc/menu/xonotic/dialog_multiplayer_playersetup.c

Merge remote branch 'origin/mirceakitsune/gametypefilter_teamspawns'

make mines "super aggressive" (turn off the friend-nearby check) when it is expired

Decrease rocket health so that it can be blown up more easily by grenades/electro

Adjust rocket health from 40 to 30, this way mortars can blow it up easier but lasers can only move it

Merge remote branch 'origin/master' into samual/balancesamual

Disable HLAC as I want to replace it with a lightning gun, and also add zoom to sniper rifle secondary as IT IS A SNIPER. If any weapon makes the most sense with zoom on secondary, this one does.

Make it so that minelayer shows up in g_weaponarena "most" -- fix the comments in the code -- change the mine firing spped and lifetime (for now 10 second lifetime, maybe in the future it will be longer)

Merge branch 'savagex/plat-movehack'

reduce offset a bit that prevents players from getting stuck in platforms. (0 0 1) was a bit excessive.

Fix Map Info dialog can be opened by pressing ENTER in the map list if there are no maps for the current gametype
This bug fix also disables other keys when there are no maps as there's nothing to do

Merge remote branch 'origin/master' into samual/balancesamual

Conflicts:
balanceXonotic.cfg

Some quick work on a feature for the minelayer which plays a sound at the end of its lifetime, this way you can have e.g. a countdown of its death alerting you it's going to die.

complete German translation

add the other 3 languages to the menu

update unfinished translations a bit

credits for Finnish translation

Finnish by Exitium

Merge branch 'master' into mirceakitsune/universal_reload_system

be more specific on alternatives to controlled movement

CPMA has teleport maxspeed of 400, I got told

Merge branch 'master' of ssh://git.xonotic.org/xonotic-data.pk3dir into savagex/plat-movehack

Merge remote branch 'origin/terencehill/typos'

Merge remote branch 'origin/terencehill/short_sliders_fix'

balanceXonotic: updates by atheros

Some force updates and some mortar balance changes... it was way overpowered before. Also: Lets remove the hagar (g_start_weapon) and take its secondary and place it on the crylink. This needs more work (hagar secondary on crylink) but should be an improvement from how it's done now.

Fix length of some sliders in 2 panel dialogs, value field text was unreadable because too much compressed
These sliders were added to the menu intentionally with a minor size to reduce graphical slider length and let value field text overflow to occupy more space. With the "compact text" system this hack made things worse.

Fix the nex a little (To be exactly like what tZork wanted), and also the seeker -- needs moar changes though (Hold on tZork)

Merge remote branch 'origin/master' into samual/balancesamual

Add a "teamspawns" and "noteamspawns" feature to gametypefilter. I believe this is useful for maps that filter entities when team spawns are used. It can avoid adding each gametype manually in entities, and updating maps if a gametype is changed or added in Xonotic. I know about the "teams" and "noteams" switch, but these ignore team games that don't use team spawns and therefore a red / blue base (like TDM or DOM).

Rename treshold to threshold in 2 cvar names

Fix some typo in cvars descriptions

add a format spec comment

player stats: also send team scoreboard

Fix laser setting the last used ammo to 1 when reloading it after having switched to another weapon. We need to use a field float there that is always 0, or the ..float sets the last ammo instead of reporting no ammo.

Prettier name for DoReload (TriggerReload)

Write a useful comment

Share the final reload code that comes to mind. The reload code is for the most part shared now :)

Create a RELOADABLE weapon flag, and use it to prevent running useless shared code for weapons where it's not set

Fix Nexgun so that it has a cvar for the animation limit separate from autocharge limit - Also make it so that my balance no longer has a limit, and remove falloff. Also: Nerf velocity charge greatly.

Commonize even more code, now that its possible

Fix laser modelindex thingie (doesn't like reload's ammo float being 0)

Adapt the laser and its no-ammo usage too

Fix ammo types

Port new code to all weapons, part 2

Port new code to all weapons, part 1

Turn weapon load floats into a single float with an array. The new system should be ready, and porting it to all guns is next