Make getNegation() abstract in BinaryComparisonOperatorNode and 
UnaryComparisonOperatorNode
-------------------------------------------------------------------------------------------

                 Key: DERBY-4412
                 URL: https://issues.apache.org/jira/browse/DERBY-4412
             Project: Derby
          Issue Type: Improvement
          Components: SQL
    Affects Versions: 10.6.0.0
            Reporter: Knut Anders Hatlen
            Assignee: Knut Anders Hatlen
            Priority: Trivial


Both BinaryComparisonOperatorNode and UnaryComparisonOperatorNode have a method 
called getNegation() with the following code:

                /* Keep the compiler happy - this method should never be called.
                 * We should always be calling the method in a sub-class.
                 */
                if (SanityManager.DEBUG)
                SanityManager.ASSERT(false,
                                        "expected to call getNegation() for 
subclass " +
                                        getClass().toString());
                return this;

Instead of relying on asserts to detect missing method overrides at run-time, 
we should make the methods abstract so that such errors are detected by the 
compiler.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to