Path: utzoo!utgpu!jarvis.csri.toronto.edu!rutgers!cs.utexas.edu!tut.cis.ohio-state.edu!ucbvax!ANDREW.CMU.EDU!cfe+ From: cfe+@ANDREW.CMU.EDU ("Craig F. Everhart") Newsgroups: comp.soft-sys.andrew Subject: Re: WP bug on SPARC, new release version? Message-ID: <4ZOejwC00VsL81FmsL@andrew.cmu.edu> Date: 22 Nov 89 14:23:56 GMT References: Sender: daemon@ucbvax.BERKELEY.EDU Organization: The Internet Lines: 13 I have to agree with Guy Harris that the correct thing to do would be to revoke the idea that wp_DeAllocate should take any of several different types. The only thing I can say in defense of the way that it currently works is that I wrote it long ago, when a relative novice to C and didn't understand the HC compiler's warnings. That being said, I've absorbed Isaac Salzman's change to flush the ``union'' from the parameter list to make the thing work moderately correctly, but it's a little late (tape going out this morning) to ``do the right thing'', implementing wp_DeAllocateSrch/wp_DeAllocatePKs, changing all the calls everywhere in all andrew code, and updating the documentation. That should be done, and maybe if we ever generate another version... Craig