From 1ce28a09af1587e98e900180b192b965cd7cc29a Mon Sep 17 00:00:00 2001
From: Paul Schaub <vanitasvitae@fsfe.org>
Date: Mon, 18 Jan 2021 18:12:53 +0100
Subject: [PATCH] Make UserId constructor private in favor of factory methods

---
 .../src/main/java/org/pgpainless/key/util/UserId.java     | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/pgpainless-core/src/main/java/org/pgpainless/key/util/UserId.java b/pgpainless-core/src/main/java/org/pgpainless/key/util/UserId.java
index bb9bb4a8..11fb1cfe 100644
--- a/pgpainless-core/src/main/java/org/pgpainless/key/util/UserId.java
+++ b/pgpainless-core/src/main/java/org/pgpainless/key/util/UserId.java
@@ -15,13 +15,13 @@
  */
 package org.pgpainless.key.util;
 
-public class UserId implements CharSequence {
+public final class UserId implements CharSequence {
 
     private final String name;
     private final String comment;
     private final String email;
 
-    public UserId(String name, String comment, String email) {
+    private UserId(String name, String comment, String email) {
         this.name = name;
         this.comment = comment;
         this.email = email;
@@ -49,6 +49,10 @@ public class UserId implements CharSequence {
         return new UserId(null, null, email);
     }
 
+    public static UserId nameAndEmail(String name, String email) {
+        return withName(name).noComment().withEmail(email);
+    }
+
     public static WithComment withName(String name) {
         if (name == null) {
             throw new IllegalArgumentException("Name must not be null.");