Skip to content

Commit a4abea4

Browse files
authored
Merge pull request #9 from Perl-Toolchain-Gang/book/fix-version-string-comparisons
fix string comparisons with $] to use numeric comparison instead
2 parents fb58de1 + 606fea3 commit a4abea4

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

lib/HTTP/Tiny.pm

+5-5
Original file line numberDiff line numberDiff line change
@@ -865,7 +865,7 @@ sub _prepare_headers_and_cb {
865865
}
866866
elsif ( length $args->{content} ) {
867867
my $content = $args->{content};
868-
if ( $] ge '5.008' ) {
868+
if ( "$]" >= 5.008 ) {
869869
utf8::downgrade($content, 1)
870870
or die(qq/Wide character in request message body\n/);
871871
}
@@ -1032,7 +1032,7 @@ my $unsafe_char = qr/[^A-Za-z0-9\-\._~]/;
10321032
sub _uri_escape {
10331033
my ($self, $str) = @_;
10341034
return "" if !defined $str;
1035-
if ( $] ge '5.008' ) {
1035+
if ( "$]" >= 5.008 ) {
10361036
utf8::encode($str);
10371037
}
10381038
else {
@@ -1189,7 +1189,7 @@ sub write {
11891189
@_ == 2 || die(q/Usage: $handle->write(buf)/ . "\n");
11901190
my ($self, $buf) = @_;
11911191

1192-
if ( $] ge '5.008' ) {
1192+
if ( "$]" >= 5.008 ) {
11931193
utf8::downgrade($buf, 1)
11941194
or die(qq/Wide character in write()\n/);
11951195
}
@@ -1474,7 +1474,7 @@ sub write_content_body {
14741474
defined $data && length $data
14751475
or last;
14761476

1477-
if ( $] ge '5.008' ) {
1477+
if ( "$]" >= 5.008 ) {
14781478
utf8::downgrade($data, 1)
14791479
or die(qq/Wide character in write_content()\n/);
14801480
}
@@ -1521,7 +1521,7 @@ sub write_chunked_body {
15211521
defined $data && length $data
15221522
or last;
15231523

1524-
if ( $] ge '5.008' ) {
1524+
if ( "$]" >= 5.008 ) {
15251525
utf8::downgrade($data, 1)
15261526
or die(qq/Wide character in write_chunked_body()\n/);
15271527
}

0 commit comments

Comments
 (0)