Networked crosshair

[nullsystem]

Description

• Since the change to string-based serialization is applied, it's now more straightforward to implement networked crosshair
• The networking is similar to how it's done for neo_name and neo_clantag as it's just strings passed around
• cl_neo_crosshair_network 0 = disable viewing other players crosshair, 1 = show other players crosshair
• "Other players crosshair" meaning when you're spectating the other person in 1st person while they ADS
• NeoUI settings - Slight alterations, added set to default button but also crosshair no longer part of the full-on reset button, so like the volume setting won't reset on full-on settings reset
• NeoUI settings - Added toggle for viewing networked crosshair
• NeoUI settings - Export/Import(/Default) buttons no longer restricted to only custom
• Improved default crosshair
• Tested with 2 computers that it's generally working

Toolchain

• Linux GCC Distro Native Gentoo/GCC 14.2.1

Linked Issues

• fixes [FEATURE] Networked crosshair #1187

[AdamTadeusz]

maybe instead of doing all those InstallChangeCallback s, pass the function as a parameter in the convar definition

[Rainyan]

Tested on LAN, seems to work well. But left some language nits that we might wanna fix before merging.

[AdamTadeusz]

Perhaps it would be better to give ConVars a max length value, or use the fmin and fmax as string length min and max

I think it should be easy to clamp len here to m_fMaxVal

[AdamTadeusz]

Seems to work well enough

[nullsystem]

@AdamTadeusz This prevents any convar that EX: uses float 0.0 to 1.0 (min + max) from being able to use properly, cause now it's only stored as "0" or "1" but "0".5 doesn't work at all

[AdamTadeusz]

I assumed ChangeStringValue was only used for chars, but that's an easy fix

[AdamTadeusz]

could also add FCVAR_NEVER_AS_STRING to every single float and int convar just in case (this is a joke)

[nullsystem]

Can't really be sure non-open part of SDK will always utilize FCVAR_NEVER_AS_STRING for int/float only convars or gets affected negatively by this change. I'd rather not alter tier1 library like this.