about summary refs log tree commit homepage
diff options
context:
space:
mode:
-rw-r--r--TODO5
-rw-r--r--lib/PublicInbox/Feed.pm4
-rw-r--r--lib/PublicInbox/MDA.pm2
-rw-r--r--lib/PublicInbox/View.pm4
4 files changed, 12 insertions, 3 deletions
diff --git a/TODO b/TODO
index e69de29b..00df8bd3 100644
--- a/TODO
+++ b/TODO
@@ -0,0 +1,5 @@
+* configurable constants (quote folding, index limits)
+* use tags as date-based skiplists for navigating history
+* per-message views should be able to quickly navigate do replies.
+  (n.b.: this is SMTP, we must account for out-of-order delivery)
+* NNTP server (is there demand for one?)
diff --git a/lib/PublicInbox/Feed.pm b/lib/PublicInbox/Feed.pm
index 12ac8a95..6996f374 100644
--- a/lib/PublicInbox/Feed.pm
+++ b/lib/PublicInbox/Feed.pm
@@ -9,8 +9,8 @@ use Date::Parse qw(strptime str2time);
 use PublicInbox::Hval;
 use PublicInbox::GitCatFile;
 use constant {
-        DATEFMT => '%Y-%m-%dT%H:%M:%SZ',
-        MAX_PER_PAGE => 25,
+        DATEFMT => '%Y-%m-%dT%H:%M:%SZ', # atom standard
+        MAX_PER_PAGE => 25, # this needs to be tunable
 };
 
 # main function
diff --git a/lib/PublicInbox/MDA.pm b/lib/PublicInbox/MDA.pm
index ee4d0afe..e2b2f914 100644
--- a/lib/PublicInbox/MDA.pm
+++ b/lib/PublicInbox/MDA.pm
@@ -6,7 +6,7 @@ use warnings;
 use Email::Address;
 use Encode qw/decode/;
 use Date::Parse qw(strptime);
-use constant MAX_SIZE => 1024 * 500; # same as spamc default
+use constant MAX_SIZE => 1024 * 500; # same as spamc default, should be tunable
 use constant cmd => qw/ssoma-mda -1/;
 
 # drop plus addressing for matching
diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm
index 52e8f0b2..674d7b7c 100644
--- a/lib/PublicInbox/View.pm
+++ b/lib/PublicInbox/View.pm
@@ -8,8 +8,11 @@ use URI::Escape qw/uri_escape_utf8/;
 use Encode qw/find_encoding/;
 use Encode::MIME::Header;
 use Email::MIME::ContentType qw/parse_content_type/;
+
+# TODO: make these constants tunable
 use constant MAX_INLINE_QUOTED => 5;
 use constant MAX_TRUNC_LEN => 72;
+
 *ascii_html = *PublicInbox::Hval::ascii_html;
 
 my $enc_utf8 = find_encoding('UTF-8');
@@ -92,6 +95,7 @@ sub add_text_body_short {
         my ($enc, $part, $part_nr, $full_pfx) = @_;
         my $n = 0;
         my $s = ascii_html($enc->decode($part->body));
+        # TODO: fold the "so-and-so wrote:" attribute line here, too:
         $s =~ s!^((?:(?:>[^\n]*)\n)+)!
                 my $cur = $1;
                 my @lines = split(/\n/, $cur);