Merge branch 'drjaska/preinc' into 'master'

micro-optimization chore: in for-loops change all post-{in,de}crements to pre-{in,de}crements

See merge request xonotic/xonotic-data.pk3dir!1241

micro-optimization chore: in for-loops change all post-{in,de}crements to pre-{in,de}crements

Fix spectator camera having a duplicate mode if 3rd person camera is enabled (chase_active 1)

Transifex autosync

Merge branch 'terencehill/playerlist_layout' into 'master'

Menu: make player list in the Server Info dialog more readable

See merge request xonotic/xonotic-data.pk3dir!1258

Make togglemenu not close the menu if called when not connected. togglemenu 0 always closes the menu even in this case

Merge branch 'terencehill/menu_registries' into 'master'

Menu: remove useless registries

See merge request xonotic/xonotic-data.pk3dir!1261

Merge branch 'bones_was_here/sv_legacy_bbox_expand_4' into 'master'

Remove legacy Quake bbox expansion & related item placement and networking upgrades

See merge request xonotic/xonotic-data.pk3dir!1234

Remove legacy Quake bbox expansion: update pipeline hash

Remove legacy Quake bbox expansion: DOM

Remove legacy Quake bbox expansion: KA

Gives ball a symmetrical bbox suitable for sv_legacy_bbox_expand 0

Remove legacy Quake bbox expansion: KH

Remove legacy Quake bbox expansion: CTF

This adds a feature: if the flag is dropped in a place its bbox doesn't
fit, the player's bbox (crouched or full size) is used as a fallback.
This prevents it getting stuck so physics can still move it, and it will
be expanded to full size if it's moved to a place where it does fit.

This MR adds this to flags only, because they already had half the code.
I'll add it to other big ents that players can drop (keys, balls,
powerups) in another MR.

items: use correct bboxes in CSQC

This only needs to send 2 bits because there's now only 3 item bbox sizes.
Fixes some prediction errors for items with physics (loot).

items: remove blind nudging of items during spawn

In some cases this isn't enough to fix a bad placement, in rare cases it
can cause a bad placement because it doesn't consider the brushwork
near the item.

It's replaced by smart nudging (DP_QC_NUDGEOUTOFSOLID).

Remove legacy Quake bbox expansion: items (and buffs)

Adds a Small 48x48x48 item bbox which is used for 5 and 25 health and
armor items. This is the biggest horizontal size that fits in the 25h
alcove on finalrage.

Changes powerup, buff and mega pickups to all have the same height.
This will make it cheaper to network item bboxes to CSQC.

Remove legacy Quake bbox expansion: bot navigation

Remove legacy Quake bbox expansion: physics

Remove legacy Quake bbox expansion: bmodel entities

Removes duplication of the box + '1 1 1' and boxesoverlap code.
This is handled in WarpZoneLib_ExactTrigger_Touch().

Disable sv_legacy_bbox_expand

This gives QC full control of bboxes
and makes SVQC and CSQC behaviour consistent.

Groups all the sv_gameplayfix cvars together in xonotic-server.cfg.

Adds a note about sv_gameplayfix_droptofloorstartsolid (cvar for unused
engine feature).

Rewrite the QC port of droptofloor()

This uses DP_QC_NUDGEOUTOFSOLID to achieve better results (especially
noticeable with sv_legacy_bbox_expand 0) in less CPU time.

Documents the design.
Removes legacy code paths.
Adds warnings for badly placed items.
Implements Quake-compliant behaviour for each BSP format.

See also: https://gitlab.com/xonotic/darkplaces/-/merge_requests/144

Removes hacky workaround for #2774 and thus depends on !1245

Signed-off-by: bones_was_here <bones_was_here@xonotic.au>

monsters: don't cast light when following a player

This made FPS too low for players using integrated graphics when
pokenades are spammed. Also was a bit weird, they're not objectives or
powerups.

monsters: ensure they can't be stuck when spawned

If too stuck to fix the spawn fails.

Closes #1657
Closes #2645

monsters: stop following player if they spectate in FFA gametypes

Fixes #2705

In teamplay the DIFF_TEAM check already handled this.

monsters: minor movement refactor

Should probably update angles field according to destination _before_
using it to update velocity.

monsters: try to avoid danger

Also makes them avoid falling off platforms and into holes, and try to
find another path, when NOT chasing an enemy.
This makes them more useful for guarding an area, especially on space
maps.

monsters: improve following and wandering behaviour

Makes monsters wander around the player they're following when nearby,
and wander towards the player using the follow update interval when
further away.
This prevents them clustering tightly around the player and increases
their chance of overcoming trivial obstacles.

Fixes trying to walk into walls.

