Remove workaround removing key flag packets before adding new ones.

This workaround can be removed, as per 6c50207 setKeyFlags() will remove existing keyFlag packets on its own

Author: vanitasvitae

pg/src/main/java/org/bouncycastle/openpgp/api/OpenPGPKeyGenerator.java

@@ -10,7 +10,6 @@
 import org.bouncycastle.bcpg.PublicKeyUtils;
 import org.bouncycastle.bcpg.PublicSubkeyPacket;
 import org.bouncycastle.bcpg.S2K;
-import org.bouncycastle.bcpg.SignatureSubpacketTags;
 import org.bouncycastle.bcpg.sig.KeyFlags;
 import org.bouncycastle.openpgp.PGPException;
 import org.bouncycastle.openpgp.PGPKeyPair;
@@ -235,7 +234,6 @@ public WithPrimaryKey signOnlyKey()
                 @Override
                 public PGPSignatureSubpacketGenerator apply(PGPSignatureSubpacketGenerator subpackets)
                 {
-                    subpackets.removePacketsOfType(SignatureSubpacketTags.KEY_FLAGS);
                     subpackets.setKeyFlags(true, KeyFlags.CERTIFY_OTHER | KeyFlags.SIGN_DATA);
                     return subpackets;
                 }