Date: Mon, 10 Apr 2000 13:42:56 +0200 From: Ernst de Haan <ernst@jollem.com> To: freebsd-java@freebsd.org Subject: Re: FYI: FIXME's for patchset 4 Message-ID: <38F1BE40.F61B4C01@jollem.com> References: <38EF83B4.B6EE7FC8@jollem.com>
next in thread | previous in thread | raw e-mail | index | archive | help
This is a multi-part message in MIME format. --------------E51C6C152A2482B121DD7BBA Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Hi, Here's a new list of FIXME's, it now includes the share/ directory as well (the former only included the freebsd directory), still for patchset 4 (I will do the same for patchset 5 when I have more time). I've also included a list of XXX occurences in src/freebsd/ and src/shared/. Ernst Ernst de Haan wrote: > > Hi, > > Here's a list of the FIXME's in JDK 1.2.2 patchset 4. > The idea is (c) Copyleft Kees-Jan :) > > Ernst > > -- > Ernst de Haan > Freelance Java Architect > > "Come to me all who are weary and burdened, > and I will give you rest" -- Jesus Christ -- Ernst de Haan Freelance Java Architect "Come to me all who are weary and burdened, and I will give you rest" -- Jesus Christ --------------E51C6C152A2482B121DD7BBA Content-Type: text/plain; charset=us-ascii; name="jdk12-ps4-fixmes2" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="jdk12-ps4-fixmes2" bash-2.03$ grep -r -n "FIXME" freebsd share freebsd/hpi/green_threads/src/threads_md.c:1104: * FIXME: Someone who knows more about green threads, please implement freebsd/hpi/green_threads/src/interrupt_md.c:143: /* FIXME: Need support for more than 32 signals in FreeBSD 4.0 */ freebsd/hpi/green_threads/src/interrupt_md.c:169: /* FIXME: Need support for more than 32 signals in FreeBSD 4.0 */ freebsd/hpi/green_threads/src/threads_md.c.orig:1071: * FIXME: Someone who knows more about green threads, please implement freebsd/native/sun/awt/awt_Robot.c:551: /* FIXME: Is this the correct FreeBSD version? */ freebsd/javavm/runtime/javai_md.c:61: /* FIXME: Caused seg fault */ share/classes/java/lang/FloatingDecimal.java:606: * FIXME -- use more precise constants here. It costs no more. share/classes/java/io/StreamTokenizer.java:138: * initializing ttype. FIXME This could be made public and share/classes/java/awt/Graphics.java:1139: // FIXME: 1.2 beta3 placeholder, replace for beta4 share/classes/java/awt/Graphics.java:1160: // FIXME: 1.2 beta3 placeholder, replace for beta4 share/classes/java/text/ChoiceFormat.java:471: // FIXME fix overflow & underflow share/classes/java/text/ChoiceFormat.java:474: // FIXME fix overflow & underflow share/classes/java/text/Collator.java:475: // FIXME: These three constants should be removed. share/classes/sun/io/CharToByteConverter.java:310: //FIXME output buffer size should use getMaxBytesPerChar value. share/classes/javax/swing/JMenu.java:1244: // FIXME: [[[WDW - probably should set this when share/classes/javax/swing/JFrame.java:627: // FIXME: [[[WDW - should also return ICONIFIED and ICONIFIABLE share/classes/javax/swing/JToolBar.java:699: // FIXME: [[[WDW - need to add orientation from BoxLayout]]] share/classes/javax/swing/JToolBar.java:700: // FIXME: [[[WDW - need to do SELECTABLE if SelectionModel is added]]] share/classes/javax/swing/JComponent.java:2147: // [[[FIXME: WDW - we probably should require this share/classes/javax/swing/JSplitPane.java:972: // FIXME: [[[WDW - Should also add BUSY if this implements share/classes/javax/swing/SwingUtilities.java:1351: // [[[FIXME: WDW - for JDK1.2 this code can be replaced with share/classes/javax/accessibility/AccessibleBundle.java:60: // [[[FIXME: WDW - obtaining resource bundles can be share/native/sun/awt/image/cvutils/img_globals.h:115: * FIXME! share/native/sun/awt/cmm/ukcpmgr.c:126: /* FIXME: Is this the number of processors actually online? */ share/native/sun/awt/cmm/winmem.c:1468:/* FIXME -- Test Program */ share/native/java/math/bn-1.1/germain.c:367: * FIXME: Actually, we don't need to compute the share/demo/jfc/Notepad/Notepad.java:474: * FIXME - I'm not very useful yet bash-2.03$ --------------E51C6C152A2482B121DD7BBA Content-Type: text/plain; charset=us-ascii; name="jdk12-ps4-xxxs" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="jdk12-ps4-xxxs" bash-2.03$ grep -r -n "\*[ ]*XXX" freebsd share freebsd/hpi/green_threads/include/context_md.h:28:#if (__FreeBSD__ <= 4) /*** XXX fix me ***/ freebsd/hpi/green_threads/src/threads_md.c:857: /* XXX We may need to do something here XXX */ freebsd/hpi/green_threads/src/threads_md.c:898: /* XXX We may need to do something here XXX */ freebsd/hpi/green_threads/src/clock.c:77: /* XXX If above comment applies may need to do something here XXX */ freebsd/hpi/green_threads/src/threads_md.c.orig:832: /* XXX We may need to do something here XXX */ freebsd/hpi/green_threads/src/threads_md.c.orig:873: /* XXX We may need to do something here XXX */ freebsd/hpi/native_threads/src/sys_api_td.c:315: * XXX: Is fsync() interruptible by the interrupt method? freebsd/hpi/native_threads/src/sys_api_td.c:335: * XXX: is the following call interruptible? If so, this might freebsd/hpi/native_threads/src/sys_api_td.c:392: * XXX: is the following call interruptible? If so, this might freebsd/native/sun/awt/font/fontpath.c:45: * XXX This algorithm doesn't handle the fonts installed by the Solaris Font Administrator freebsd/native/sun/awt/font/fontpath.c:302: /* XXX Intentionally don't check for the ".PS" suffix, which is not unique for fonts freebsd/native/sun/awt/awt_p.h:295:/* XXX: Motif internals. Need to fix 4090493. */ freebsd/native/sun/awt/awt_Font.c:637: /* XXX: sometimes XLoadQueryFont returns a bogus font structure */ freebsd/native/sun/awt/awt_Choice.c:224: /* XXX: XmNuserData doesn't seem to work when passing in zero */ freebsd/native/sun/awt/canvas.c:1053: /* XXX: Button2Mask and Button3Mask don't get set on ButtonPress */ freebsd/native/sun/awt/tiny_Font.c:132: /* XXX: sometimes XLoadQueryFont returns a bogus font structure */ freebsd/native/java/lang/UNIXProcess_md.c:169:#ifdef __FreeBSD__ /*** XXX ***/ freebsd/native/java/lang/UNIXProcess_md.c:208:#ifdef __FreeBSD__ /*** XXX ***/ freebsd/native/java/lang/java_props_md.c:283:#ifdef __FreeBSD__ /*** XXX ***/ share/javavm/runtime/reflect_util.c:106: ThrowNoClassDefFoundError(0, sig); /* XXX severe error */ share/javavm/runtime/jvm.c:275: /*XXX: Use i <= n, NOT i < n, to get copy_data[n] share/javavm/runtime/jvm.c:2989: /* XXX: Is this too severe? */ share/javavm/runtime/jvm.c:3515: /* XXX So that SignalError finds the InvocationTargetException class */ share/javavm/runtime/jvm.c:3984: * XXX should be 'mb' or 'imb'? share/javavm/runtime/jvm.c:4049: * XXX 'mb' or 'imb'? share/javavm/runtime/jvm.c:4353: if (classes[i++] != cbHandle(pcb)) /* XXX match on handles */ share/javavm/runtime/jvm.c:4576: /* XXX what about <clinit>? */ share/classes/java/util/jar/Attributes.java:266: * XXX Need to handle UTF8 values and break up lines longer than 72 bytes share/classes/java/util/jar/Attributes.java:288: * XXX Need to handle UTF8 values and break up lines longer than 72 bytes share/classes/java/util/jar/Attributes.java:324: * XXX Need to handle UTF8 values. share/classes/java/text/MergeCollation.java:49: * XXX: make '' be a single quote. share/classes/sun/net/www/http/HttpClient.java:217: * XXX That documentation is wrong ... it's not package-private any more share/classes/sun/security/x509/CertAndKeyGen.java:126: * XXX Note: This behaviour is needed for backwards compatibility. share/classes/sun/security/x509/AlgorithmId.java:256: * XXX This method needs to be enhanced so that we can also pass the share/classes/sun/security/x509/X509Cert.java:757: * XXX for v2 and later, a bunch of tagged options follow share/classes/sun/security/util/DerOutputStream.java:326: * XXX what character set is this? share/classes/sun/security/provider/PolicyFile.java:1029: * XXX Decide if/how to add validity fields and "purpose" fields to share/classes/sun/security/provider/PolicyFile.java:1030: * XXX policy entries share/classes/sun/security/tools/PolicyTool.java:692: * XXX: the next two lines are commented out on purpose. share/classes/sun/security/tools/PolicyTool.java:1255: * XXX in windows, the double backslash originally in share/classes/sun/security/tools/PolicyTool.java:2109: * XXX should be a StringTokenizer, huh... share/classes/sun/security/tools/PolicyTool.java:2599: * XXX hate to add a special case check here ... but ... share/classes/sun/security/tools/PolicyTool.java:2675: * XXX hate to add a special case check here ... but ... share/classes/com/sun/CORBA/ServerSubcontract.java:48: * XXX: should the argument be the servant or the share/classes/com/sun/CORBA/idl/ORB.java:440: /* XXX Is this needed ??? bash-2.03$ grep -r -n "//[ ]*XXX" freebsd share freebsd/classes/sun/awt/motif/MComponentPeer.java:292: // XXX: bogus share/test/security/sqe/x509/SimpleSigner.java:30: private X500Name agent; // XXX should be X509CertChain share/classes/java/util/Locale.java:862: // XXX Depending on performance of synchronized, may want to share/classes/java/util/Locale.java:863: // XXX just compute in constructor. share/classes/java/util/Locale.java:901: // XXX instance and class variables. For now keep these separate, since it is share/classes/java/util/jar/Manifest.java:203: //XXX: Fix for when the average is 0. When it is 0, share/classes/java/io/FilePermission.java:226: // XXX: at this point the path should be absolute. die if it isn't? share/classes/java/net/InetAddress.java:187: //XXX: if it looks a spoof just return the address? share/classes/java/net/SocketPermission.java:720: // XXX: if all else fails, compare hostnames? share/classes/java/security/Permissions.java:157: // XXX this could be public. Question is, do we want to make it public? share/classes/sun/applet/AppletViewer.java:504: // XXX: this privileged block should be made smaller share/classes/sun/applet/AppletPanel.java:759: //XXX: this is needed to be able to create the classloader itself! share/classes/sun/awt/robot/probe/ProbeBuffer.java:105: // XXX: There is a general need to filter events share/classes/sun/awt/robot/probe/ProbePrinter.java:115: // XXX: There is a general need to filter events share/classes/sun/misc/Launcher.java:506: // XXX? share/classes/sun/security/x509/CertAndKeyGen.java:291: // XXX should have a way to pass prng to the signature share/classes/sun/security/x509/X500Name.java:633: int j_max = 1; // XXX j_max = f(data)!! share/classes/sun/security/x509/AlgorithmId.java:231:// XXXX cleaning required share/classes/sun/security/x509/AlgIdDSA.java:61: // XXX deprecated for general use share/classes/sun/security/x509/X500Signer.java:101: private X500Name agent; // XXX should be X509CertChain share/classes/sun/security/x509/X509Cert.java:100: // XXX deprecated, delete this share/classes/sun/security/x509/X509Cert.java:668: rawCert = seq [0].toByteArray (); // XXX slow; fixme! share/classes/sun/security/x509/X509Cert.java:675: // XXX why was this error check commented out? share/classes/sun/security/x509/EDIPartyName.java:118: // XXX - shd check is chars fit into PrintableString share/classes/sun/security/x509/EDIPartyName.java:126: // XXX - shd check is chars fit into PrintableString share/classes/sun/security/util/DerOutputStream.java:339:// // XXX IMPLEMENT ME share/classes/sun/security/util/DerValue.java:191: // XXX must also parse BER-encoded constructed share/classes/sun/security/util/DerValue.java:300: // XXX yeech, excess copies ... DerInputBuffer.write(OutStream) share/classes/sun/security/util/ObjectIdentifier.java:282: // XXX this API should probably facilitate the JDK sort utility share/classes/sun/security/util/BigInt.java:58: // XXX we assume exactly _one_ sign byte is used... share/classes/sun/security/util/ManifestDigester.java:106: return; // XXX: exception? share/classes/sun/security/util/ManifestDigester.java:136: return; // XXX: exception? share/classes/sun/security/util/SignatureFileVerifier.java:147: // XXX: should this be an exception? share/classes/sun/security/util/SignatureFileVerifier.java:239: //XXX: we will continue and verify each section share/classes/sun/security/provider/PolicyFile.java:706: //XXX we might want to keep a hash of created factories... share/classes/sun/security/pkcs/SignerInfo.java:345: // XXX We may want to check for additional extensions in the share/classes/sun/security/tools/JarSigner.java:664: // XXX need to disable echo; on UNIX, call getpass(char *prompt) share/classes/sun/security/tools/PolicyTool.java:252: // XXX We should add a command-line option that lets users share/classes/sun/security/tools/PolicyTool.java:358: // XXX: should i display an error dialog to the user? share/classes/sun/security/tools/PolicyTool.java:400: // XXX: should i display an error dialog to the user? share/classes/sun/security/tools/PolicyTool.java:461: //XXX we might want to keep a hash of created factories... share/classes/sun/security/tools/PolicyTool.java:2187: // XXX share/classes/com/sun/CORBA/iiop/ConnectionTable.java:224: // XXX is necessary to do a GC to reclaim closed network connections ?? share/classes/com/sun/CORBA/iiop/ConnectionTable.java:243: // XXX Need to worry about wrap around some day share/classes/com/sun/CORBA/iiop/ClientResponseImpl.java:90: // XXX To be implemented share/classes/com/sun/CORBA/iiop/ORB.java:59: // XXX Is this necessary here? share/classes/com/sun/CORBA/iiop/ORB.java:190: // XXX What to do here?? share/classes/com/sun/CORBA/iiop/IIOPConnection.java:569: // XXX share/classes/com/sun/CORBA/iiop/IIOPOutputStream.java:138: // XXX figure out if alignment change is needed by checking if share/classes/com/sun/CORBA/idl/RequestImpl.java:375: // XXX return this for deferred sends share/native/sun/awt/font/fontmanager/fontobjects/cmaps.cpp:835: case 103: // XXX cmap format 1 share/native/sun/awt/font/fontmanager/fontobjects/fontObject.cpp:1704: // XXX throw a Java exception istead share/native/sun/awt/font/fontmanager/fontobjects/fontObject.cpp.orig:1704: // XXX throw a Java exception istead bash-2.03$ --------------E51C6C152A2482B121DD7BBA Content-Type: text/x-vcard; charset=us-ascii; name="ernst.vcf" Content-Transfer-Encoding: 7bit Content-Description: Card for Ernst de Haan Content-Disposition: attachment; filename="ernst.vcf" begin:vcard n:de Haan;Ernst tel;fax:+31 (0)26 3645634 tel;work:+31 (0)26 3623895 x-mozilla-html:FALSE url:http://www.znerd.demon.nl/ org:Jollem adr:;;Rozendaalselaan 35;Velp;GLD;6881 KZ;Netherlands version:2.1 email;internet:ernst@jollem.com title:Java Architect fn:Ernst de Haan end:vcard --------------E51C6C152A2482B121DD7BBA-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-java" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?38F1BE40.F61B4C01>