Address PR Comments

Author: rdestefa

commonmark-ext-autolink/src/main/java/org/commonmark/ext/autolink/AutolinkExtension.java

@@ -60,10 +60,6 @@ public Builder linkTypes(AutolinkType... linkTypes) {
                 throw new NullPointerException("linkTypes must not be null");
             }
 
-            if (linkTypes.length == 0) {
-                throw new IllegalArgumentException("linkTypes must not be empty");
-            }
-
             return this.linkTypes(Set.of(linkTypes));
         }
 

commonmark-ext-autolink/src/main/java/org/commonmark/ext/autolink/AutolinkType.java

@@ -1,5 +1,8 @@
 package org.commonmark.ext.autolink;
 
+/**
+ * The types of strings that can be automatically turned into links.
+ */
 public enum AutolinkType {
     /**
      * URL such as {@code http://example.com}

commonmark-ext-autolink/src/main/java/org/commonmark/ext/autolink/internal/AutolinkPostProcessor.java

@@ -14,10 +14,6 @@ public class AutolinkPostProcessor implements PostProcessor {
 
     private final LinkExtractor linkExtractor;
 
-    public AutolinkPostProcessor() {
-        this(EnumSet.of(AutolinkType.URL, AutolinkType.EMAIL));
-    }
-
     public AutolinkPostProcessor(Set<AutolinkType> linkTypes) {
         if (linkTypes == null) {
             throw new NullPointerException("linkTypes must not be null");
@@ -27,7 +23,7 @@ public AutolinkPostProcessor(Set<AutolinkType> linkTypes) {
             throw new IllegalArgumentException("linkTypes must not be empty");
         }
 
-        EnumSet<LinkType> types = EnumSet.noneOf(LinkType.class);
+        var types = EnumSet.noneOf(LinkType.class);
         for (AutolinkType linkType : linkTypes) {
             switch (linkType) {
                 case URL:
@@ -99,12 +95,12 @@ private static Text createTextNode(String literal, Span span, SourceSpan sourceS
     }
 
     private static String getDestination(LinkSpan linkSpan, String linkText) {
-        LinkType type = linkSpan.getType();
+        var type = linkSpan.getType();
 
         if (type == LinkType.EMAIL) {
             return "mailto:" + linkText;
         } else if (type == LinkType.WWW) {
-            return "http://" + linkText;
+            return "https://" + linkText;
         } else {
             return linkText;
         }

commonmark-ext-autolink/src/test/java/org/commonmark/ext/autolink/AutolinkTest.java

@@ -72,7 +72,7 @@ public void wwwLinksDontWorkByDefault() {
     @Test
     public void wwwLinks() {
         String html = WWW_RENDERER.render(WWW_PARSER.parse("www.example.com"));
-        assertThat(html).isEqualTo("<p><a href=\"http://www.example.com\">www.example.com</a></p>\n");
+        assertThat(html).isEqualTo("<p><a href=\"https://www.example.com\">www.example.com</a></p>\n");
     }
 
     @Test