Skip to content

Commit

Permalink
Remove redundant variable in SaveOrderConfig
Browse files Browse the repository at this point in the history
  • Loading branch information
lenhard committed Feb 19, 2016
1 parent 7e51c23 commit 54f7d56
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,6 @@ private void setValues() {
saveInOriginalOrder.setSelected(true);
selected = false;
} else {
assert (saveOrderConfig.saveInSpecifiedOrder);
saveInSpecifiedOrder.setSelected(true);
selected = true;
}
Expand Down
11 changes: 1 addition & 10 deletions src/main/java/net/sf/jabref/logic/config/SaveOrderConfig.java
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
public class SaveOrderConfig {

public boolean saveInOriginalOrder;
public boolean saveInSpecifiedOrder;

// quick hack for outside modifications
public final SortCriterion[] sortCriteria = new SortCriterion[3];
Expand Down Expand Up @@ -61,13 +60,12 @@ public boolean equals(Object o) {
sortCriteria[2].equals(that.sortCriteria[2]);

return Objects.equals(saveInOriginalOrder, that.saveInOriginalOrder) &&
Objects.equals(saveInSpecifiedOrder, that.saveInSpecifiedOrder) &&
sortCriteriaEquals;
}

@Override
public int hashCode() {
return Objects.hash(saveInOriginalOrder, saveInSpecifiedOrder, Arrays.hashCode(sortCriteria));
return Objects.hash(saveInOriginalOrder, Arrays.hashCode(sortCriteria));
}

public SaveOrderConfig() {
Expand Down Expand Up @@ -112,16 +110,10 @@ public SaveOrderConfig(List<String> data) {

public void setSaveInOriginalOrder() {
this.saveInOriginalOrder = true;
this.saveInSpecifiedOrder = false;
}

public void setSaveInSpecifiedOrder() {
this.saveInOriginalOrder = false;
this.saveInSpecifiedOrder = true;
}

public SortCriterion[] getSortCriteria() {
return sortCriteria;
}

public static SaveOrderConfig loadExportSaveOrderFromPreferences(JabRefPreferences preferences) {
Expand Down Expand Up @@ -166,7 +158,6 @@ public Vector<String> getVector() {
if (saveInOriginalOrder) {
res.insertElementAt("original", 0);
} else {
assert saveInSpecifiedOrder;
res.insertElementAt("specified", 0);
}

Expand Down

0 comments on commit 54f7d56

Please sign in to comment.