nits

Signed-off-by: Jack Cherng <jfcherng@gmail.com>

Author: jfcherng

UPGRADING/UPGRADING_v6.md

@@ -9,7 +9,7 @@ there is no breaking change for you so you do not have to do anything.
 
 ### External Breaking Changes
 
-- `Diff` has been renamed to `Differ`.
+- The `Diff` class has been renamed to `Differ`.
   It's easy to adapt to this by changing the class name.
 
 - Now a `Renderer` has `render()` API, but a `Differ` does not.

src/Differ.php

@@ -44,7 +44,7 @@ final class Differ
     private $isCacheDirty = true;
 
     /**
-     * @var null|SequenceMatcher the sequence matcher
+     * @var SequenceMatcher the sequence matcher
      */
     private $sequenceMatcher;
 

src/Renderer/Html/AbstractHtml.php

@@ -37,6 +37,8 @@ abstract class AbstractHtml extends AbstractRenderer
      * based differences. Generally called by subclasses that generate a
      * HTML based diff and return an array of the changes to show in the diff.
      *
+     * @param Differ $differ the differ object
+     *
      * @return array an array of the generated changes, suitable for presentation in HTML
      */
     public function getChanges(Differ $differ): array
@@ -47,9 +49,6 @@ public function getChanges(Differ $differ): array
             $this->options
         );
 
-        // As we'll be modifying old & new to include our change markers,
-        // we need to get the contents and store them here. That way
-        // we're not going to destroy the original data
         $old = $differ->getOld();
         $new = $differ->getNew();