diff --git a/forge-game/src/main/java/forge/game/KeywordsChange.java b/forge-game/src/main/java/forge/game/KeywordsChange.java index fb281581ad9..c42cdc2e587 100644 --- a/forge-game/src/main/java/forge/game/KeywordsChange.java +++ b/forge-game/src/main/java/forge/game/KeywordsChange.java @@ -17,7 +17,6 @@ */ package forge.game; -import java.util.ArrayList; import java.util.List; import com.google.common.collect.Lists; @@ -31,27 +30,21 @@ import com.google.common.collect.Lists; * @version $Id$ */ public class KeywordsChange { - // takes care of individual card types - private List keywords = new ArrayList(); - private List removeKeywords = new ArrayList(); - private boolean removeAllKeywords = false; + private final List keywords; + private final List removeKeywords; + private final boolean removeAllKeywords; /** * - * Card_Keywords. + * Construct a new {@link KeywordsChange}. * - * @param keywordList - * an ArrayList - * @param removeKeywordList - * a ArrayList - * @param removeAll - * a boolean - * @param stamp - * a long + * @param keywordList the list of keywords to add. + * @param removeKeywordList the list of keywords to remove. + * @param removeAll whether to remove all keywords. */ public KeywordsChange(final List keywordList, final List removeKeywordList, final boolean removeAll) { - this.keywords = Lists.newArrayList(keywordList); - this.removeKeywords = Lists.newArrayList(removeKeywordList); + this.keywords = keywordList == null ? Lists.newArrayList() : Lists.newArrayList(keywordList); + this.removeKeywords = removeKeywordList == null ? Lists.newArrayList() : Lists.newArrayList(removeKeywordList); this.removeAllKeywords = removeAll; }