This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] java.security.Permission


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am Montag, 7. Juni 2004 14:21 schrieb Jerry Quinn:
> Michael Koch writes:
>  > -----BEGIN PGP SIGNED MESSAGE-----
>  > Hash: SHA1
>  >
>  > Am Montag, 7. Juni 2004 07:25 schrieb Jerry Quinn:
>  > > This fixes a java.io.FilePermission.simple failure in mauve. 
>  > > I don't know if it's right, but it seems reasonable.
>  > >
>  > > 2004-06-07  Jerry Quinn  <jlquinn@optonline.net>
>  > >
>  > > 	* java/security/Permission.java (Permission): Reject null
>  > >         name.
>  >
>  > No, its not correct. Neither SUN's API documentation says this
>  > nor a quick test proves your assumption (testcase attached). It
>  > gives no NullPointerException with SUN's JDK 1.4.2.
>
> Hmm, ok, a java.io.FilePermission with a null name will throw
> NullPointerException, even though the API docs don't mention this. 
> I incorrectly extrapolated that the base class should do this as
> well.
>
> There are mauve tests testing a bunch of misformed inputs to the
> constructor of java.io.FilePermission, which fail right now, since
> they expect various errors to be thrown.  The JDK throws these
> errors in the constructor, but doesn't document the errors being
> thrown. Our implementation never throws an error.
>
> Changing isn't hard, but what's the policy for things like this? 
> Follow the written docs or emulate the JDK?

Emulate JDK in this case. Docs are known to be incomplete or wrong 
sometimes.


Michael
- -- 
Homepage: http://www.worldforge.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFAxGIYWSOgCCdjSDsRAteDAJ9hrDPKdev/ykg5F+Bp5IXQzXLnFACgkQSb
1JoI41717IP0T99Qp6bcaHs=
=rvaH
-----END PGP SIGNATURE-----


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]