Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.326%. Comparing base (611e567) to head (0946eee).
Report is 6 commits behind head on master.

@@             Coverage Diff             @@
##              master     #4789   +/-   ##
===========================================
  Coverage     58.326%   58.326%           
  Complexity      2514      2514           
===========================================
  Files            671       671           
  Lines          38782     38782           
  Branches        7041      7041           
===========================================
  Hits           22620     22620           
  Misses         13271     13271           
  Partials        2891      2891           

It seems to me that this refers to an old method that no longer exists, but I don't think it's necessary to introduce a reference to the StaticJavaParser class. I'd opt for the Name class constructor instead.

It seems to me that this refers to an old method that no longer exists, but I don't think it's necessary to introduce a reference to the StaticJavaParser class. I'd opt for the Name class constructor instead.

I was looking for a constructor that accepts one fully qualified name string and did not find one. So I was actually glad that the comment hinted at the existence of a helper method, even though I still had to find it.

So my suggestion is to either add that link or to move StaticJavaParser.parseName() to the Name class (and having the old method call the new one).