]> Sergey Matveev's repositories - public-inbox.git/commit
get rid of unnecessary bytes::length usage
authorEric Wong <e@80x24.org>
Thu, 26 Aug 2021 12:33:31 +0000 (12:33 +0000)
committerEric Wong <e@80x24.org>
Sat, 28 Aug 2021 10:36:58 +0000 (10:36 +0000)
commit5c8909925072804901e9c3b45bbf25446d379e7b
tree32b55b97dc531395284dc9458634c9f5b8c4d5b8
parent26c635060dcae35feae836b02a18a6a11e408312
get rid of unnecessary bytes::length usage

The only place where we could return wide characters with -httpd
was the raw $INBOX_DIR/description text, which is now converted
to octets.

All daemon (HTTP/NNTP/IMAP) sockets are opened in binary mode,
so length() and bytes::length() are equivalent on reads.  For
socket writes, any non-octet data would warn about wide characters
and we are strict in warnings with test_httpd.

All gzipped buffers are also octets, as is PublicInbox::Eml->body,
and anything from PerlIO objects ("git cat-file --batch" output,
filesystems), so bytes::length was unnecessary in all those places.
16 files changed:
lib/PublicInbox/HTTP.pm
lib/PublicInbox/ManifestJsGz.pm
lib/PublicInbox/NNTP.pm
lib/PublicInbox/View.pm
lib/PublicInbox/ViewVCS.pm
lib/PublicInbox/WWW.pm
lib/PublicInbox/WwwAttach.pm
lib/PublicInbox/WwwHighlight.pm
lib/PublicInbox/WwwListing.pm
lib/PublicInbox/WwwStatic.pm
lib/PublicInbox/WwwStream.pm
lib/PublicInbox/WwwText.pm
t/psgi_search.t
t/search-thr-index.t
t/www_listing.t
xt/cmp-msgstr.t