First round of code cleanups.

Author: stebet

_site

@@ -65,7 +65,7 @@ public AmqpTimestamp(long unixTime) : this()
         /// <summary>
         /// Unix time.
         /// </summary>
-        public long UnixTime { get; private set; }
+        public long UnixTime { get; }
 
         public bool Equals(AmqpTimestamp other) => UnixTime == other.UnixTime;
 

projects/RabbitMQ.Client/client/api/AmqpTimestamp.cs

@@ -91,22 +91,22 @@ public BasicGetResult(ulong deliveryTag, bool redelivered, string exchange, stri
         /// <summary>
         /// Retrieves the Basic-class content header properties for this message.
         /// </summary>
-        public IBasicProperties BasicProperties { get; private set; }
+        public IBasicProperties BasicProperties { get; }
 
         /// <summary>
         /// Retrieves the body of this message.
         /// </summary>
-        public ReadOnlyMemory<byte> Body { get; private set; }
+        public ReadOnlyMemory<byte> Body { get; }
 
         /// <summary>
         /// Retrieve the delivery tag for this message. See also <see cref="IModel.BasicAck"/>.
         /// </summary>
-        public ulong DeliveryTag { get; private set; }
+        public ulong DeliveryTag { get; }
 
         /// <summary>
         /// Retrieve the exchange this message was published to.
         /// </summary>
-        public string Exchange { get; private set; }
+        public string Exchange { get; }
 
         /// <summary>
         /// Retrieve the number of messages pending on the queue, excluding the message being delivered.
@@ -115,17 +115,17 @@ public BasicGetResult(ulong deliveryTag, bool redelivered, string exchange, stri
         /// Note that this figure is indicative, not reliable, and can
         /// change arbitrarily as messages are added to the queue and removed by other clients.
         /// </remarks>
-        public uint MessageCount { get; private set; }
+        public uint MessageCount { get; }
 
         /// <summary>
         /// Retrieve the redelivered flag for this message.
         /// </summary>
-        public bool Redelivered { get; private set; }
+        public bool Redelivered { get; }
 
         /// <summary>
         /// Retrieve the routing key with which this message was published.
         /// </summary>
-        public string RoutingKey { get; private set; }
+        public string RoutingKey { get; }
 
         /// <inheritdoc />
         public void Dispose()

projects/RabbitMQ.Client/client/api/BasicGetResult.cs

@@ -79,17 +79,17 @@ public PublicationAddress(string exchangeType, string exchangeName, string routi
         /// <summary>
         /// Retrieve the exchange name.
         /// </summary>
-        public string ExchangeName { get; private set; }
+        public string ExchangeName { get; }
 
         /// <summary>
         /// Retrieve the exchange type string.
         /// </summary>
-        public string ExchangeType { get; private set; }
+        public string ExchangeType { get; }
 
         /// <summary>
         ///Retrieve the routing key.
         /// </summary>
-        public string RoutingKey { get; private set; }
+        public string RoutingKey { get; }
 
         /// <summary>
         /// Parse a <see cref="PublicationAddress"/> out of the given string,

projects/RabbitMQ.Client/client/api/PublicationAddress.cs

@@ -52,17 +52,17 @@ public QueueDeclareOk(string queueName, uint messageCount, uint consumerCount)
         /// <summary>
         /// Consumer count.
         /// </summary>
-        public uint ConsumerCount { get; private set; }
+        public uint ConsumerCount { get; }
 
         /// <summary>
         /// Message count.
         /// </summary>
-        public uint MessageCount { get; private set; }
+        public uint MessageCount { get; }
 
         /// <summary>
         /// Queue name.
         /// </summary>
-        public string QueueName { get; private set; }
+        public string QueueName { get; }
 
         public static implicit operator string(QueueDeclareOk declareOk)
         {

projects/RabbitMQ.Client/client/api/QueueDeclareOk.cs

@@ -67,32 +67,32 @@ public ShutdownEventArgs(ShutdownInitiator initiator, ushort replyCode, string r
         /// <summary>
         /// Object causing the shutdown, or null if none.
         /// </summary>
-        public object Cause { get; private set; }
+        public object Cause { get; }
 
         /// <summary>
         /// AMQP content-class ID, or 0 if none.
         /// </summary>
-        public ushort ClassId { get; private set; }
+        public ushort ClassId { get; }
 
         /// <summary>
         /// Returns the source of the shutdown event: either the application, the library, or the remote peer.
         /// </summary>
-        public ShutdownInitiator Initiator { get; private set; }
+        public ShutdownInitiator Initiator { get; }
 
         /// <summary>
         /// AMQP method ID within a content-class, or 0 if none.
         /// </summary>
-        public ushort MethodId { get; private set; }
+        public ushort MethodId { get; }
 
         /// <summary>
         /// One of the standardised AMQP reason codes. See RabbitMQ.Client.Framing.*.Constants.
         /// </summary>
-        public ushort ReplyCode { get; private set; }
+        public ushort ReplyCode { get; }
 
         /// <summary>
         /// Informative human-readable reason text.
         /// </summary>
-        public string ReplyText { get; private set; }
+        public string ReplyText { get; }
 
         /// <summary>
         /// Override ToString to be useful for debugging.

projects/RabbitMQ.Client/client/api/ShutdownEventArgs.cs

@@ -163,7 +163,7 @@ public X509CertificateCollection Certs
         /// </summary>
         internal SslProtocols UseFallbackTlsVersions()
         {
-            this.Version = SslProtocols.Tls12;
+            Version = SslProtocols.Tls12;
             return Version;
         }
     }

projects/RabbitMQ.Client/client/api/SslOption.cs

@@ -45,10 +45,10 @@ public BaseExceptionEventArgs(Exception exception)
 
         ///<summary>Access helpful information about the context in
         ///which the wrapped exception was thrown.</summary>
-        public IDictionary<string, object> Detail { get; private set; }
+        public IDictionary<string, object> Detail { get; }
 
         ///<summary>Access the wrapped exception.</summary>
-        public Exception Exception { get; private set; }
+        public Exception Exception { get; }
 
         public IDictionary<string, object> UpdateDetails(IDictionary<string, object> other)
         {

projects/RabbitMQ.Client/client/events/CallbackExceptionEventArgs.cs

@@ -46,6 +46,6 @@ public ConnectionBlockedEventArgs(string reason)
         /// <summary>
         /// Access the reason why connection is blocked.
         /// </summary>
-        public string Reason { get; private set; }
+        public string Reason { get; }
     }
 }

projects/RabbitMQ.Client/client/events/ConnectionBlockedEventArgs.cs

@@ -40,6 +40,6 @@ public ConnectionRecoveryErrorEventArgs(Exception ex)
             Exception = ex;
         }
 
-        public Exception Exception { get; private set; }
+        public Exception Exception { get; }
     }
 }