Gerolf Seitz wrote:
Hi Bernd,

what java version do you use?
I'm on JDK 1.6_10, which seems to have no problems with the
@Override for methods specified in an interface.

I am on 1.5_something. I wondered indeed if you didn't face the same problem. When I read the javadoc on @Override, which is the same for 1.5 & 1.6, I just committed this patch. The docs say:

"If a method is annotated with this annotation type but does not override a superclass method, compilers are required to generate an error message."

Interface are not superclasses, so getting a compiler error sounded reasonable.

Do you have a problem with leaving it like it is now in this case?
Still, it would be interesting to know why we encounter different behavior.

  Bernd


  Gerolf

On Fri, May 2, 2008 at 7:56 PM, <[EMAIL PROTECTED]> wrote:

Author: berndf
Date: Fri May  2 10:56:05 2008
New Revision: 652849

URL: http://svn.apache.org/viewvc?rev=652849&view=rev
Log:
[vysper] fix compile-time error

Modified:

 
labs/vysper/src/main/java/org/apache/vysper/xmpp/server/AbstractSessionContext.java

Modified:
labs/vysper/src/main/java/org/apache/vysper/xmpp/server/AbstractSessionContext.java
URL:
http://svn.apache.org/viewvc/labs/vysper/src/main/java/org/apache/vysper/xmpp/server/AbstractSessionContext.java?rev=652849&r1=652848&r2=652849&view=diff

==============================================================================
---
labs/vysper/src/main/java/org/apache/vysper/xmpp/server/AbstractSessionContext.java
(original)
+++
labs/vysper/src/main/java/org/apache/vysper/xmpp/server/AbstractSessionContext.java
Fri May  2 10:56:05 2008
@@ -109,7 +109,6 @@
        return
getServerRuntimeContext().getResourceRegistry().bindSession(this);
    }

-    @Override
    public String nextSequenceValue() {
       return sequence.create();
    }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to