From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on polar.synack.me X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham autolearn_force=no version=3.4.4 X-Google-Thread: 103376,2843c5eea3415584 X-Google-Attributes: gid103376,public X-Google-Language: ENGLISH,ASCII-7-bit Path: g2news1.google.com!news4.google.com!news.glorb.com!meganewsservers.com!feeder2.on.meganewsservers.com!feed.cgocable.net!read1.cgocable.net.POSTED!53ab2750!not-for-mail From: "Warren W. Gay VE3WWG" User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.2) Gecko/20040804 Netscape/7.2 (ax) X-Accept-Language: en-us, en MIME-Version: 1.0 Newsgroups: comp.lang.ada Subject: Re: APQ References: In-Reply-To: Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Message-ID: Date: Sat, 18 Dec 2004 10:52:52 -0500 NNTP-Posting-Host: 24.150.168.167 X-Complaints-To: abuse@cogeco.ca X-Trace: read1.cgocable.net 1103385110 24.150.168.167 (Sat, 18 Dec 2004 10:51:50 EST) NNTP-Posting-Date: Sat, 18 Dec 2004 10:51:50 EST Organization: Cogeco Cable Xref: g2news1.google.com comp.lang.ada:7048 Date: 2004-12-18T10:52:52-05:00 List-Id: Brian May wrote: >>>>>>"Brian" == Brian May writes: > > > Brian> Good point. I meant to change it in both places, but I think I ended > Brian> up changing it in only one place... > > You were right, I missed it up. > > Here is a patch that turns all methods into abstract methods. Ok, Brian, hang on to your patches. I am trying to get my Source Forge account reactivated (something's happened to it), and then get this project up there. I am surprised that that was all there was to it. I must have made the same mistake. > Unfortunately, I could not make Finalize abstract, the compiler > complained that abstract methods must be visible (Finalize is in the > private section). This restriction seems strange to me, but I am not > going to argue with the compiler and left the function generating an > exception at run-time. > > I suspect Finalize doesn't need to be abstract, but didn't go > exploring enough to verify why it is abstract. Normally, you would want to override Finalize, because you will have allocated references within the derived object to clean up etc. However, 90% is better than nothing ;-) > Also, I noticed that the pointers are "access all". I couldn't see any > requirement for "all", and my code still compiles, so I changed them > to "access". This eliminates a potential source of errors. I can't imagine it making a practical difference here, but perhaps I lack imagination on this point. Warren.