Skip to content

Commit

Permalink
fixing some naming and typos that caused wrong value to be updated (#…
Browse files Browse the repository at this point in the history
…13031)

Motivation:

I accidentally find some typos in the transport-udt module, and two of them actually can affect the correctness of execution.

Modification:

Only changed some variable namings.
Variables in some setters in DefaultUdtChannelConfig.java and DefaultUdtServerChannelConfig.java are named incorrectly.

Result:

Now, DefaultUdtChannelConfig won't set SystemReceiveBufferSize and SystemSendBufferSize incorrectly.
  • Loading branch information
zazf committed Dec 4, 2022
1 parent 22d3151 commit 7cc8428
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
Expand Up @@ -214,7 +214,7 @@ public int getSystemReceiveBufferSize() {
}

@Override
public UdtChannelConfig setSystemSendBufferSize(
public UdtChannelConfig setSystemReceiveBufferSize(
final int systemReceiveBufferSize) {
this.systemReceiveBufferSize = systemReceiveBufferSize;
return this;
Expand All @@ -233,7 +233,7 @@ public UdtChannelConfig setProtocolSendBufferSize(
}

@Override
public UdtChannelConfig setSystemReceiveBufferSize(
public UdtChannelConfig setSystemSendBufferSize(
final int systemSendBufferSize) {
this.systemSendBufferSize = systemSendBufferSize;
return this;
Expand Down
Expand Up @@ -128,15 +128,15 @@ public UdtServerChannelConfig setSoLinger(final int soLinger) {

@Override
public UdtServerChannelConfig setSystemReceiveBufferSize(
final int systemSendBufferSize) {
super.setSystemReceiveBufferSize(systemSendBufferSize);
final int systemReceiveBufferSize) {
super.setSystemReceiveBufferSize(systemReceiveBufferSize);
return this;
}

@Override
public UdtServerChannelConfig setSystemSendBufferSize(
final int systemReceiveBufferSize) {
super.setSystemSendBufferSize(systemReceiveBufferSize);
final int systemSendBufferSize) {
super.setSystemSendBufferSize(systemSendBufferSize);
return this;
}

Expand Down

0 comments on commit 7cc8428

Please sign in to comment.