fix spelling leveraging Oxford comma

Author: Vincent Potucek

impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultPluginXmlFactory.java

@@ -18,9 +18,6 @@
  */
 package org.apache.maven.impl;
 
-import javax.xml.stream.XMLStreamException;
-
-import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.io.Reader;
@@ -48,71 +45,58 @@
 @Named
 @Singleton
 public class DefaultPluginXmlFactory implements PluginXmlFactory {
-
     @Override
     public PluginDescriptor read(@Nonnull XmlReaderRequest request) throws XmlReaderException {
         nonNull(request, "request");
         Path path = request.getPath();
         URL url = request.getURL();
         Reader reader = request.getReader();
         InputStream inputStream = request.getInputStream();
-        if (inputStream == null && reader == null && path == null && url == null) {
+        if (path == null && url == null && reader == null && inputStream == null) {
             throw new IllegalArgumentException("path, url, reader or inputStream must be non null");
         }
-        return read(request.isAddDefaultEntities(), request.isStrict(), inputStream, reader, path, url);
-    }
-
-    private static PluginDescriptor read(
-            boolean addDefaultEntities, boolean strict, InputStream inputStream, Reader reader, Path path, URL url) {
         try {
             PluginDescriptorStaxReader xml = new PluginDescriptorStaxReader();
-            xml.setAddDefaultEntities(addDefaultEntities);
+            xml.setAddDefaultEntities(request.isAddDefaultEntities());
             if (inputStream != null) {
-                return read(xml, inputStream, strict);
+                return xml.read(inputStream, request.isStrict());
             } else if (reader != null) {
-                return xml.read(reader, strict);
+                return xml.read(reader, request.isStrict());
             } else if (path != null) {
                 try (InputStream is = Files.newInputStream(path)) {
-                    return read(xml, is, strict);
+                    return xml.read(is, request.isStrict());
+                }
+            } else {
+                try (InputStream is = url.openStream()) {
+                    return xml.read(is, request.isStrict());
                 }
             }
-            try (InputStream is = url.openStream()) {
-                return read(xml, is, strict);
-            }
-        } catch (IOException | XMLStreamException e) {
+        } catch (Exception e) {
             throw new XmlReaderException("Unable to read plugin: " + getMessage(e), getLocation(e), e);
         }
     }
 
-    private static PluginDescriptor read(PluginDescriptorStaxReader xml, InputStream is, boolean strict)
-            throws XMLStreamException {
-        return xml.read(is, strict);
-    }
-
     @Override
     public void write(XmlWriterRequest<PluginDescriptor> request) throws XmlWriterException {
         nonNull(request, "request");
+        PluginDescriptor content = nonNull(request.getContent(), "content");
         Path path = request.getPath();
         OutputStream outputStream = request.getOutputStream();
         Writer writer = request.getWriter();
         if (writer == null && outputStream == null && path == null) {
             throw new IllegalArgumentException("writer, output stream, or path must be non null");
         }
-        write(writer, request.getContent(), outputStream, path);
-    }
-
-    private static void write(Writer writer, PluginDescriptor content, OutputStream outputStream, Path path) {
         try {
             if (writer != null) {
                 new PluginDescriptorStaxWriter().write(writer, content);
             } else if (outputStream != null) {
                 new PluginDescriptorStaxWriter().write(outputStream, content);
             } else {
                 try (OutputStream os = Files.newOutputStream(path)) {
-                    new PluginDescriptorStaxWriter().write(os, content);
+                    new PluginDescriptorStaxWriter().write(outputStream, content);
                 }
             }
-        } catch (IOException | XMLStreamException e) {
+        } catch (Exception e) {
             throw new XmlWriterException("Unable to write plugin: " + getMessage(e), getLocation(e), e);
         }
     }