Re: [patch #1859] [Patch #1859] java.sql.Time#setTime(long)

2003-08-29 Thread Arnaud Vandyck 
I think I'll stop committing patches late at night! ;)

On Wed, 27 Aug 2003 18:43:13 -0400
[EMAIL PROTECTED] wrote:

 Patch #1859 has been updated. 
 
 Project: 
 Category: None
 Status: Closed
 Summary: java.sql.Time#setTime(long)
 
 Follow-Ups:
 
 Date: Thu 08/28/03 at 00:43
 By: mark
 
 Comment:
 If the only thing we have todo is just call super.setTime() with the same argument 
 then this is unnecessary.

 

 I don't know why japitools is listing this since it isn't a real API difference, 
 just an implementation issue.

 
 ---
 
 ---
 For more info, visit:
 
 http://savannah.gnu.org/patch/?func=detailpatchpatch_id=1859group_id=85
 
 ___
   Message sent via/by Savannah
   http://savannah.gnu.org/
 
 


-- Arnaud Vandyck, STE fi, ULg
   Formateur Cellule Programmation.


___
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath

java.sql.Time#setTime(long)

2003-08-24 Thread Arnaud Vandyck 
This methode does not exist in classpath but does in J2SE1.4

http://java.sun.com/j2se/1.4.2/docs/api/java/sql/Time.html#setTime(long)

... and is not deprecated. 

Also, Is  it normal that the  deprecated methode of this  class throw an
IllegalArgumentException?

-- Arnaud Vandyck, STE fi, ULg
   Formateur Cellule Programmation.


___
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath

Re: java.sql.Time#setTime(long)

2003-08-24 Thread Mark Wielaard 
Hi,

On Sun, 2003-08-24 at 20:08, Arnaud Vandyck wrote:
 This methode does not exist in classpath but does in J2SE1.4
 
 http://java.sun.com/j2se/1.4.2/docs/api/java/sql/Time.html#setTime(long)
 
 ... and is not deprecated. 

And it seems to be defined in earlier versions like 1.3.
It looks like there is a constructor that does almost exactly what this
method does. Could you take a look at that and try to create a patch
that implements this method?

 Also, Is  it normal that the  deprecated methode of this  class throw an
 IllegalArgumentException?

Yes, because that is what the documentation says it should do.

Cheers,

Mark



___
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath