From 869d57c2f4a581f02e997da597575dd50149ccad Mon Sep 17 00:00:00 2001
From: Alexandre Archambault <alexandre.archambault@gmail.com>
Date: Thu, 28 Apr 2016 23:17:07 +0200
Subject: [PATCH] Tidying

---
 .../main/scala-2.10/coursier/MakeIvyXml.scala | 38 ++++++++++---------
 1 file changed, 20 insertions(+), 18 deletions(-)

diff --git a/plugin/src/main/scala-2.10/coursier/MakeIvyXml.scala b/plugin/src/main/scala-2.10/coursier/MakeIvyXml.scala
index 8a75ae8d9..4c1bc7c74 100644
--- a/plugin/src/main/scala-2.10/coursier/MakeIvyXml.scala
+++ b/plugin/src/main/scala-2.10/coursier/MakeIvyXml.scala
@@ -6,27 +6,26 @@ object MakeIvyXml {
 
   def apply(project: Project): Node = {
 
-    val baseInfoAttrs = <x
-      organisation={project.module.organization}
-      module={project.module.name}
-      revision={project.version}
-    />.attributes
-
-    val infoAttrs = project.module.attributes.foldLeft(baseInfoAttrs) {
+    val infoAttrs = project.module.attributes.foldLeft[xml.MetaData](xml.Null) {
       case (acc, (k, v)) =>
         new PrefixedAttribute("e", k, v, acc)
     }
 
     val licenseElems = project.info.licenses.map {
       case (name, urlOpt) =>
-        var n = <license name={name} />
-        for (url <- urlOpt)
-          n = n % <x url={url} />.attributes
-        n
+        val n = <license name={name} />
+
+        urlOpt.fold(n) { url =>
+          n % <x url={url} />.attributes
+        }
     }
 
     val infoElem = {
-      <info>
+      <info
+        organisation={project.module.organization}
+        module={project.module.name}
+        revision={project.version}
+      >
         {licenseElems}
         <description>{project.info.description}</description>
       </info>
@@ -34,10 +33,11 @@ object MakeIvyXml {
 
     val confElems = project.configurations.toVector.map {
       case (name, extends0) =>
-        var n = <conf name={name} visibility="public" description="" />
+        val n = <conf name={name} visibility="public" description="" />
         if (extends0.nonEmpty)
-          n = n % <x extends={extends0.mkString(",")} />.attributes
-        n
+          n % <x extends={extends0.mkString(",")} />.attributes
+        else
+          n
     }
 
     val publications = project
@@ -47,10 +47,12 @@ object MakeIvyXml {
 
     val publicationElems = publications.map {
       case (pub, configs) =>
-        var n = <artifact name={pub.name} type={pub.`type`} ext={pub.ext} conf={configs.mkString(",")} />
+        val n = <artifact name={pub.name} type={pub.`type`} ext={pub.ext} conf={configs.mkString(",")} />
+
         if (pub.classifier.nonEmpty)
-          n = n % <x e:classifier={pub.classifier} />.attributes
-        n
+          n % <x e:classifier={pub.classifier} />.attributes
+        else
+          n
     }
 
     val dependencyElems = project.dependencies.toVector.map {