You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by "Bernd Eckenfels (JIRA)" <ji...@apache.org> on 2018/11/02 23:03:00 UTC

[jira] [Resolved] (VFS-678) LGTM (automatic code review) issues

     [ https://issues.apache.org/jira/browse/VFS-678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bernd Eckenfels resolved VFS-678.
---------------------------------
    Resolution: Fixed

Ok, no LGTM alerts left.

> LGTM (automatic code review) issues
> -----------------------------------
>
>                 Key: VFS-678
>                 URL: https://issues.apache.org/jira/browse/VFS-678
>             Project: Commons VFS
>          Issue Type: Bug
>    Affects Versions: 2.2
>            Reporter: Bernd Eckenfels
>            Assignee: Bernd Eckenfels
>            Priority: Minor
>             Fix For: 2.2.1
>
>
> The LGTM.com code review system has some issues with VFS2 code:
> https://lgtm.com/projects/g/apache/commons-vfs/alerts/?mode=list
> Namely: 
> DefaultCryptor.java: 111,114 - index might be equal to array length
> MonitorInputStream.java: 60, 86 - method overwrites a synchronized method
> UserAuthenticationData.java: 132 - test always false
> Shell.java: 157 - array to string
> DefaultFileMonitor.java: 493: - may be null
> ShowFileTask.java: 126 - resource leak
> FileName.java: 144 - javadoc
> CompressedFileFileProvider.java: 42 - javadoc



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)