Refactor bulk collection operations

Author: miasma

commonmark-ext-image-attributes/src/main/java/org/commonmark/ext/image/attributes/internal/ImageAttributesAttributeProvider.java

@@ -26,9 +26,7 @@ public void setAttributes(Node node, String tagName, final Map<String, String> a
                 public void visit(CustomNode node) {
                     if (node instanceof ImageAttributes) {
                         ImageAttributes imageAttributes = (ImageAttributes) node;
-                        for (Map.Entry<String, String> entry : imageAttributes.getAttributes().entrySet()) {
-                            attributes.put(entry.getKey(), entry.getValue());
-                        }
+                        attributes.putAll(imageAttributes.getAttributes());
                         // Now that we have used the image attributes we remove the node.
                         imageAttributes.unlink();
                     }

commonmark/src/main/java/org/commonmark/internal/DocumentParser.java

@@ -105,9 +105,8 @@ public static Set<Class<? extends Block>> getDefaultBlockParserTypes() {
     }
 
     public static List<BlockParserFactory> calculateBlockParserFactories(List<BlockParserFactory> customBlockParserFactories, Set<Class<? extends Block>> enabledBlockTypes) {
-        List<BlockParserFactory> list = new ArrayList<>();
         // By having the custom factories come first, extensions are able to change behavior of core syntax.
-        list.addAll(customBlockParserFactories);
+        List<BlockParserFactory> list = new ArrayList<>(customBlockParserFactories);
         for (Class<? extends Block> blockType : enabledBlockTypes) {
             list.add(NODES_TO_CORE_FACTORIES.get(blockType));
         }