diff options
author | Eric Wong <e@yhbt.net> | 2020-04-05 07:53:47 +0000 |
---|---|---|
committer | Eric Wong <e@yhbt.net> | 2020-04-05 22:06:20 +0000 |
commit | fec19e492eacb10f990091592f423542ab4249bd (patch) | |
tree | 075f20f90ae319b2e30e949c635fa53ff06d47f3 /lib/PublicInbox/Mbox.pm | |
parent | b6fc8916a05176ef006b07bba977b59cdf6a0bce (diff) | |
download | public-inbox-fec19e492eacb10f990091592f423542ab4249bd.tar.gz |
Using `undef EXPR' like a function call actually frees the heap memory associated with the scalar, whereas `$sv = undef' or `$sv = ""' will hold the buffer around until $sv goes out of scope. The `sv_set_undef' documentation in the perlapi(1) manpage explicitly states this: The perl equivalent is "$sv = undef;". Note that it doesn't free any string buffer, unlike "undef $sv". And I've confirmed by reading Dump() output from Devel::Peek. We'll also inline the old index_body sub in SearchIdx.pm to make the scope of the scalar more obvious. This change saves several hundred kB RSS on both -index and -httpd when hitting large emails with thousands of lines.
Diffstat (limited to 'lib/PublicInbox/Mbox.pm')
0 files changed, 0 insertions, 0 deletions