Login
Decided that my last modification to is_method_call was a bad idea.
authorJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Fri, 23 Mar 2007 06:28:54 +0000 (06:28 +0000)
committerJeffrey Ryan Thalhammer <jeff@imaginative-software.com>
Fri, 23 Mar 2007 06:28:54 +0000 (06:28 +0000)
commitf7a00b57a09596e4e17924e0af0eb0870837ccc7
tree9d8426de11c49f566b3f31513d39a6ec4e7f40d4
parent8053d3742bb20e0c47c2099bad5921231196fb8a
Decided that my last modification to is_method_call was a bad idea.
So I reverted is_method_call back to its old behavior.

But now there is a new method, is_class_name(), which returns true
when given a bareword that is on the left side of "->".  For example,
it would return true for the "Foo::Bar" element in these expressions:

  my $whatever = Foo::Bar->new()
  Foo::Bar->quux();

The terminology used in the code is starting to get a little confusing.
Words like "class", "package", "method", "subroutine" are starting to
get jumbled up.  Also don't have a good convention of knowing when an
variable refers to a real PPI element, or just the string-ified form
of the element.  Will have to sort this out later.
lib/Perl/Critic/Utils.pm