From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.io/gmane.emacs.gnus.general/85247 Path: news.gmane.org!not-for-mail From: Ted Zlatanov Newsgroups: gmane.emacs.gnus.general Subject: Re: [PATCH] Two issues with the gnus-registry Date: Mon, 10 Nov 2014 08:54:01 -0500 Organization: =?utf-8?B?0KLQtdC+0LTQvtGAINCX0LvQsNGC0LDQvdC+0LI=?= @ Cienfuegos Message-ID: <87bnoff9fq.fsf@lifelogs.com> References: <87egtx70hy.fsf@ericabrahamsen.net> <87wq7lsggo.fsf@lifelogs.com> <87zjchxr1v.fsf@ericabrahamsen.net> <87h9yogjer.fsf@ericabrahamsen.net> <87a942lg39.fsf@ericabrahamsen.net> <8761eqlfvk.fsf@ericabrahamsen.net> <87mw82jdbf.fsf@ericabrahamsen.net> Reply-To: ding@gnus.org NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1415627816 23841 80.91.229.3 (10 Nov 2014 13:56:56 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 10 Nov 2014 13:56:56 +0000 (UTC) To: ding@gnus.org Original-X-From: ding-owner+M33491@lists.math.uh.edu Mon Nov 10 14:56:49 2014 Return-path: Envelope-to: ding-account@gmane.org Original-Received: from util0.math.uh.edu ([129.7.128.18]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1XnpS9-0001GX-Ed for ding-account@gmane.org; Mon, 10 Nov 2014 14:56:21 +0100 Original-Received: from localhost ([127.0.0.1] helo=lists.math.uh.edu) by util0.math.uh.edu with smtp (Exim 4.63) (envelope-from ) id 1XnpS1-0000SX-Sa; Mon, 10 Nov 2014 07:56:13 -0600 Original-Received: from mx2.math.uh.edu ([129.7.128.33]) by util0.math.uh.edu with esmtps (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1XnpS0-0000SK-OI for ding@lists.math.uh.edu; Mon, 10 Nov 2014 07:56:12 -0600 Original-Received: from quimby.gnus.org ([80.91.231.51]) by mx2.math.uh.edu with esmtps (TLSv1:AES128-SHA:128) (Exim 4.76) (envelope-from ) id 1XnpRz-0003wJ-FN for ding@lists.math.uh.edu; Mon, 10 Nov 2014 07:56:12 -0600 Original-Received: from plane.gmane.org ([80.91.229.3]) by quimby.gnus.org with esmtp (Exim 4.80) (envelope-from ) id 1XnpRy-0006dq-3c for ding@gnus.org; Mon, 10 Nov 2014 14:56:10 +0100 Original-Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1XnpQs-0007x6-FU for ding@gnus.org; Mon, 10 Nov 2014 14:55:02 +0100 Original-Received: from c-98-229-61-72.hsd1.ma.comcast.net ([98.229.61.72]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 10 Nov 2014 14:55:02 +0100 Original-Received: from tzz by c-98-229-61-72.hsd1.ma.comcast.net with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 10 Nov 2014 14:55:02 +0100 X-Injected-Via-Gmane: http://gmane.org/ Mail-Followup-To: ding@gnus.org Original-Lines: 28 Original-X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: c-98-229-61-72.hsd1.ma.comcast.net X-Face: bd.DQ~'29fIs`T_%O%C\g%6jW)yi[zuz6;d4V0`@y-~$#3P_Ng{@m+e4o<4P'#(_GJQ%TT= D}[Ep*b!\e,fBZ'j_+#"Ps?s2!4H2-Y"sx" Mail-Copies-To: never User-Agent: Gnus/5.130012 (Ma Gnus v0.12) Emacs/25.0.50 (gnu/linux) Cancel-Lock: sha1:nEBzZVmGyzRrOyKOHStxkx4XAJ4= X-Spam-Score: -3.5 (---) List-ID: Precedence: bulk Xref: news.gmane.org gmane.emacs.gnus.general:85247 Archived-At: On Sat, 08 Nov 2014 16:39:48 +0800 Eric Abrahamsen wrote: EA> So unless I'm really missing something, my proposal is: EA> 1. Only provide one limit: max-size. EA> 2. Allow customization of prune-factor. EA> That seems like all the customization you'd need. Cap the size, and EA> provide a reasonable control of how often the registry gets pruned. OK. I am afraid I made the whole thing too complicated and it took another set of eyes to recognize it. Thanks for that, and the fixes. Please go ahead and apply any changes you have in mind. I am sure no one is tuning soft/hard pruning. Simpler is definitely better. EA> That would require a change in the object signature, which would mean EA> some extra handling code for "upgrading". But once we're doing that, we EA> could also take the opportunity to add :prune-function and EA> :sort-function slots on the base registry object, which would be EA> nice. We could even change the default store filename from its "eioio" EA> extension to "eieio". :) Yeah, I remember that bug report about the extension of the database file, if we're upgrading anyway... It's a good chance to batch these changes. Ted