Skip to content

Misc refactors to appease the compiler#1078

Open
anf3is wants to merge 2 commits into
ReactiveDrop:reactivedrop_betafrom
anf3is:rf/fix-compiler-warnings
Open

Misc refactors to appease the compiler#1078
anf3is wants to merge 2 commits into
ReactiveDrop:reactivedrop_betafrom
anf3is:rf/fix-compiler-warnings

Conversation

@anf3is

@anf3is anf3is commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Most of these changes are simple renames to prevent name collisions between:

  1. Function parameters and local declarations
  2. Local declarations and loop variables
  3. Variables in nested and outer loops

Nearly all renames were performed using LSP rename.

@anf3is anf3is force-pushed the rf/fix-compiler-warnings branch 3 times, most recently from 34dc5dd to 8fa2cc0 Compare June 11, 2026 14:16
anf3is added 2 commits June 11, 2026 16:49
move type into for

rename filter.GetRecipientCount

rename RGB -> RGBnext

rename for loop i -> j

rename KeyValues

fix i visiblity

remove unused attackerTeam

rename index -> iIndex

rename pPRecipient

rename mass -> massHeld

remove unused result of V_snprintf

inline fPhysObjectMass

rename pPlayer -> pPClient

inline count

fix sit visibility (while -> for)

inline dir

rename te -> tempEntity

rename pTartget -> pTarget2

rename a -> ia

rename inc -> incAdj

rename color -> blockedColor

move x declaration inside scopes where it is used

rename hFunc -> hScopedFunc

rename value -> position

rename ptag -> ptagOriginal

rename flGain

rename origin/angles -> shadowOrigin/shadowAngles

rename origin -> hanging_origin

rename args -> scriptArgs

rename taskID -> taskID2

rename goal -> lateralCoverGoal

add comment for coverGoal

remove second compute flTotalDist

suppress warning p4 hides global declaration
inline getstat

move dir and angles inside

rename overshadowed i -> j

move int i inside for

rename te -> tempEntity

inline effects

rename observerTarget

inline bVisible

specify ParticleEffect and ControlPoint variables

rename idx

rename iBuffAttachment

rename outer i

remove unused var tPos

rename p -> p2

move int declare into for

rename info -> infoNew

fix i -> t

fix i -> j

rename i -> idx

rename pMaterialWf

rename msg -> kvMsg

move white and blue

remove duplicate fDelay

rename buf ->pathBuf

remove duplicate nNumHistogramBuckets

rename textTall -> smallTextTall

rename kv -> kvActive

add scope for player helper

rename matrix -> matrix2

rename i -> tail

renames in bWireframe

rename msg -> kvMsg

rename pButton -> phButton

rename pProfile -> pProfilePref

rename uvScale -> uvScale2

inline GetModelType

rename pEntry -> pEntry2

rename nNumShadows -> nNumBlobbyShadows

rename angle -> angle_yaw/angle_pitch

rename input -> x_input

rename w/h -> text_w/text_h

rename iRenderCache -> tail

remove unused GetBounds

add ifndef INFESTED_DLL for hudCloseCaption definition

remove duplicated definition of fBracketTime

rename entry -> repeatEntry

rename target -> targetI

rename entry -> repeatEntry

rename pParticle/offset -> pFlameParticle/randomOffset

remove unused variable offset

remove second local player check

remove second local base player check

suppress warning p4 hides global declaration
@anf3is anf3is force-pushed the rf/fix-compiler-warnings branch from 8fa2cc0 to b22042c Compare June 11, 2026 14:56
@anf3is anf3is marked this pull request as ready for review June 11, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant