Add code improvements to merge request version feature

* Add MergeRequestDiff#latest?
* Remove unnecessary variable assignment

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

1 files changed, 8 insertions, 8 deletions

diff --git a/app/models/merge_request_diff.rb b/app/models/merge_request_diff.rb
index 42ab6b620bd..e353bdb24b8 100644
--- a/app/models/merge_request_diff.rb
+++ b/app/models/merge_request_diff.rb
@@ -79,14 +79,10 @@ class MergeRequestDiff < ActiveRecord::Base
   def raw_diffs(options = {})
     if options[:ignore_whitespace_change]
       @diffs_no_whitespace ||=
-        begin
-          compare = Gitlab::Git::Compare.new(
-            repository.raw_repository,
-            safe_start_commit_sha,
-            safe_head_commit_sha
-          )
-          compare.diffs(options)
-        end
+        Gitlab::Git::Compare.new(
+          repository.raw_repository,
+          safe_start_commit_sha,
+          safe_head_commit_sha).diffs(options)
     else
       @raw_diffs ||= {}
       @raw_diffs[options] ||= load_diffs(st_diffs, options)
@@ -164,6 +160,10 @@ class MergeRequestDiff < ActiveRecord::Base
       end
   end
 
+  def latest?
+    self == merge_request.merge_request_diff
+  end
+
   private
 
   def dump_commits(commits)