avoid shadowing warning

[neheb]

No description provided.

[kmilos]

size as the original input variable is used below outside the while's scope... Isn't this a local variable that needs to be renamed instead?

[neheb]

Sure. But it’s simpler to abuse the fact that it’s copied by value into the function. Modifications don’t make it outside

[Copilot]

Pull Request Overview

This PR removes variable shadowing by eliminating redundant declarations and renaming conflicting locals.

• Eliminates a shadowed size variable in userDataDecoder by assigning directly to the parameter.
• Renames the addrinfo* result variable in the HTTP function to avoid shadowing.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/quicktimevideo.cpp	Removed local const size declaration and now reuses the size parameter
src/http.cpp	Renamed result to r for addrinfo*, avoiding a naming conflict

Comments suppressed due to low confidence (2)

src/quicktimevideo.cpp:858

• [nitpick] The parameter size is still ambiguous after removing the shadowing declaration. Consider renaming the function parameter (e.g., remainingSize) or introducing a descriptively named local variable (e.g., chunkSize) to clarify intent and improve readability.

    size = buf.read_uint32(0, bigEndian);

src/http.cpp:204

• [nitpick] The variable name r is too short and unclear. Use a more descriptive identifier like addrinfoResult or resolvedInfo to improve maintainability and clarity.

    struct addrinfo* r;