We want to avoid having multiple returns out of this function.  In this case it's not needed:

My recommendation:

1. Add "Saxon" to the list of supportedXSLEngins
2. Push the test into the assignment for example:

````scala
 xsle =  engine match {
       case "Saxon" => if (depUseSaxonEEValidation) "SaxonEE" else "SaxonHE"
       case _ => engine
 }
````