From e1334348889c5d690311de1d470525f36f4ba59c Mon Sep 17 00:00:00 2001
From: Paul Schaub <vanitasvitae@fsfe.org>
Date: Sun, 28 Nov 2021 13:09:21 +0100
Subject: [PATCH] Remove unused methods from SignatureSubpacketGeneratorUtil

---
 .../SignatureSubpacketGeneratorUtil.java      | 42 -------------------
 1 file changed, 42 deletions(-)

diff --git a/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketGeneratorUtil.java b/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketGeneratorUtil.java
index eeb87a10..137e5508 100644
--- a/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketGeneratorUtil.java
+++ b/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketGeneratorUtil.java
@@ -4,16 +4,12 @@
 
 package org.pgpainless.signature.subpackets;
 
-import java.util.ArrayList;
 import java.util.Date;
-import java.util.List;
 import javax.annotation.Nonnull;
 
 import org.bouncycastle.bcpg.SignatureSubpacket;
 import org.bouncycastle.bcpg.SignatureSubpacketTags;
-import org.bouncycastle.bcpg.sig.KeyFlags;
 import org.bouncycastle.openpgp.PGPSignatureSubpacketGenerator;
-import org.pgpainless.algorithm.KeyFlag;
 
 /**
  * Utility class that helps dealing with BCs SignatureSubpacketGenerator class.
@@ -24,25 +20,6 @@ public final class SignatureSubpacketGeneratorUtil {
 
     }
 
-    /**
-     * Return a list of {@link SignatureSubpacket SignatureSubpackets} from the subpacket generator, which correspond
-     * to the given {@link org.pgpainless.algorithm.SignatureSubpacket} type.
-     *
-     * @param type subpacket type
-     * @param generator subpacket generator
-     * @param <P> generic subpacket type
-     * @return possibly empty list of subpackets
-     */
-    public static <P extends SignatureSubpacket> List<P> getSubpacketsOfType(org.pgpainless.algorithm.SignatureSubpacket type,
-                                                               PGPSignatureSubpacketGenerator generator) {
-        SignatureSubpacket[] subpackets = generator.getSubpackets(type.getCode());
-        List<P> list = new ArrayList<>();
-        for (SignatureSubpacket p : subpackets) {
-            list.add((P) p);
-        }
-        return list;
-    }
-
     /**
      * Remove all packets of the given type from the {@link PGPSignatureSubpacketGenerator PGPSignatureSubpacketGenerators}
      * internal set.
@@ -95,23 +72,4 @@ public final class SignatureSubpacketGeneratorUtil {
         long secondsToExpire = SignatureSubpacketsUtil.getKeyLifetimeInSeconds(expirationDate, creationDate);
         subpacketGenerator.setKeyExpirationTime(true, secondsToExpire);
     }
-
-    /**
-     * Return true, if the subpacket generator has a {@link KeyFlags} subpacket which carries the given key flag.
-     * Returns false, if no {@link KeyFlags} subpacket is present.
-     * If there are more than one instance of a {@link KeyFlags} packet present, only the last occurrence will
-     * be tested.
-     *
-     * @param keyFlag flag to test for
-     * @param generator subpackets generator
-     * @return true if the generator has the given key flag set
-     */
-    public static boolean hasKeyFlag(KeyFlag keyFlag, PGPSignatureSubpacketGenerator generator) {
-        List<KeyFlags> keyFlagPackets = getSubpacketsOfType(org.pgpainless.algorithm.SignatureSubpacket.keyFlags, generator);
-        if (keyFlagPackets.isEmpty()) {
-            return false;
-        }
-        KeyFlags last = keyFlagPackets.get(keyFlagPackets.size() - 1);
-        return KeyFlag.hasKeyFlag(last.getFlags(), keyFlag);
-    }
 }