[avro] branch master updated: AVRO-3361 simplify if statement (#1512)

[mg...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mgrigorov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/avro.git


The following commit(s) were added to refs/heads/master by this push:
     new e57df5e  AVRO-3361 simplify if statement (#1512)
e57df5e is described below

commit e57df5ea723fe403ab17d012180564ffbce1f993
Author: Kyle Schoonover <ky...@minmaxcorp.com>
AuthorDate: Fri Feb 4 01:31:51 2022 -0800

    AVRO-3361 simplify if statement (#1512)
    
    * AVRO-3360 Updated XML documentation
    
    * Revert "AVRO-3360 Updated XML documentation"
    
    This reverts commit b8601c072a5083380d30b580804dd0908b8cf4cc.
    
    * AVRO-3361 simplify if statement
    
    * AVRO-3361: Simplify boolean expression
    
    Co-authored-by: Kyle T. Schoonover <Ky...@nordstrom.com>
    Co-authored-by: Martin Grigorov <ma...@users.noreply.github.com>
---
 lang/csharp/src/apache/main/File/NullCodec.cs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/lang/csharp/src/apache/main/File/NullCodec.cs b/lang/csharp/src/apache/main/File/NullCodec.cs
index 78803dd..295d33a 100644
--- a/lang/csharp/src/apache/main/File/NullCodec.cs
+++ b/lang/csharp/src/apache/main/File/NullCodec.cs
@@ -27,7 +27,7 @@ namespace Avro.File
     public class NullCodec : Codec
     {
         /// <summary>
-        /// Initializes a new instance of the <see cref="NullCodec"/> class.
+        /// Initializes a new instance of the <see cref="NullCodec" /> class.
         /// </summary>
         public NullCodec() { }
 
@@ -59,9 +59,7 @@ namespace Avro.File
         /// <inheritdoc/>
         public override bool Equals(object other)
         {
-            if (this == other)
-                return true;
-            return this.GetType().Name == other.GetType().Name;
+            return this == other || GetType().Name == other.GetType().Name;
         }
 
         /// <inheritdoc/>