Merge pull request #205 from technosoftware-gmbh/technosoftware-patch-1 #268
Annotations
10 warnings
tutorials/SampleCompany/Simple/SampleClient/MyUaClient.cs#L44
Provide an overridable implementation of Dispose(bool) on 'MyUaClient' or mark the type as sealed. A call to Dispose(false) should only clean up native resources. A call to Dispose(true) should clean up both managed and native resources. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1063)
|
tutorials/SampleCompany/Simple/SampleServer/UaServerPlugin.cs#L313
Change type of variable 'stream' from 'System.IO.Stream' to 'System.IO.FileStream' for improved performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1859)
|
tutorials/SampleCompany/Common/ConsoleUtils.cs#L102
In externally visible method 'string ConsoleUtils.ProcessCommandLine(TextWriter output, string[] args, OptionSet options, ref bool showHelp, string environmentPrefix, bool additionalArgs = false)', validate parameter 'output' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
tutorials/SampleCompany/Simple/SampleServer/SampleCompany.SampleServerNodeManager.cs#L168
Test for empty strings using 'string.Length' property or 'string.IsNullOrEmpty' method instead of an Equality check (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1820)
|
tutorials/SampleCompany/Common/ConsoleUtils.cs#L71
In externally visible method 'string ConsoleUtils.ProcessCommandLine(TextWriter output, string[] args, OptionSet options, ref bool showHelp, string environmentPrefix, bool additionalArgs = false)', validate parameter 'options' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
tutorials/SampleCompany/Simple/SampleServer/SampleCompany.SampleServerNodeManager.cs#L248
The behavior of 'BigInteger.Parse(string)' could vary based on the current user's locale settings. Replace this call in 'SampleServerNodeManager.CreateAddressSpace(IDictionary<NodeId, IList<IReference>>)' with a call to 'BigInteger.Parse(string, IFormatProvider)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
|
tutorials/SampleCompany/Common/ConsoleUtils.cs#L246
Modify 'CtrlCHandler' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
tutorials/SampleCompany/Simple/SampleServer/SampleCompany.SampleServerNodeManager.cs#L352
Modify 'CreateAddressSpace' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
tutorials/SampleCompany/Common/ApplicationMessageDlg.cs#L63
Modify 'ShowAsync' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
tutorials/SampleCompany/Simple/SampleServer/SampleCompany.SampleServerNodeManager.cs#L195
In externally visible method 'void SampleServerNodeManager.CreateAddressSpace(IDictionary<NodeId, IList<IReference>> externalReferences)', validate parameter 'externalReferences' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
This job succeeded
Loading