Change default criticality of signature subpackets to mirror those of sequoia

pgpainless-core/src/main/java/org/pgpainless/signature/builder/RevocationSignatureBuilder.java

@@ -19,7 +19,7 @@ public class RevocationSignatureBuilder extends AbstractSignatureBuilder<Revocat
 
     public RevocationSignatureBuilder(SignatureType signatureType, PGPSecretKey signingKey, SecretKeyRingProtector protector) throws PGPException {
         super(signatureType, signingKey, protector);
-        getHashedSubpackets().setRevocable(true, false);
+        getHashedSubpackets().setRevocable(false);
     }
 
     @Override

pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/BaseSignatureSubpackets.java

@@ -82,10 +82,14 @@ public interface BaseSignatureSubpackets {
 
     BaseSignatureSubpackets clearIntendedRecipientFingerprints();
 
+    BaseSignatureSubpackets setExportable(boolean isExportable);
+
     BaseSignatureSubpackets setExportable(boolean isCritical, boolean isExportable);
 
     BaseSignatureSubpackets setExportable(@Nullable Exportable exportable);
 
+    BaseSignatureSubpackets setRevocable(boolean revocable);
+
     BaseSignatureSubpackets setRevocable(boolean isCritical, boolean isRevocable);
 
     BaseSignatureSubpackets setRevocable(@Nullable Revocable revocable);

pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpackets.java

@@ -116,7 +116,7 @@ public class SignatureSubpackets
 
     @Override
     public SignatureSubpackets setIssuerKeyId(long keyId) {
-        return setIssuerKeyId(true, keyId);
+        return setIssuerKeyId(false, keyId);
     }
 
     @Override
@@ -303,7 +303,7 @@ public class SignatureSubpackets
 
     @Override
     public SignatureSubpackets setPreferredCompressionAlgorithms(Set<CompressionAlgorithm> algorithms) {
-        return setPreferredCompressionAlgorithms(true, algorithms);
+        return setPreferredCompressionAlgorithms(false, algorithms);
     }
 
     @Override
@@ -342,7 +342,7 @@ public class SignatureSubpackets
 
     @Override
     public SignatureSubpackets setPreferredSymmetricKeyAlgorithms(Set<SymmetricKeyAlgorithm> algorithms) {
-        return setPreferredSymmetricKeyAlgorithms(true, algorithms);
+        return setPreferredSymmetricKeyAlgorithms(false, algorithms);
     }
 
     @Override
@@ -381,7 +381,7 @@ public class SignatureSubpackets
 
     @Override
     public SignatureSubpackets setPreferredHashAlgorithms(Set<HashAlgorithm> algorithms) {
-        return setPreferredHashAlgorithms(true, algorithms);
+        return setPreferredHashAlgorithms(false, algorithms);
     }
 
     @Override
@@ -465,6 +465,11 @@ public class SignatureSubpackets
         return new ArrayList<>(intendedRecipientFingerprintList);
     }
 
+    @Override
+    public SignatureSubpackets setExportable(boolean exportable) {
+        return setExportable(true, exportable);
+    }
+
     @Override
     public SignatureSubpackets setExportable(boolean isCritical, boolean isExportable) {
         return setExportable(new Exportable(isCritical, isExportable));
@@ -480,6 +485,11 @@ public class SignatureSubpackets
         return exportable;
     }
 
+    @Override
+    public SignatureSubpackets setRevocable(boolean revocable) {
+        return setRevocable(true, revocable);
+    }
+
     @Override
     public SignatureSubpackets setRevocable(boolean isCritical, boolean isRevocable) {
         return setRevocable(new Revocable(isCritical, isRevocable));
@@ -530,7 +540,7 @@ public class SignatureSubpackets
 
     @Override
     public SignatureSubpackets setRevocationReason(RevocationAttributes revocationAttributes) {
-        return setRevocationReason(true, revocationAttributes);
+        return setRevocationReason(false, revocationAttributes);
     }
 
     @Override