Skip to content

Commit 964c523

Browse files
mus65mus65
committed
fix IDE0031
https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0031
1 parent 8030503 commit 964c523

File tree

58 files changed

+186
-459
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

58 files changed

+186
-459
lines changed

‎src/Renci.SshNet/NetConfClient.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -299,11 +299,8 @@ protected override void Dispose(bool disposing)
299299

300300
if (disposing)
301301
{
302-
if (_netConfSession != null)
303-
{
304-
_netConfSession.Dispose();
305-
_netConfSession = null;
306-
}
302+
_netConfSession?.Dispose();
303+
_netConfSession = null;
307304
}
308305
}
309306

‎src/Renci.SshNet/Netconf/NetConfSession.cs

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -213,17 +213,11 @@ protected override void Dispose(bool disposing)
213213

214214
if (disposing)
215215
{
216-
if (_serverCapabilitiesConfirmed != null)
217-
{
218-
_serverCapabilitiesConfirmed.Dispose();
219-
_serverCapabilitiesConfirmed = null;
220-
}
216+
_serverCapabilitiesConfirmed?.Dispose();
217+
_serverCapabilitiesConfirmed = null;
221218

222-
if (_rpcReplyReceived != null)
223-
{
224-
_rpcReplyReceived.Dispose();
225-
_rpcReplyReceived = null;
226-
}
219+
_rpcReplyReceived?.Dispose();
220+
_rpcReplyReceived = null;
227221
}
228222
}
229223
}

‎src/Renci.SshNet/Session.cs

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1561,17 +1561,11 @@ internal void OnNewKeysReceived(NewKeysMessage message)
15611561
disposableClientCipher.Dispose();
15621562
}
15631563

1564-
if (_serverMac != null)
1565-
{
1566-
_serverMac.Dispose();
1567-
_serverMac = null;
1568-
}
1564+
_serverMac?.Dispose();
1565+
_serverMac = null;
15691566

1570-
if (_clientMac != null)
1571-
{
1572-
_clientMac.Dispose();
1573-
_clientMac = null;
1574-
}
1567+
_clientMac?.Dispose();
1568+
_clientMac = null;
15751569

15761570
// Update negotiated algorithms
15771571
_serverCipher = _keyExchange.CreateServerCipher(out _serverAead);

‎src/Renci.SshNet/SshClient.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -325,11 +325,8 @@ protected override void Dispose(bool disposing)
325325

326326
if (disposing)
327327
{
328-
if (_inputStream != null)
329-
{
330-
_inputStream.Dispose();
331-
_inputStream = null;
332-
}
328+
_inputStream?.Dispose();
329+
_inputStream = null;
333330

334331
_isDisposed = true;
335332
}

‎test/Renci.SshNet.Tests/Classes/Channels/ChannelDirectTcpipTest_Dispose_SessionIsConnectedAndChannelIsOpen.cs

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -46,17 +46,11 @@ public void Initialize()
4646
[TestCleanup]
4747
public void CleanUp()
4848
{
49-
if (_client != null)
50-
{
51-
_client.Dispose();
52-
_client = null;
53-
}
49+
_client?.Dispose();
50+
_client = null;
5451

55-
if (_listener != null)
56-
{
57-
_listener.Stop();
58-
_listener = null;
59-
}
52+
_listener?.Stop();
53+
_listener = null;
6054
}
6155

6256
private void Arrange()

‎test/Renci.SshNet.Tests/Classes/Channels/ChannelForwardedTcpipTest_Dispose_SessionIsConnectedAndChannelIsOpen.cs

Lines changed: 6 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -46,23 +46,14 @@ public void Initialize()
4646
[TestCleanup]
4747
public void CleanUp()
4848
{
49-
if (_remoteListener != null)
50-
{
51-
_remoteListener.Stop();
52-
_remoteListener = null;
53-
}
49+
_remoteListener?.Stop();
50+
_remoteListener = null;
5451

55-
if (_channelThread != null)
56-
{
57-
_channelThread.Join();
58-
_channelThread = null;
59-
}
52+
_channelThread?.Join();
53+
_channelThread = null;
6054

61-
if (_channel != null)
62-
{
63-
_channel.Dispose();
64-
_channel = null;
65-
}
55+
_channel?.Dispose();
56+
_channel = null;
6657
}
6758

6859
private void Arrange()

‎test/Renci.SshNet.Tests/Classes/Channels/ChannelTest_Dispose_SessionIsConnectedAndChannelIsOpen_EofNotReceived_SendEofInvoked.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,8 @@ public class ChannelTest_Dispose_SessionIsConnectedAndChannelIsOpen_EofNotReceiv
3131
[TestCleanup]
3232
public void TearDown()
3333
{
34-
if (_channelClosedEventHandlerCompleted != null)
35-
{
36-
_channelClosedEventHandlerCompleted.Dispose();
37-
_channelClosedEventHandlerCompleted = null;
38-
}
34+
_channelClosedEventHandlerCompleted?.Dispose();
35+
_channelClosedEventHandlerCompleted = null;
3936
}
4037

4138
protected override void SetupData()

‎test/Renci.SshNet.Tests/Classes/Channels/ChannelTest_Dispose_SessionIsConnectedAndChannelIsOpen_EofReceived.cs

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -91,23 +91,14 @@ protected override void SetupMocks()
9191
[TestCleanup]
9292
public void TearDown()
9393
{
94-
if (_channelClosedReceived != null)
95-
{
96-
_channelClosedReceived.Dispose();
97-
_channelClosedReceived = null;
98-
}
99-
100-
if (_raiseChannelCloseReceivedThread != null)
101-
{
102-
_raiseChannelCloseReceivedThread.Join();
103-
_raiseChannelCloseReceivedThread = null;
104-
}
105-
106-
if (_channelClosedEventHandlerCompleted != null)
107-
{
108-
_channelClosedEventHandlerCompleted.Dispose();
109-
_channelClosedEventHandlerCompleted = null;
110-
}
94+
_channelClosedReceived?.Dispose();
95+
_channelClosedReceived = null;
96+
97+
_raiseChannelCloseReceivedThread?.Join();
98+
_raiseChannelCloseReceivedThread = null;
99+
100+
_channelClosedEventHandlerCompleted?.Dispose();
101+
_channelClosedEventHandlerCompleted = null;
111102
}
112103

113104
protected override void Arrange()

‎test/Renci.SshNet.Tests/Classes/Connection/ProtocolVersionExchangeTest_ConnectionClosedByServer_NoDataSentByServer.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,8 @@ public void Setup()
3535
[TestCleanup]
3636
public void Cleanup()
3737
{
38-
if (_server != null)
39-
{
40-
_server.Dispose();
41-
_server = null;
42-
}
38+
_server?.Dispose();
39+
_server = null;
4340

4441
if (_client != null)
4542
{

‎test/Renci.SshNet.Tests/Classes/Connection/ProtocolVersionExchangeTest_ServerResponseContainsNullCharacter.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,8 @@ public void Setup()
3838
[TestCleanup]
3939
public void Cleanup()
4040
{
41-
if (_server != null)
42-
{
43-
_server.Dispose();
44-
_server = null;
45-
}
41+
_server?.Dispose();
42+
_server = null;
4643

4744
if (_client != null)
4845
{

0 commit comments

Comments
 (0)