[238] reduant statements bug

Hello Vlad,

In the first example ReSharper is wrong.
In the second you should override GetHashCode as
Id.GetHashCode()

Here two samples:

1. Sample One:

public class BaseObject : DataObject {
/// <summary>
/// Initializes a new instance of the BaseObject class.
/// </summary>
public BaseObject() { }
private int m_Id = Int32.MinValue;
/// <summary>
/// Gets or sets ID.
/// </summary>
public virtual int Id {
get { return m_Id; }
set { m_Id = value; }
}
}
[ MapTable("concrete") ]
public class ConcreteObject : BaseObject {
/// <summary>
/// Initializes a new instance of the ConcreteObject class.
/// </summary>
public ConcreteObject() { }
/// <summary>
/// Gets or sets ID.
/// </summary>
[ MapField("concrete_id") ]
public override int Id {
get { return base.Id; }
set { base.Id = value; }
}
}
Overriden property assumed by ReSharper as reduant, but it is wrong
because it uses to attach attribute that cannnot be attached at base
class.

2. Sample Two:

[ MapTable("concrete") ]
public class ConcreteObject : BaseObject {
/// <summary>
/// Initializes a new instance of the ConcreteObject class.
/// </summary>
public ConcreteObject() { }
/// <summary>
/// Determines whether the specified System.Object is equal to the
current System.Object.
/// </summary>
public override bool Equals(object obj) {
ConcreteObject xx = this as ConcreteObject;
ConcreteObject yy = obj as ConcreteObject;
if (yy != null) {
return xx.Id == yy.Id;
} else {
return base.Equals(obj);
}
}
/// <summary>
/// Serves as a hash function for a particular type, suitable for
use in hashing algorithms
/// and data structures like a hash table.
/// </summary>
public override int GetHashCode() {
return base.GetHashCode();
}
}
Overriden method GetHashCode assumed by ReSharper as reduant, but it
is wrong because it uses to prevent compiler warning about using
overriden Equals without overriden GetHashCode().

Thanks,
Andrey Simanovsky