You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Later, when the user supplies their own Configuration we don't need that anymore so it's wasted cycles and memory.
Suggestion: Create a new instance of AutoDetectConfiguration in the getter of CommandLineArguments.Configuration when the backing field _configuration is null and assign it to the field before returning it.
The text was updated successfully, but these errors were encountered:
We always create a new instance of
AutoDetectConfiguration
inCommandLineArguments
:AtleX.CommandLineArguments/src/AtleX.CommandLineArguments/CommandLineArguments.cs
Line 16 in 0d2cde1
Later, when the user supplies their own Configuration we don't need that anymore so it's wasted cycles and memory.
Suggestion: Create a new instance of
AutoDetectConfiguration
in the getter ofCommandLineArguments.Configuration
when the backing field_configuration
is null and assign it to the field before returning it.The text was updated successfully, but these errors were encountered: