SD-128 fix override checks for default methods

The check for inheriting two conflicting members was wrong for default
methods, leading to a missing error message.

We were also not issuing "needs `override' modifier" when overriding a
default method.

Removes two methods:

  - `isDeferredOrJavaDefault` had a single use that is removed in this commit.

  - `isDeferredNotJavaDefault` is redundant with `isDeferred`, because
no default method has the `DEFERRED` flag:
    - For symbols originating in the classfile parser this was the case
      from day one: default methods don't receive the `DEFERRED` flag.
      Only abstract interface methods do, as they have the `JAVA_ACC_ABSTRACT`
      flag in bytecode, which the classfile parser translates to `DEFERRED`.

    - For symbols created by the Java source parser, we don't add the
      `DEFERRED` to default methods anymore since 373db1e.

Fixes scala/scala-dev#128

1 files changed, 3 insertions, 0 deletions

diff --git a/test/files/neg/sd128/A.java b/test/files/neg/sd128/A.java
new file mode 100644
index 0000000000..6774deba2e
--- /dev/null
+++ b/test/files/neg/sd128/A.java
@@ -0,0 +1,3 @@
+interface A {
+  default int f() { return -10; }
+}