Server list: check libcurl is loaded before reporting xonstat as available

See also 7ab7da87c6e4cdee0c74da001faeaf493dc07022 derp.

Merge branch 'bones_was_here/movetype_toss_lagfix' into 'master'

Fix 100% cpu when entities using TOSS-based movetypes get stuck

Closes #2774

See merge request xonotic/xonotic-data.pk3dir!1245

Fix 100% cpu when entities using TOSS-based movetypes get stuck

This is a bug in sv_gameplayfix_slidemoveprojectiles which gives nicer
physics interactions but didn't abort if the entity is immovable.

Server list: only report xonstat as available if IDs are supported

Merge branch 'bones_was_here/checkextension' into 'master'

Add engine extension checks at VM startup and some fallbacks

See merge request xonotic/xonotic-data.pk3dir!1225

Optimize _LOG_HEADER to avoid bloating compilation output files and reduce program size

Add fallback path for when engine lacks DP_QC_FINDBOX

Improve visibility of warnings, related cleanup and fixes

The reason for strcat1n is now fixed properly by darkplaces@b0abeba2 and a986d478

Double printing of backtraces with `developer 1` is fixed.
Failure to print fatal messages with `developer 1` is fixed.

Made the pipeline's `sv_cmd runtest` accept results prefixed with (eg)
"[::SVQC::INFO]" (as long as they end with exactly "All tests OK"), also
fixed incorrect hash being printed in green and fixed "PASS: unbound
variable" error.

Add engine extension checks at VM startup and a nudgeoutofsolid fallback

This allows printing proper messages immediately rather than crashing
later or silently failing to download maps or support player IDs, which
may prevent broken 3rd party packages.

Implements a fallback to WarpZoneLib_MoveOutOfSolid() for when the new
DP_QC_NUDGEOUTOFSOLID isn't available.  Adds a "wasn't-stuck" `-1`
result to WarpZoneLib_MoveOutOfSolid(), to prevent warning spam in the
fallback path.

items: Apply vertical offset on Q3 maps to match Q3 bbox floor height

The lack of this caused high-placed items to be stuck in a "ceiling"
brush in a few cases (problematic for droptofloor). It also meant
suspended items were too high, causing cases where a jump pad sent the
player slightly under the item, with nothing they could do to reach it.

Merge branch 'terencehill/respawn_ghosts_keepcolors_enabled' into 'master'

Display respawn ghost with player's color instead of white

See merge request xonotic/xonotic-data.pk3dir!1263

Remove .obj entity model orientation hack

This seems to be causing an orientation problem not fixing one.

Fix .gitattributes

A later line overrides an earlier line.

Use C syntax highlighting for QC files on gitlab

Fix wrong respawn ghost colors with cl_forceuniqueplayercolors 1

Display respawn ghost with player's color instead of white

Merge branch 'bones_was_here/sv_playerthink_optimise' into 'master'

Move some code that only needs to run once per frame out of PlayerP*Think()

See merge request xonotic/xonotic-data.pk3dir!1238

Merge branch 'terencehill/intermission_restart_fix' into 'master'

Fix restart applying different gametype if called after gametype vote

See merge request xonotic/xonotic-data.pk3dir!1256

Menu: remove useless registries and related code (TempEntities, LinkedEntities, C2S_Protocol) and fix a couple registries not matching the corresponding client/server versions (Buff entry was missing from the RadarIcons and Waypoints registries)

Merge branch 'drjaska/teleport-minspeed' into 'master'

Mininum velocity for teleporter exits

See merge request xonotic/xonotic-data.pk3dir!1165

Mininum velocity for teleporter exits

Fix string length checks in world.qc and add VM_TEMPSTRING_MAXSIZE

Merge branch 'Spike29/balance_council_vote19' into 'master'

Apply the balance council vote 19

See merge request xonotic/xonotic-data.pk3dir!1260

Apply the balance council vote 19

Merge branch 'bones_was_here/maplist_fixes' into 'master'

Map list fixes

See merge request xonotic/xonotic-data.pk3dir!1247

Fix GUI map voting with g_maplist_shuffle <= 0 && g_maplist_selectrandom 0

MaplistMethod_Iterate() returned the same map every time because
Map_Current wasn't updated.

Documents g_maplist_shuffle -1.

g_maplist: refactor initialisation

Reduces code duplication.

Checks/initialises once when adding maps to the voting screen, instead
of doing it again for each map added.

Always uses gametype-specific g_maplist generation, instead of sometimes
including maps from other gametypes (depending which code path was used).

Updates some declarations and removes an unused global.

g_maplist: update with `gametype` command, as well as GUI gametype voting

