diff options
| author | http://jneen.net/ <jneen@jneen.net> | 2016-12-13 12:15:42 -0800 |
|---|---|---|
| committer | http://jneen.net/ <jneen@jneen.net> | 2016-12-21 09:51:42 -0800 |
| commit | 1b8a576d3fe9d283ad20ebb863bd11ec58eb5fb5 (patch) | |
| tree | 090892af4e09fd040e72212e2390d1b3bba321f0 /lib/gitlab/email/reply_parser.rb | |
| parent | f02f238d52f061ee386438fb8bbe14b379e41f42 (diff) | |
| download | gitlab-ce-1b8a576d3fe9d283ad20ebb863bd11ec58eb5fb5.tar.gz | |
Be more certain in the comment
Diffstat (limited to 'lib/gitlab/email/reply_parser.rb')
| -rw-r--r-- | lib/gitlab/email/reply_parser.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/gitlab/email/reply_parser.rb b/lib/gitlab/email/reply_parser.rb index 971afe4a878..7dedd5ec4aa 100644 --- a/lib/gitlab/email/reply_parser.rb +++ b/lib/gitlab/email/reply_parser.rb @@ -18,8 +18,9 @@ module Gitlab # not using /\s+$/ here because that deletes empty lines body = body.gsub(/[ \t]$/, '') - # TODO [jneen]: do we want to allow empty-quoting? (replies only containing a blockquote) - # EmailReplyTrimmer allows this as a special case, so we detect it manually here. + # NOTE: We currently don't support empty quotes. + # EmailReplyTrimmer allows this as a special case, + # so we detect it manually here. return "" if body.lines.all? { |l| l.strip.empty? || l.start_with?('>') } body.force_encoding(encoding).encode("UTF-8") |