Release Update

[gsfbuildbot]

This PR was Generated by a release of openHistorian

In general, when a local variable is assigned but never read, either the variable should be removed (if it is genuinely unused) or the code should be corrected to actually use it (if it was intended to participate in the logic). Here, omega is the angular frequency derived from nominalFreq. Since no later code reads omega in the provided method, the best fix that preserves existing behavior is to remove the declaration/assignment of omega. This eliminates the useless assignment without changing any program logic, because nothing depended on omega.

Concretely, in src/Gemstone.PhasorProtocols/SelCWS/RollingPhaseEstimationTest.cs, within TestNominalFrequency(), delete the line that declares and assigns omega (line 56). No additional imports, methods, or definitions are needed, and no other lines need to change.

To fix the issue, replace the if/else block that only assigns to sincVal with a single assignment using the conditional (?:) operator. This will condense the logic into one line, better expressing that sincVal is chosen between two values based solely on the condition, without changing functionality.

Concretely, in src/Gemstone.PhasorProtocols/SelCWS/RollingPhaseEstimator.cs, in the M-class filter section inside the for (int i = 0; i <= N; i++) loop (around lines 670–675), remove the multi-line if (Math.Abs(sincArg) < 1e-15D) ... else ... that assigns to sincVal, and replace it with a single line:

double sincVal = Math.Abs(sincArg) < 1e-15D ? 1.0D : Math.Sin(sincArg) / sincArg;

No new methods, imports, or definitions are required; this is a pure refactor of existing logic, keeping the exact same condition and branch expressions.