This fixes the need to manually set g_maplist to \"\" after running the
gametype command on a votable server.

This is consistent with GUI gametype voting (sv_vote_gametype 1)
behaviour and shares its code to reduce duplication.

mapinfo: fix and rename noautomaplist flag to donotwant

This flag was ignored in the code paths used on most servers, and its
name and behaviour were not ideal for its intended use, see
7d48e976c8a9b3f80350b2ad41d9feec2797cf76 description which seems clear
that it's for allowing a certain kind of map to be available on the
server but not be included in the usual voting pool.

This updated implementation omits the flagged maps from GUI voting and
prevents them being selected by other GotoNextMap()/GetNextMap() code
paths, unless there aren't enough viable maps in which case they
can still be used as fallbacks.

The flagged maps are always listed in CLI and may be nominated with
`suggestmap` or vcalled with `gotomap` or `nextmap`.

Q3 compat: implement a 1ms fallback traveltime for all movers

This fixes the doors failing to open on q3df map sdc-uk_08 which uses
trigger_multiple -> func_button.speed = -1 -> func_door
Buttons trigger targets at the end of their movement and that one didn't
move at all.

Signed-off-by: bones_was_here <bones_was_here@xonotic.au>

Merge branch 'terencehill/registry_check' into 'master'

Fix REGISTRY_CHECK

See merge request xonotic/xonotic-data.pk3dir!1259

Merge branch 'drjaska/ca_prevent_stalemate_2' into 'master'

Separated `g_ca_prevent_stalemate`'s survivor count and health checking.

See merge request xonotic/xonotic-data.pk3dir!1189

Separated `g_ca_prevent_stalemate`'s survivor count and health checking.

Remove a few dangling semicolons

Fix REGISTRY_CHECK not checking all elements of a huge registry like the Notifications registry

Code is optimized to have basically the same performance as before while adding string length checks.

Merge MR 'func_door and func_plat fixes and Q3 compatibility'

See merge request xonotic/xonotic-data.pk3dir!1242

Merge branch 'drjaska/spawnfunc-item_buff_name' into 'master'

restore item_buff_speed and item_buff_invisibility spawnfuncs

See merge request xonotic/xonotic-data.pk3dir!1160

Transifex autosync

Menu: make player list in the Server Info dialog more readable

playerlist.qc: remove some useless operations in 2 draw_Text calls; while at it clean up code

Fix #2893 "cl_zoomfactor 1 results in division by zero"; also simplify the current_viewzoom check (almost_equals performs redundant checks in this case)

Transifex autosync

Transifex autosync

Transifex autosync

Merge branch 'drjaska/hashtest' into 'master'

Create an usable tool for local hash testing

See merge request xonotic/xonotic-data.pk3dir!1150

Create an usable tool for local hash testing

Transifex autosync

Fix and make  indentation consistent in the Overkill weapon files

Use cvar_set

Freeze Tag: fix players getting 1 point for a teamkill (they get -1 points now) and not getting -1 points if killed by a map entity (e.g. triggerhurt)

hud_postprocessing: disable the postprocess shader when not in use.

Works best with upcoming r_skipblend.

Fix some damagetext accumulation edge cases

Fixes cl_damagetext_accumulate_alpha_rel 1 and
cl_damagetext_accumulate_lifetime 0 not being able to opt
out of DT accumulation

Merge branch 'drjaska/damagetext' into 'master'

DamageText code quality changes and new DT lifetime and accumulation lifetime cvars

See merge request xonotic/xonotic-data.pk3dir!1249

DamageText code quality changes and new DT lifetime and accumulation lifetime cvars

Update pipeline for DP engine upgrade

The FIXME was fixed (or at least worked around) in DP's Makefile, see
-mno-avx and its comments.

Transifex autosync

Merge branch 'terencehill/playerstats_update' into 'master'

Minor updates to playerstats

See merge request xonotic/xonotic-data.pk3dir!1255

Transifex autosync

Transifex autosync

Apply voted game type on map switch, not on game type vote end so that the "restart" command, if called, properly restarts the map applying the current game type

Don't send match stats if game ends during warmup (xonstats doesn't register them anyway)

Send total number of rounds played to xonstats

Transifex autosync

Transifex autosync

Merge branch 'drjaska/shotorigincleanup' into 'master'

Allow for negative z offsets for shot origin

See merge request xonotic/xonotic-data.pk3dir!1253

Transifex autosync

cleanup and a few comments

Transifex autosync

Transifex autosync

Transifex autosync

Transifex autosync

Transifex autosync

Don't allow negative values for the y component of the shot origin offset because the gun can't be aligned to the left by default. Thanks to this change it's possible to restore precision of the y and z components back to 0.25