Fix CS0642 - Possible mistaken empty statement

[Mervill]

About the PR

This took me a minute to disambiguate so I'll recount the tale here.

This is the original statement:

using(args.PushGroup(nameof(GasTankComponent)));
if (args.IsInDetailsRange)
   ...

This statement throws a warning on the ending semicolon: CS0642 Possible mistaken empty statement. What it's trying to tell us is that the above statement actually becomes:

using(args.PushGroup(nameof(GasTankComponent)));
{
    // empty using block!
}
if (args.IsInDetailsRange)
   ...

What the code is actually trying to do is use a variant of the using statement that creates an implied using scope that is equal to the (rest of) the function scope (i.e. the Dispose() is called just before the function returns). That variant of the using statement must explicitly assign to a variable or else the using statement is actually treated like a using statement with a single-line code block following it (i.e. the example in the code example above).

So we need to explicitly assign to something, but we don't actually need the result of PushGroup so we can just assign to the discard value.

This properly sets the scope of the using statement to the whole function and clears the warning.

I think C# probably has a bit too much syntax these days, but maybe I'm just old.

Requirements

• I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
• I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase