FindBugs appeased

This commit is contained in:
Iskra Delta 2016-10-13 10:57:15 +00:00 committed by AndyScherzinger
parent c5901e7a70
commit bf97ba9c7d
No known key found for this signature in database
GPG key ID: 6CADC7E3523C308B
5 changed files with 14 additions and 18 deletions

View file

@ -39,8 +39,6 @@ import com.owncloud.android.ui.activity.Preferences;
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
/**
* Main Application of the project

View file

@ -1113,10 +1113,8 @@ public class FileDataStorageManager {
}
OCShare share = null;
if (cursor != null) {
if (cursor.moveToFirst()) {
share = createShareInstance(cursor);
}
if (cursor != null && cursor.moveToFirst()) {
share = createShareInstance(cursor);
cursor.close();
}
return share;
@ -1606,14 +1604,11 @@ public class FileDataStorageManager {
}
ArrayList<OCShare> shares = new ArrayList<OCShare>();
OCShare share = null;
if (cursor != null) {
if (cursor.moveToFirst()) {
do {
share = createShareInstance(cursor);
shares.add(share);
} while (cursor.moveToNext());
}
if (cursor != null && cursor.moveToFirst()) {
do {
share = createShareInstance(cursor);
shares.add(share);
} while (cursor.moveToNext());
cursor.close();
}

View file

@ -42,6 +42,7 @@ import com.owncloud.android.utils.FileStorageUtils;
import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.lang.ref.WeakReference;

View file

@ -52,8 +52,10 @@ import com.owncloud.android.ui.dialog.ShareLinkToDialog;
import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.FileReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.nio.charset.StandardCharsets;
import java.util.Collection;
import java.util.List;
import java.util.regex.Matcher;

View file

@ -291,7 +291,7 @@ public class FileStorageUtils {
List<File> files = new ArrayList<File>(Arrays.asList(filesArray));
Collections.sort(files, new Comparator<File>() {
@SuppressFBWarnings(value = "Bx")
@SuppressFBWarnings(value = "Bx", justification = "Would require stepping up API level")
public int compare(File o1, File o2) {
Long obj1 = o1.lastModified();
return multiplier * obj1.compareTo(o2.lastModified());
@ -309,7 +309,7 @@ public class FileStorageUtils {
final int multiplier = mSortAscending ? 1 : -1;
Collections.sort(files, new Comparator<OCFile>() {
@SuppressFBWarnings(value = "Bx")
@SuppressFBWarnings(value = "Bx", justification = "Would require stepping up API level")
public int compare(OCFile o1, OCFile o2) {
if (o1.isFolder() && o2.isFolder()) {
Long obj1 = o1.getFileLength();
@ -338,7 +338,7 @@ public class FileStorageUtils {
List<File> files = new ArrayList<File>(Arrays.asList(filesArray));
Collections.sort(files, new Comparator<File>() {
@SuppressFBWarnings(value = "Bx")
@SuppressFBWarnings(value = "Bx", justification = "Would require stepping up API level")
public int compare(File o1, File o2) {
if (o1.isDirectory() && o2.isDirectory()) {
Long obj1 = getFolderSize(o1);
@ -362,7 +362,7 @@ public class FileStorageUtils {
* Sorts list by Name
* @param files files to sort
*/
@SuppressFBWarnings(value = "Bx")
@SuppressFBWarnings(value = "Bx", justification = "Would require stepping up API level")
public static Vector<OCFile> sortOCFilesByName(Vector<OCFile> files){
final int multiplier = mSortAscending ? 1 : -1;