Refactor according to findbugs

Autoported commit.
Original commit hash: [6f034ab85]
Manual files:
kernel/findbugs-filter.xml

Author: ar3em

itext/itext.kernel/itext/kernel/pdf/PdfCatalog.cs

@@ -44,6 +44,7 @@ source product.
 using System;
 using System.Collections.Generic;
 using Common.Logging;
+using iText.IO.Util;
 using iText.Kernel;
 using iText.Kernel.Pdf.Action;
 using iText.Kernel.Pdf.Collection;
@@ -69,6 +70,14 @@ public class PdfCatalog : PdfObjectWrapper<PdfDictionary> {
 
         private bool outlineMode;
 
+        private static readonly ICollection<PdfName> PAGE_MODES = new HashSet<PdfName>(JavaUtil.ArraysAsList(PdfName
+            .UseNone, PdfName.UseOutlines, PdfName.UseThumbs, PdfName.FullScreen, PdfName.UseOC, PdfName.UseAttachments
+            ));
+
+        private static readonly ICollection<PdfName> PAGE_LAYOUTS = new HashSet<PdfName>(JavaUtil.ArraysAsList(PdfName
+            .SinglePage, PdfName.OneColumn, PdfName.TwoColumnLeft, PdfName.TwoColumnRight, PdfName.TwoPageLeft, PdfName
+            .TwoPageRight));
+
         protected internal PdfCatalog(PdfDictionary pdfObject)
             : base(pdfObject) {
             //This HashMap contents all pages of the document and outlines associated to them
@@ -173,9 +182,7 @@ public virtual iText.Kernel.Pdf.PdfCatalog SetAdditionalAction(PdfName key, PdfA
         /// <param name="pageMode">page mode.</param>
         /// <returns>current instance of PdfCatalog</returns>
         public virtual iText.Kernel.Pdf.PdfCatalog SetPageMode(PdfName pageMode) {
-            if (pageMode.Equals(PdfName.UseNone) || pageMode.Equals(PdfName.UseOutlines) || pageMode.Equals(PdfName.UseThumbs
-                ) || pageMode.Equals(PdfName.FullScreen) || pageMode.Equals(PdfName.UseOC) || pageMode.Equals(PdfName.
-                UseAttachments)) {
+            if (PAGE_MODES.Contains(pageMode)) {
                 return Put(PdfName.PageMode, pageMode);
             }
             return this;
@@ -188,9 +195,7 @@ public virtual PdfName GetPageMode() {
         /// <summary>This method sets a page layout of the document</summary>
         /// <param name="pageLayout"/>
         public virtual iText.Kernel.Pdf.PdfCatalog SetPageLayout(PdfName pageLayout) {
-            if (pageLayout.Equals(PdfName.SinglePage) || pageLayout.Equals(PdfName.OneColumn) || pageLayout.Equals(PdfName
-                .TwoColumnLeft) || pageLayout.Equals(PdfName.TwoColumnRight) || pageLayout.Equals(PdfName.TwoPageLeft)
-                 || pageLayout.Equals(PdfName.TwoPageRight)) {
+            if (PAGE_LAYOUTS.Contains(pageLayout)) {
                 return Put(PdfName.PageLayout, pageLayout);
             }
             return this;

port-hash

@@ -1 +1 @@
-aa0c58894a091657d1ae8f13810606cea44884b6
+6f034ab850913cfde4dc59309b71ba81973597e2