Open-Harbor/osrm-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

revert:

Browse Source 
- GCC <4.8 does not support thread_local keyword.
- sticking with boost::thread_specific_ptr<> for now
This commit is contained in:
Dennis Luxen 2014-05-06 16:01:05 +02:00
parent 46b207e08e
commit 2706a0bec1
1 changed files with 42 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
DataStructures/StaticRTree.h
View File

@ -47,6 +47,7 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#include <boost/filesystem/fstream.hpp> #include <boost/filesystem/fstream.hpp>
#include <boost/noncopyable.hpp> #include <boost/noncopyable.hpp>
#include <boost/shared_ptr.hpp> #include <boost/shared_ptr.hpp>
#include <boost/thread.hpp>
#include <algorithm> #include <algorithm>
#include <array> #include <array>
@ -60,7 +61,7 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
const static uint32_t RTREE_BRANCHING_FACTOR = 64; const static uint32_t RTREE_BRANCHING_FACTOR = 64;
const static uint32_t RTREE_LEAF_NODE_SIZE = 1024; const static uint32_t RTREE_LEAF_NODE_SIZE = 1024;
thread_local boost::filesystem::ifstream thread_local_rtree_stream; static boost::thread_specific_ptr<boost::filesystem::ifstream> thread_local_rtree_stream;
// Implements a static, i.e. packed, R-tree // Implements a static, i.e. packed, R-tree
template <class DataT, template <class DataT,
@ -164,21 +165,23 @@ class StaticRTree : boost::noncopyable
min_max_dist = std::min( min_max_dist = std::min(
min_max_dist, min_max_dist,
std::max(FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_left), std::max(
FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_left),
FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_right))); FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_right)));
min_max_dist = std::min( min_max_dist = std::min(
min_max_dist, min_max_dist,
std::max(FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_right), std::max(
FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_right),
FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_right))); FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_right)));
min_max_dist = min_max_dist = std::min(
std::min(min_max_dist, min_max_dist,
std::max(FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_right), std::max(FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_right),
FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_left))); FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_left)));
min_max_dist = min_max_dist = std::min(
std::min(min_max_dist, min_max_dist,
std::max(FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_left), std::max(FixedPointCoordinate::ApproximateEuclideanDistance(location, lower_left),
FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_left))); FixedPointCoordinate::ApproximateEuclideanDistance(location, upper_left)));
return min_max_dist; return min_max_dist;
@ -478,7 +481,10 @@ class StaticRTree : boost::noncopyable
leaf_node_file.read((char *)&m_element_count, sizeof(uint64_t)); leaf_node_file.read((char *)&m_element_count, sizeof(uint64_t));
leaf_node_file.close(); leaf_node_file.close();
thread_local_rtree_stream.close(); if (thread_local_rtree_stream.get())
{
thread_local_rtree_stream->close();
}
// SimpleLogger().Write() << tree_size << " nodes in search tree"; // SimpleLogger().Write() << tree_size << " nodes in search tree";
// SimpleLogger().Write() << m_element_count << " elements in leafs"; // SimpleLogger().Write() << m_element_count << " elements in leafs";
@ -501,7 +507,8 @@ class StaticRTree : boost::noncopyable
m_search_tree[0].minimum_bounding_rectangle.GetMinDist(input_coordinate); m_search_tree[0].minimum_bounding_rectangle.GetMinDist(input_coordinate);
traversal_queue.emplace(0, current_min_dist); traversal_queue.emplace(0, current_min_dist);
BOOST_ASSERT_MSG(std::numeric_limits<double>::epsilon() > std::abs(traversal_queue.top().min_dist), BOOST_ASSERT_MSG(std::numeric_limits<double>::epsilon() >
std::abs(traversal_queue.top().min_dist),
"Root element in NN Search has min dist != 0."); "Root element in NN Search has min dist != 0.");
while (!traversal_queue.empty()) while (!traversal_queue.empty())
@ -526,7 +533,8 @@ class StaticRTree : boost::noncopyable
continue; continue;
} }
double current_minimum_distance = FixedPointCoordinate::ApproximateEuclideanDistance( double current_minimum_distance =
FixedPointCoordinate::ApproximateEuclideanDistance(
input_coordinate.lat, input_coordinate.lat,
input_coordinate.lon, input_coordinate.lon,
m_coordinate_list->at(current_edge.u).lat, m_coordinate_list->at(current_edge.u).lat,
@ -540,7 +548,8 @@ class StaticRTree : boost::noncopyable
found_a_nearest_edge = true; found_a_nearest_edge = true;
} }
current_minimum_distance = FixedPointCoordinate::ApproximateEuclideanDistance( current_minimum_distance =
FixedPointCoordinate::ApproximateEuclideanDistance(
input_coordinate.lat, input_coordinate.lat,
input_coordinate.lon, input_coordinate.lon,
m_coordinate_list->at(current_edge.v).lat, m_coordinate_list->at(current_edge.v).lat,
@ -747,18 +756,19 @@ class StaticRTree : boost::noncopyable
private: private:
inline void LoadLeafFromDisk(const uint32_t leaf_id, LeafNode &result_node) inline void LoadLeafFromDisk(const uint32_t leaf_id, LeafNode &result_node)
{ {
if (!thread_local_rtree_stream.is_open()) if (!thread_local_rtree_stream.get() || !thread_local_rtree_stream->is_open())
{ {
thread_local_rtree_stream.open(m_leaf_node_filename, std::ios::in | std::ios::binary); thread_local_rtree_stream.reset(new boost::filesystem::ifstream(
m_leaf_node_filename, std::ios::in | std::ios::binary));
} }
if (!thread_local_rtree_stream.good()) if (!thread_local_rtree_stream->good())
{ {
thread_local_rtree_stream.clear(std::ios::goodbit); thread_local_rtree_stream->clear(std::ios::goodbit);
SimpleLogger().Write(logDEBUG) << "Resetting stale filestream"; SimpleLogger().Write(logDEBUG) << "Resetting stale filestream";
} }
uint64_t seek_pos = sizeof(uint64_t) + leaf_id * sizeof(LeafNode); uint64_t seek_pos = sizeof(uint64_t) + leaf_id * sizeof(LeafNode);
thread_local_rtree_stream.seekg(seek_pos); thread_local_rtree_stream->seekg(seek_pos);
thread_local_rtree_stream.read((char *)&result_node, sizeof(LeafNode)); thread_local_rtree_stream->read((char *)&result_node, sizeof(LeafNode));
} }
inline bool EdgesAreEquivalent(const FixedPointCoordinate &a, inline bool EdgesAreEquivalent(const FixedPointCoordinate &a,