From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Durand Newsgroups: gmane.emacs.devel Subject: Re: [ELPA] Want to submit two packages "ilist" and "blist" Date: Mon, 20 Sep 2021 16:39:19 +0800 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="36672"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (x86_64-apple-darwin20.5.0) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Mon Sep 20 10:40:31 2021 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mSEql-0009R8-2c for ged-emacs-devel@m.gmane-mx.org; Mon, 20 Sep 2021 10:40:31 +0200 Original-Received: from localhost ([::1]:57996 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mSEqj-0003JY-UZ for ged-emacs-devel@m.gmane-mx.org; Mon, 20 Sep 2021 04:40:29 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:54692) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mSEpj-000269-5w for emacs-devel@gnu.org; Mon, 20 Sep 2021 04:39:27 -0400 Original-Received: from mail-pg1-x536.google.com ([2607:f8b0:4864:20::536]:39714) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1mSEpg-0003OI-FD for emacs-devel@gnu.org; Mon, 20 Sep 2021 04:39:26 -0400 Original-Received: by mail-pg1-x536.google.com with SMTP id g184so16568129pgc.6 for ; Mon, 20 Sep 2021 01:39:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:subject:references:date:in-reply-to:message-id:user-agent :mime-version; bh=FcWTazMPU1y2cLv0EBwRo8gHP2OQHfTCBnefDgYpiG8=; b=iD7Vhl6aj86A6MiqHjTSqCssFdUxknd3PHRF2f13XMR065OKjIkkAVdfcf+MYkgJER P8nhbSI92Rqdkv75Q2LJhoQWeSkus6URQXIQgIIqsBYz0wwDs35PAYvYRZphmxVWtxLb m6hhT/Sg8ck07RWK5UVCGtgj2T3I1BLbo12h1jLd4NvhMOj3TOjJsqDZBzOCIyPMr4+B kgMNdu7dVnSvzVGK3UYqk8Tf69dbCkqLd7m5/woog0CiescpGJI4Ag/Jujsi1w35GaQz xtS0028lqadawAulrEqfQbxpLhbmzuq93MsfvxSfZuP7WPv5T4G+Q5Enk5gJugV2q8zd +cwQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:subject:references:date:in-reply-to :message-id:user-agent:mime-version; bh=FcWTazMPU1y2cLv0EBwRo8gHP2OQHfTCBnefDgYpiG8=; b=YNgK4lVPl11ZioGH6fp9ItGN/VoYVahdVKJO0v/r4D0qdTn9y+bKxuD+O9QAUZXEz/ 4KRkag9E9MGUJolLWI0YVTeMtEfk6yQ9D4E+Qf7xlw36mZ2ID8mP5MwrBXO1CVKXJ7VK 5AVstocTqsUZD4Z08tmZSaUdZSNSEXicFuUkIsQBmQVnNOpCAl2k0erjGhiW/3hJS+As /4T2pzGP8unRgAUiy8Y/z/Zp6R152KMPWGhCbEOjBWu9u/ARX2GiDfJ2sRdkASU7siyS N97f646+QuXOIQM/ArbnaZKr5hun/3tE1vbqw+PlTY48RTg5qKD847VV/ml1RH9rbmDy qQ+A== X-Gm-Message-State: AOAM530dAZhWVnya5GOz+9+VYh5sNzF18zvytbijj6QARLuDI6UgMadN ru23CXI7q9Mb9+iXSUrOv+uYdCb3+6Y= X-Google-Smtp-Source: ABdhPJxger33gFnOA53nOmrBoPP/xZjXmTXbT0Hm9NHR94TvdCP00YTIK3wQfpnvDWqSvjU8CZGY0w== X-Received: by 2002:a62:824a:0:b0:445:2372:86ae with SMTP id w71-20020a62824a000000b00445237286aemr15204214pfd.43.1632127162247; Mon, 20 Sep 2021 01:39:22 -0700 (PDT) Original-Received: from MacBook-Pro-de-Severe.local.gmail.com (123-193-46-223.dynamic.kbronet.com.tw. [123.193.46.223]) by smtp.gmail.com with ESMTPSA id e13sm9291227pfn.212.2021.09.20.01.39.21 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Sep 2021 01:39:21 -0700 (PDT) In-Reply-To: (Stefan Kangas's message of "Sun, 19 Sep 2021 03:15:42 -0700") Received-SPF: pass client-ip=2607:f8b0:4864:20::536; envelope-from=mmemmew@gmail.com; helo=mail-pg1-x536.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:275117 Archived-At: >>>>> On Sun, 19 Sep 2021 03:15:42 -0700, Stefan Kangas said: Stefan> Durand writes: >> I have written two Emacs pacakges, called "ilist" and "blist" >> (the former is the "engine", and hence a dependency, of the >> latter). Now I am thinking about submitting the packages to GNU >> ELPA. >> >> The package "blist" is to display the list of bookmarks, in the >> sense of "bookmark.el", in a similar way as Ibuffer. Stefan> Thanks, this looks useful at a first glance, and something Stefan> that I think we should definitely welcome on GNU ELPA. Thanks. Stefan> I wonder why you chose to write this as an entirely new Stefan> package instead of improving the bookmark list we already Stefan> have in bookmark.el. Could you explain your rationale for Stefan> this? Stefan> One thing that stands out is that you list bookmarks by Stefan> category, like in ibuffer, something that I don't think is Stefan> currently possible with `tabulated-list-mode'. Yes, this is one reason: it would be a breaking change if I want to modify the existing library. And when I was thinking about this idea my main concern was in fact the package "ilist", which is an abstract library package. It seems natural that "ilist" should be a separate package, and I did not think too much when I built "blist" upon "ilist". I guess I am not used to the idea of modifying existing Emacs packages. Stefan> Some other scattered comments: Stefan> - Is any of this suitable for inclusion in bookmark.el? I'm Stefan> thinking of `blist-show-all-annotations', for example. I think maybe some functions of bookmark.el can be improved, like its annotation-showing functions: it would be a good idea to refrain from showing empty annotations. Also I think the function "bookmark-load" can be improved as well: its documentation string says that the newly loaded bookmarks will be appended at the front of the list, but from the codes it seems that the new list will be appended at the end (as seen in the function "bookmark-import-new-list"). But in general my codes are built upon the outputs of ilist, so might not be directly suitable for inclusion in bookmark.el. Stefan> - I would add (defalias 'blist 'blist-list-bookmarks) for Stefan> discoverability. This is a good idea. Thanks! Stefan> - blist-show-annotation says "No bookmarks to show" when it Stefan> should probably say "No annotation for this bookmark". I Stefan> think? Thanks, I will modify this later. Stefan> - Instead of the in-buffer header, you could use Stefan> `header-line-format', which means that the header stays on Stefan> top even as you scroll down. I am not sure a header is that useful to see all the time. But I understand that some users might prefer the header. I will attempt to provide a user option to control this behaviour. Thanks for the suggestion. -- Durand