<html>
<head>
<style>
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Verdana
}
</style>
</head>
<body class='hmmessage'>
First, I'm not against incremental changes, I'm for all the way.<BR>
Second, we're not talking about entire clos, but a very specific problem. This problem is not unique for clos, but just heavily used by it.<BR>
What I'm afraid of is there is a more fundamental issue here and this patch makes it even more hidden.<BR>
BTW, ANSI standard does not allow &aux in generic functions (sec 3.4.2), but this version of clos treats it as a legitimate citizen.<BR>
Question: how do you explain why the patch does not catch reinitialize-instance.error.1? How does this case differ from the other 3?<BR> <BR>> Date: Tue, 28 Jul 2009 01:37:13 +0300<BR>> Subject: Re: [armedbear-devel] A clos patch for review<BR>> From: ville.voutilainen@gmail.com<BR>> To: ptsenter@hotmail.com<BR>> CC: armedbear-devel@common-lisp.net<BR>> <BR>> 2009/7/28 Peter Tsenter <ptsenter@hotmail.com>:<BR>> > The patch seems to produce a result as claimed.<BR>> > But it's not generic enough. It does not catch, e.g., defgeneric.error.20<BR>> > nor reinitialize-instance.error.1, which follow the same pattern: supply<BR>> > bogus arguments in initargs. Introducing this patch might create an<BR>> > impression that such cases handled properly always.<BR>> <BR>> Well, I'd rather do these fixes piecemeal than attempt to produce some perfect<BR>> megapatch that fixes everything clos-related in a single sweep.<BR><br /><hr />Bing™ brings you maps, menus, and reviews organized in one place. <a href='http://www.bing.com/search?q=restaurants&form=MLOGEN&publ=WLHMTAG&crea=TXT_MLOGEN_Local_Local_Restaurants_1x1' target='_new'>Try it now.</a></body>
</html>