Skip Menu |

This queue is for tickets about the Text-Diff CPAN distribution.

Report information
The Basics
Id: 128669
Status: new
Priority: 0/
Queue: Text-Diff

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 1.45
Fixed in: (no value)



Subject: Table plugin: "use utf8" not necessary
The "use utf8" line in the escape subroutine (https://metacpan.org/source/NEILB/Text-Diff-1.45/lib/Text/Diff/Table.pm#L60 ) is very probably not necessary. "use utf8" is only needed if there are literal utf8 characters in the source, which is not the case here. By removing the "use utf8" line the whole string-eval construct may be simplified; the "escape" sub could be made a normal subroutine.