From 141725d681dd69f77e993c3d59d613d1ad7014f9 Mon Sep 17 00:00:00 2001
From: Adrian Kummerlaender
Date: Sat, 12 Jul 2014 09:51:09 +0200
Subject: Marked constructDocument member methods as const * they don't modifiy
 the class state so there is no reason for them not being marked as const **
 all calling methods are also const ** this enables us to remove the
 const_cast in FunctionBase * modified external function implementations
 accordingly * inlined handleError method in FunctionTransform as it is not
 needed in multiple places anymore

---
 src/function/transform.cc | 25 ++++++-------------------
 1 file changed, 6 insertions(+), 19 deletions(-)

(limited to 'src/function/transform.cc')

diff --git a/src/function/transform.cc b/src/function/transform.cc
index 6cda7a7..7eab3fd 100644
--- a/src/function/transform.cc
+++ b/src/function/transform.cc
@@ -7,29 +7,12 @@
 #include "support/dom/result_node_facade.h"
 #include "support/error/error_capacitor.h"
 
-namespace {
-
-using InputXSLT::ErrorCapacitor;
-
-inline void handleErrors(
-	InputXSLT::ResultNodeFacade&       result,
-	const ErrorCapacitor::error_cache& errors
-) {
-	result.setAttribute("result", "error");
-
-	for ( auto&& error : errors ) {
-		result.setValueNode("error", error);
-	}
-}
-
-}
-
 namespace InputXSLT {
 
 DomDocumentCache::document_ptr FunctionTransform::constructDocument(
 	xalan::XSLTInputSource  inputSource,
 	xalan::XSLTInputSource  transformationSource
-) {
+) const {
 	DomDocumentCache::document_ptr domDocument(
 		DomDocumentCache::createDocument("content")
 	);
@@ -52,7 +35,11 @@ DomDocumentCache::document_ptr FunctionTransform::constructDocument(
 		result.setAttribute("result", "success");
 	}
 	catch (const ErrorCapacitor::exception& exception) {
-		handleErrors(result, *exception);
+		result.setAttribute("result", "error");
+
+		for ( auto&& error : *exception ) {
+			result.setValueNode("error", error);
+		}
 	}
 
 	WarningCapacitor::warning_cache_ptr warnings(
-- 
cgit v1.2.3