[
https://issues.apache.org/jira/browse/THRIFT-4382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16474194#comment-16474194
]
ASF GitHub Bot commented on THRIFT-4382:
----------------------------------------
jeking3 commented on a change in pull request #1261: THRIFT-4382: Replace the
use of Indirect Object Syntax calls to new()
URL: https://github.com/apache/thrift/pull/1261#discussion_r187947528
##########
File path: test/perl/TestServer.pl
##########
@@ -48,7 +48,7 @@
$|++;
sub usage {
- print <<EOF;
+ print <<"EOF";
Review comment:
Per http://perldoc.perl.org/perlop.html#Quote-and-Quote-like-Operators, the
original syntax is canonical (without quotes).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Replace the use of Perl Indirect Object Syntax calls to new()
> --------------------------------------------------------------
>
> Key: THRIFT-4382
> URL: https://issues.apache.org/jira/browse/THRIFT-4382
> Project: Thrift
> Issue Type: Improvement
> Components: Perl - Compiler
> Reporter: Dean Hamstead
> Priority: Minor
>
> Using Indirect Object Syntax to instantiate objects is not recommended as per
> https://perldoc.perl.org/perlobj.html#Invoking-Class-Methods
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
