Skip to content

Commit 09a405e

Browse files
committed
Code tidy
Signed-off-by: Jack Cherng <[email protected]>
1 parent 6be0a0e commit 09a405e

File tree

5 files changed

+12
-6
lines changed

5 files changed

+12
-6
lines changed

src/Renderer/AbstractRenderer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
use Jfcherng\Diff\Utility\Language;
99

1010
/**
11-
* Abstract class for diff renderers.
11+
* Base class for diff renderers.
1212
*/
1313
abstract class AbstractRenderer implements RendererInterface
1414
{

src/Renderer/Html/AbstractHtml.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
use Jfcherng\Utility\MbString;
1313

1414
/**
15-
* Base renderer for rendering HTML based diffs.
15+
* Base renderer for rendering HTML-based diffs.
1616
*/
1717
abstract class AbstractHtml extends AbstractRenderer
1818
{
@@ -137,8 +137,8 @@ protected function renderChangedExtent(AbstractLineRenderer $lineRenderer, strin
137137
{
138138
static $mbFrom, $mbTo;
139139

140-
$mbFrom = $mbFrom ?? new MbString('', 'UTF-8');
141-
$mbTo = $mbTo ?? new MbString('', 'UTF-8');
140+
$mbFrom = $mbFrom ?? new MbString();
141+
$mbTo = $mbTo ?? new MbString();
142142

143143
$mbFrom->set($from);
144144
$mbTo->set($to);
@@ -243,7 +243,7 @@ protected function htmlSafe(string $string): string
243243
protected function htmlFixSpaces(string $string): string
244244
{
245245
return \preg_replace_callback(
246-
'# {2,}#S', // only fix for more than 1 space
246+
'/ {2,}/S', // only fix for more than 1 space
247247
function (array $matches): string {
248248
$count = \strlen($matches[0]);
249249

src/Renderer/Html/LineRenderer/AbstractLineRenderer.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@
66

77
use Jfcherng\Diff\SequenceMatcher;
88

9+
/**
10+
* Base renderer for rendering HTML-based line diffs.
11+
*/
912
abstract class AbstractLineRenderer implements LineRendererInterface
1013
{
1114
/**

src/Renderer/RendererConstant.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ final class RendererConstant
2525
/**
2626
* The delimiter to be used as the glue in string/array functions.
2727
*
28-
* this delimiter contains chars from the Unicode reserved areas
28+
* this delimiter contains chars from the 15-16th Unicode reserved areas
2929
* hopefully, it won't appear in our lines
3030
*
3131
* @var string

src/Renderer/Text/AbstractText.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@
66

77
use Jfcherng\Diff\Renderer\AbstractRenderer;
88

9+
/**
10+
* Base renderer for rendering text-based diffs.
11+
*/
912
abstract class AbstractText extends AbstractRenderer
1013
{
1114
/**

0 commit comments

Comments
 (0)