diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2015-01-15 14:37:11 -0800 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2015-01-15 14:37:11 -0800 |
commit | c5ba87a2fbd4eb0bcce5431683e26e8d345db5c4 (patch) | |
tree | 3337cbcf38a72eaa275be7e995c7ed7c83002ca4 /app/models/commit.rb | |
parent | 9c4015f3e8080f41daaac82cfe1465f6839ca427 (diff) | |
parent | cd688a60111853f63413a87ad6632ad57368e886 (diff) | |
download | gitlab-ce-c5ba87a2fbd4eb0bcce5431683e26e8d345db5c4.tar.gz |
Merge pull request #8096 from cirosantilli/regex-to-string
Replace regex methods by string ones since faster and more readable
Diffstat (limited to 'app/models/commit.rb')
-rw-r--r-- | app/models/commit.rb | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/app/models/commit.rb b/app/models/commit.rb index 37dd371ec00..baccf286740 100644 --- a/app/models/commit.rb +++ b/app/models/commit.rb @@ -75,11 +75,11 @@ class Commit return no_commit_message if title.blank? - title_end = title.index(/\n/) + title_end = title.index("\n") if (!title_end && title.length > 100) || (title_end && title_end > 100) title[0..79] << "…".html_safe else - title.split(/\n/, 2).first + title.split("\n", 2).first end end @@ -87,11 +87,11 @@ class Commit # # cut off, ellipses (`&hellp;`) are prepended to the commit message. def description - title_end = safe_message.index(/\n/) + title_end = safe_message.index("\n") @description ||= if (!title_end && safe_message.length > 100) || (title_end && title_end > 100) "…".html_safe << safe_message[80..-1] else - safe_message.split(/\n/, 2)[1].try(:chomp) + safe_message.split("\n", 2)[1].try(:chomp) end end |