Skip to content

Commit 11bba62

Browse files
author
David R. MacIver
committed
clean up comments for AmqpTcpEndpoint to reflect the fact that it no longer carries a Protocol
1 parent f3dbcef commit 11bba62

File tree

1 file changed

+10
-14
lines changed

1 file changed

+10
-14
lines changed

projects/client/RabbitMQ.Client/src/client/api/AmqpTcpEndpoint.cs

Lines changed: 10 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,9 @@
6161
namespace RabbitMQ.Client
6262
{
6363
///<summary>Represents a TCP-addressable AMQP peer, including the
64-
///protocol variant to use, and a host name and port
65-
///number.</summary>
64+
/// a host name and port number. Where the default protocol port
65+
/// should be used, this is represented by a port number of -1
66+
/// </summary>
6667
///<para>
6768
/// Some of the constructors take, as a convenience, a System.Uri
6869
/// instance representing an AMQP server address. The use of Uri
@@ -109,8 +110,7 @@ public SslOption Ssl
109110

110111
///<summary>Construct an AmqpTcpEndpoint with the given
111112
///hostname, port number and ssl option. If the port
112-
///number is -1, the default port number for the IProtocol
113-
///will be used.</summary>
113+
///number is -1 </summary>
114114
public AmqpTcpEndpoint(string hostName, int portOrMinusOne, SslOption ssl)
115115
{
116116
m_hostName = hostName;
@@ -119,17 +119,14 @@ public AmqpTcpEndpoint(string hostName, int portOrMinusOne, SslOption ssl)
119119
}
120120

121121
///<summary>Construct an AmqpTcpEndpoint with the given
122-
/// hostname, and port number. If the port number is
123-
///-1, the default port number for the IProtocol will be
124-
///used.</summary>
122+
/// hostname, and port number. </summary>
125123
public AmqpTcpEndpoint(string hostName, int portOrMinusOne) :
126124
this(hostName, portOrMinusOne, new SslOption())
127125
{
128126
}
129127

130128
///<summary>Construct an AmqpTcpEndpoint with the given
131-
///hostname, using the default port for the
132-
///IProtocol.</summary>
129+
///hostname, using the default port </summary>
133130
public AmqpTcpEndpoint(string hostName) :
134131
this(hostName, -1)
135132
{
@@ -143,7 +140,7 @@ public AmqpTcpEndpoint() :
143140
}
144141

145142
///<summary>Construct an AmqpTcpEndpoint with the given
146-
///IProtocol, Uri and ssl options.</summary>
143+
/// Uri and ssl options.</summary>
147144
///<remarks>
148145
/// Please see the class overview documentation for
149146
/// information about the Uri format in use.
@@ -174,8 +171,8 @@ public override string ToString()
174171
return "amqp://" + HostName + ":" + Port;
175172
}
176173

177-
///<summary>Compares this instance by value (protocol,
178-
///hostname, port) against another instance</summary>
174+
///<summary>Compares this instance by value (hostname, port)
175+
///against another instance</summary>
179176
public override bool Equals(object obj)
180177
{
181178
AmqpTcpEndpoint other = obj as AmqpTcpEndpoint;
@@ -201,8 +198,7 @@ public override int GetHashCode()
201198
/// If the address string passed in contains ":", it is split
202199
/// into a hostname and a port-number part. Otherwise, the
203200
/// entire string is used as the hostname, and the port-number
204-
/// is set to -1 (meaning the default number for the protocol
205-
/// variant specified).
201+
/// is set to the default.
206202
///</remarks>
207203
public static AmqpTcpEndpoint Parse(string address) {
208204
int index = address.IndexOf(':');

0 commit comments

Comments
 (0)