songsenand
/
cgal
mirror of https://github.com/CGAL/cgal
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Improve comments and remove useless comments

This commit is contained in:
Weisheng Si 2016-06-05 12:29:52 +10:00
parent d2f6062ac1
commit a86d84e68c
4 changed files with 12 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Cone_spanners_2/include/CGAL/Compute_cone_boundaries_2.h
View File

@ -27,9 +27,6 @@
#ifndef CGAL_COMPUTE_CONE_BOUNDARIES_2_H #ifndef CGAL_COMPUTE_CONE_BOUNDARIES_2_H
#define CGAL_COMPUTE_CONE_BOUNDARIES_2_H #define CGAL_COMPUTE_CONE_BOUNDARIES_2_H
// if leda::real is used, pls modify the following definition
//#define CGAL_USE_CORE 1
#include <iostream> #include <iostream>
#include <cstdlib> #include <cstdlib>
#include <vector> #include <vector>
@ -58,7 +55,8 @@ namespace CGAL {
* Of course, this functor can also be used in other applications where the plane needs to be divided * Of course, this functor can also be used in other applications where the plane needs to be divided
* into equally-angled cones. * into equally-angled cones.
* *
* \tparam Traits_ must be either `CGAL::Exact_predicates_exact_constructions_kernel_with_root_of` or `CGAL::Exact_predicates_inexact_constructions_kernel`. * \tparam Traits_ Must be either `CGAL::Exact_predicates_exact_constructions_kernel_with_root_of`
* or `CGAL::Exact_predicates_inexact_constructions_kernel`.
* *
*/ */
template <typename Traits_> template <typename Traits_>
@ -76,8 +74,7 @@ private:
typedef typename Traits::Aff_transformation_2 Transformation; typedef typename Traits::Aff_transformation_2 Transformation;
public: public:
/* No member variables in this class, so a custom constructor is not needed. */ /* Note: No member variables in this class, so a custom constructor is not needed. */
// Compute_cone_boundaries_2() {};
/*! \brief The operator(). /*! \brief The operator().
* *
@ -137,8 +134,7 @@ private:
typedef Kernel_type::Aff_transformation_2 Transformation; typedef Kernel_type::Aff_transformation_2 Transformation;
public: public:
/* No member variables in this class, so a Constructor is not needed. */ /* Note: No member variables in this class, so a Constructor is not needed. */
// Compute_cone_boundaries_2() {};
/* The operator(). /* The operator().
@ -160,8 +156,6 @@ public:
std::exit(1); std::exit(1);
} }
//std::cout << "Specialization is called!" << std::endl;
// Since CGAL::root_of() gives the k-th smallest root, // Since CGAL::root_of() gives the k-th smallest root,
// here -x is actually used instead of x. // here -x is actually used instead of x.
// But we want the second largest one with no need to count. // But we want the second largest one with no need to count.

2
Cone_spanners_2/include/CGAL/Cone_spanners_2/Plane_scan_tree.h
View File

@ -211,7 +211,7 @@ public:
return res; return res;
} }
/* Constant Reverse order End Iterator */ /* Constant Reverse order End Iterator */
const_reverse_iterator rend() const { const_reverse_iterator rend() const {
static const_reverse_iterator res; static const_reverse_iterator res;

20
Cone_spanners_2/include/CGAL/Construct_yao_graph_2.h
View File

@ -39,14 +39,15 @@ namespace CGAL {
/*! \ingroup PkgConeBasedSpanners /*! \ingroup PkgConeBasedSpanners
\brief A template functor for constructing Yao graphs with a given set of 2D points and \brief A template functor for constructing Yao graphs with a given set of 2D points and
a given initial direction for the cone boundaries. a given initial direction for the cone boundaries.
\tparam Traits_ must be either `CGAL::Exact_predicates_exact_constructions_kernel_with_root_of` or `CGAL::Exact_predicates_inexact_constructions_kernel`. \tparam Traits_ Must be either `CGAL::Exact_predicates_exact_constructions_kernel_with_root_of`
or `CGAL::Exact_predicates_inexact_constructions_kernel`.
\tparam Graph_ is the graph type to store the constructed cone based spanner.
It must be <A HREF="http://www.boost.org/libs/graph/doc/adjacency_list.html">`boost::adjacency_list`</A> with `Traits_::Point_2` as `VertexProperties`
\tparam Graph_ The graph type to store the constructed cone based spanner.
It must be <A HREF="http://www.boost.org/libs/graph/doc/adjacency_list.html">`boost::adjacency_list`</A>
with `Traits_::Point_2` as `VertexProperties`
*/ */
template <typename Traits_, typename Graph_> template <typename Traits_, typename Graph_>
class Construct_yao_graph_2 { class Construct_yao_graph_2 {
@ -102,12 +103,6 @@ public:
compute_cones(k, initial_direction, rays.begin()); compute_cones(k, initial_direction, rays.begin());
} }
/* \brief Copy constructor. Not needed.
\param x another Construct_yao_graph_2 object to copy from.
Construct_yao_graph_2 (const Construct_yao_graph_2& x) : cone_number(x.cone_number), rays(x.rays) {}
*/
/*! /*!
\brief Function operator to construct a Yao graph. \brief Function operator to construct a Yao graph.
@ -123,7 +118,6 @@ public:
Graph_& operator()(const PointInputIterator& start, Graph_& operator()(const PointInputIterator& start,
const PointInputIterator& end, const PointInputIterator& end,
Graph_& g) { Graph_& g) {
// add vertices into the graph // add vertices into the graph
for (PointInputIterator curr = start; curr != end; ++curr) { for (PointInputIterator curr = start; curr != end; ++curr) {
g[boost::add_vertex(g)] = *curr; g[boost::add_vertex(g)] = *curr;
@ -210,8 +204,6 @@ protected:
boost::tie(existing_e, existing)=boost::edge(*it, *min, g); boost::tie(existing_e, existing)=boost::edge(*it, *min, g);
if (!existing) if (!existing)
boost::add_edge(*it, *min, g); boost::add_edge(*it, *min, g);
//else
// std::cout << "Edge " << *it << ", " << *min << " already exists!" << std::endl;
} }
} // end of for } // end of for

2
Cone_spanners_2/include/CGAL/gnuplot_output_2.h
View File

@ -172,7 +172,7 @@ void gnuplot_output_2 (const Graph& g, const std::string& prefix)
fs << "unset ytics" << std::endl; fs << "unset ytics" << std::endl;
fs << "unset border" << std::endl; fs << "unset border" << std::endl;
/* /* Uncomment if you need the following:
ss << "set xtics" << std::endl; ss << "set xtics" << std::endl;
ss << "set ytics" << std::endl; ss << "set ytics" << std::endl;
ss << "set border" << std::endl; ss << "set border" << std::endl;