From 20fca5978b55606062cdaf4c94dec82f5318a62a Mon Sep 17 00:00:00 2001
From: Adrian Kummerländer
Date: Sun, 11 May 2014 20:09:34 +0200
Subject: Switched internal DomDocumentCache structure to std::stack * using
 std::unordered_map for named cache lookups was a nice idea but is not useful
 when one keeps in mind that: ** we don't want cached external function
 responses in a situation where we are able to execute transformations inside
 transformations ** cached responses mean that we can not read a xml file,
 overwrite it using another transformation and then read it again * the
 currently available InputXSLT frontend only processes a single transformation
 anyway, so increased memory requirements should not pose a problem ** if a
 future frontend should offer batch processing of transformation tasks, one
 could implement a cache reset method to free memory after a transformation
 has been processed

---
 src/support/dom/document_cache.cc | 27 ++++-----------------------
 1 file changed, 4 insertions(+), 23 deletions(-)

(limited to 'src/support/dom/document_cache.cc')

diff --git a/src/support/dom/document_cache.cc b/src/support/dom/document_cache.cc
index 81d90aa..0ad540c 100644
--- a/src/support/dom/document_cache.cc
+++ b/src/support/dom/document_cache.cc
@@ -6,33 +6,14 @@ namespace InputXSLT {
 
 DomDocumentCache::DomDocumentCache():
 	write_mutex_(),
-	map_() { }
+	cache_() { }
 
-DomDocumentCache::optional_item DomDocumentCache::get(const std::string& key) {
-	auto itemIter = this->map_.find(key);
-
-	if ( itemIter == this->map_.end() ) {
-		return optional_item(false, nullptr);
-	} else {
-		return optional_item(true, (*itemIter).second->getXalanDocument());
-	}
-}
-
-DomDocumentCache::optional_item DomDocumentCache::create(
-	const std::string&    key,
-	xercesc::DOMDocument* document
-) {
+xalan::XalanDocument* DomDocumentCache::create(xercesc::DOMDocument* document) {
 	std::lock_guard<std::mutex> guard(this->write_mutex_);
 
-	auto result = this->map_.emplace(
-		std::make_pair(key, std::unique_ptr<item>(new item(document)))
-	);
+	this->cache_.emplace(new item(document));
 
-	if ( result.second ) {
-		return optional_item(true, (*(result.first)).second->getXalanDocument());
-	} else {
-		return optional_item(false, nullptr);
-	}
+	return this->cache_.top()->getXalanDocument();
 }
 
 }
-- 
cgit v1.2.3