searchmsg: favor direct hash access over accessor methods

This is faster, smaller, and more straighforward to me with
fewer layers of indirection.

1 files changed, 2 insertions, 31 deletions

diff --git a/lib/PublicInbox/SearchMsg.pm b/lib/PublicInbox/SearchMsg.pm
index 4522eb62..5bb00778 100644
--- a/lib/PublicInbox/SearchMsg.pm
+++ b/lib/PublicInbox/SearchMsg.pm
@@ -103,7 +103,7 @@ sub from_name {
 
 sub ts {
         my ($self) = @_;
-        $self->{ts} ||= eval { str2time($self->mime->header('Date')) } || 0;
+        $self->{ts} ||= eval { str2time($self->{mime}->header('Date')) } || 0;
 }
 
 sub to_doc_data {
@@ -146,36 +146,7 @@ sub mid ($;$) {
         }
 }
 
-sub _extract_mid { mid_clean(mid_mime($_[0]->mime)) }
-
-sub blob {
-        my ($self, $x40) = @_;
-        if (defined $x40) {
-                $self->{blob} = $x40;
-        } else {
-                $self->{blob};
-        }
-}
-
-sub mime {
-        my ($self, $mime) = @_;
-        if (defined $mime) {
-                $self->{mime} = $mime;
-        } else {
-                # TODO load from git
-                $self->{mime};
-        }
-}
-
-sub doc_id {
-        my ($self, $doc_id) = @_;
-        if (defined $doc_id) {
-                $self->{doc_id} = $doc_id;
-        } else {
-                # TODO load from xapian
-                $self->{doc_id};
-        }
-}
+sub _extract_mid { mid_clean(mid_mime($_[0]->{mime})) }
 
 sub thread_id {
         my ($self) = @_;