user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 4/7] lei_overview: remove pointless map {} op
  2022-08-04  8:16  6% [PATCH 0/7] various op/allocation golfing Eric Wong
@ 2022-08-04  8:17  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2022-08-04  8:17 UTC (permalink / raw)
  To: meta

We can rely on //g and autovivification, here.
---
 lib/PublicInbox/LeiOverview.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/LeiOverview.pm b/lib/PublicInbox/LeiOverview.pm
index 2d3db9f4..066c40bd 100644
--- a/lib/PublicInbox/LeiOverview.pm
+++ b/lib/PublicInbox/LeiOverview.pm
@@ -143,7 +143,7 @@ sub _unbless_smsg {
 	$smsg->{dt} = iso8601(delete $smsg->{ds}); # JMAP UTCDate
 	$smsg->{pct} = get_pct($mitem) if $mitem;
 	if (my $r = delete $smsg->{references}) {
-		$smsg->{refs} = [ map { $_ } ($r =~ m/$MID_EXTRACT/go) ];
+		@{$smsg->{refs}} = ($r =~ m/$MID_EXTRACT/go);
 	}
 	if (my $m = delete($smsg->{mid})) {
 		$smsg->{'m'} = $m;

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/7] various op/allocation golfing
@ 2022-08-04  8:16  6% Eric Wong
  2022-08-04  8:17  7% ` [PATCH 4/7] lei_overview: remove pointless map {} op Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2022-08-04  8:16 UTC (permalink / raw)
  To: meta

Nothing major, but some of these reduce Perl internal pad space,
which tends to add up across codebases with many cold paths.

Eric Wong (7):
  http: coerce SERVER_PORT to integer
  over: get_xref3: modify rows in-place
  isearch: mset_to_artnums: avoid unnecessary ops
  lei_overview: remove pointless map {} op
  imap: ensure_slices_exist: drop needless map and array
  feed: avoid unnecessary map loop in non-over path
  view: avoid intermediate array when streaming thread

 lib/PublicInbox/Feed.pm        | 13 ++++++-------
 lib/PublicInbox/HTTP.pm        |  2 +-
 lib/PublicInbox/IMAP.pm        |  7 ++-----
 lib/PublicInbox/Isearch.pm     |  5 ++---
 lib/PublicInbox/LeiOverview.pm |  2 +-
 lib/PublicInbox/Over.pm        |  8 ++++----
 lib/PublicInbox/View.pm        |  4 ++--
 7 files changed, 18 insertions(+), 23 deletions(-)

^ permalink raw reply	[relevance 6%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2022-08-04  8:16  6% [PATCH 0/7] various op/allocation golfing Eric Wong
2022-08-04  8:17  7% ` [PATCH 4/7] lei_overview: remove pointless map {} op Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).