(Suggestions from Karel Sprenger, patch prepared by myself)
I am looking at version 0.6.28 of perleditor in EPIC 0.6.40. File SourceCritic.java contains the code:
additionalOptions.add("%f~|~%s~|~%l~|~%c~|~%m~|~%e~|~%p" + getSystemLineSeparator());
This is incorrect and should be
The "%n" instructs perlcritic to add the system line separator at the end of each formatted line, which is what EPIC expects. The current code simply adds a useless system line separator at the end of the additionalOptions entry.
Furthermore I have my doubts about "-verbose" (shouldn't this be "--verbose"?) and the order of the perlcritic command line options. The documentation states the options should come first and the file name last.
Finally, perlcritic complains the file is not in the proper directory for a package. I noticed perlcritic.el in Emacs doesn't report this problem, but there perlcritic is invoked with the directory of the file as the current working directory.
Log in to post a comment.