{"thread":{"id":"64315","subject":"[RFC Outreachy] gpg-interface: Replace strbuf_split()* with string_list_split()","startedAt":"2025-10-14T00:44:29Z","lastAt":"2025-10-14T10:31:33Z","messageCount":3,"participants":["Bello Olamide","Christian Couder"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"528680","messageId":"CAD=f0L9Z2kPWv-=wPbtytbMS7BuUtmAKCv2X_2rq0hB_Wsh=UA@mail.gmail.com","threadId":"64315","inReplyTo":null,"subject":"[RFC Outreachy] gpg-interface: Replace strbuf_split()* with string_list_split()","fromName":"Bello Olamide","fromEmail":"belkid98@gmail.com","sentAt":"2025-10-14T00:44:28Z","receivedAt":"2025-10-14T00:44:29Z","isPatch":false,"sender":{"key":"belkid98@gmail.com","avatar":"https://avatars.githubusercontent.com/u/73387291?v=4"},"body":"Hello,\n\nIn continuation of my search for an appropriate microproject after\ntemporarily moving on from my mail to the list which was about\nthe check-whitespace script check for \\No newline, I stumbled on the\nthread\nhttps://public-inbox.org/git/20250731225433.4028872-1-gitster@pobox.com/.\nAfter studying the thread and searching the codebase for other\ninstances where strbuf_split()* can be replaced with string_list_split(),\nI saw some cases in gpg-interface.c where get_ssh_key_fingerprint and\nget_default_ssh_signing_key functions use the strbuf_split_max to split the\ninput string and I think they can be good candidates for replacement with\nstring_list_split().\n\nPlease, will this be good for a microproject?\nI will also appreciate any guides.\n\nThanks\nBello\n"},{"id":"528693","messageId":"CAP8UFD2azLtbeMzGhgxHcdnCF=jD5esOs3BjhLWGCXvMmppX4w@mail.gmail.com","threadId":"64315","inReplyTo":"CAD=f0L9Z2kPWv-=wPbtytbMS7BuUtmAKCv2X_2rq0hB_Wsh=UA@mail.gmail.com","subject":"Re: [RFC Outreachy] gpg-interface: Replace strbuf_split()* with string_list_split()","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2025-10-14T08:56:53Z","receivedAt":"2025-10-14T08:57:07Z","isPatch":false,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"Hi,\n\nOn Tue, Oct 14, 2025 at 2:44 AM Bello Olamide <belkid98@gmail.com> wrote:\n>\n> Hello,\n>\n> In continuation of my search for an appropriate microproject after\n> temporarily moving on from my mail to the list which was about\n> the check-whitespace script check for \\No newline, I stumbled on the\n> thread\n> https://public-inbox.org/git/20250731225433.4028872-1-gitster@pobox.com/.\n> After studying the thread and searching the codebase for other\n> instances where strbuf_split()* can be replaced with string_list_split(),\n> I saw some cases in gpg-interface.c where get_ssh_key_fingerprint and\n> get_default_ssh_signing_key functions use the strbuf_split_max to split the\n> input string and I think they can be good candidates for replacement with\n> string_list_split().\n>\n> Please, will this be good for a microproject?\n> I will also appreciate any guides.\n\nYeah, I think it could be a good microproject.\n\nThe functions you mention don't modify the strings resulting from the\nsplit, so those strings don't need to be strbufs. This should be noted\nin the commit message (which should provide the reasons why the change\nmakes sense).\n"},{"id":"528699","messageId":"CAD=f0L8F585EDRUBJW-WbCd3299wQi832rMK+69Lhfuo2atywQ@mail.gmail.com","threadId":"64315","inReplyTo":"CAP8UFD2azLtbeMzGhgxHcdnCF=jD5esOs3BjhLWGCXvMmppX4w@mail.gmail.com","subject":"Re: [RFC Outreachy] gpg-interface: Replace strbuf_split()* with string_list_split()","fromName":"Bello Olamide","fromEmail":"belkid98@gmail.com","sentAt":"2025-10-14T10:31:31Z","receivedAt":"2025-10-14T10:31:33Z","isPatch":false,"sender":{"key":"belkid98@gmail.com","avatar":"https://avatars.githubusercontent.com/u/73387291?v=4"},"body":"On Tue, 14 Oct 2025 at 09:57, Christian Couder\n<christian.couder@gmail.com> wrote:\n>\n> Hi,\n>\n> On Tue, Oct 14, 2025 at 2:44 AM Bello Olamide <belkid98@gmail.com> wrote:\n> >\n> > Hello,\n> >\n> > In continuation of my search for an appropriate microproject after\n> > temporarily moving on from my mail to the list which was about\n> > the check-whitespace script check for \\No newline, I stumbled on the\n> > thread\n> > https://public-inbox.org/git/20250731225433.4028872-1-gitster@pobox.com/.\n> > After studying the thread and searching the codebase for other\n> > instances where strbuf_split()* can be replaced with string_list_split(),\n> > I saw some cases in gpg-interface.c where get_ssh_key_fingerprint and\n> > get_default_ssh_signing_key functions use the strbuf_split_max to split the\n> > input string and I think they can be good candidates for replacement with\n> > string_list_split().\n> >\n> > Please, will this be good for a microproject?\n> > I will also appreciate any guides.\n>\n> Yeah, I think it could be a good microproject.\n>\n> The functions you mention don't modify the strings resulting from the\n> split, so those strings don't need to be strbufs. This should be noted\n> in the commit message (which should provide the reasons why the change\n> makes sense).\n\nHello Christian,\nThank you very much for your response.\nI will now prepare the patch.\n\nRegards\nBello.\n"}]}