From 36843ccdba25f698a5e6f1d3de74529003d1c902 Mon Sep 17 00:00:00 2001 From: Nathan Ridge Date: Thu, 3 Jan 2013 01:01:55 +0000 Subject: [PATCH] [range] docs for writing adaptors: removed clarification added in r82315 (it was redundant); fixed typos that messed up numbering [SVN r82326] --- doc/html/index.html | 2 +- .../extending/method_3/method_3_1.html | 6 ---- .../extending/method_3/method_3_2.html | 31 +++++-------------- doc/reference/extending.qbk | 8 ----- 4 files changed, 9 insertions(+), 38 deletions(-) diff --git a/doc/html/index.html b/doc/html/index.html index 3415cbe..02c06d1 100644 --- a/doc/html/index.html +++ b/doc/html/index.html @@ -147,7 +147,7 @@

- +

Last revised: January 02, 2013 at 23:39:56 GMT

Last revised: January 03, 2013 at 00:57:19 GMT
diff --git a/doc/html/range/reference/extending/method_3/method_3_1.html b/doc/html/range/reference/extending/method_3/method_3_1.html index a79e7b3..6628403 100644 --- a/doc/html/range/reference/extending/method_3/method_3_1.html +++ b/doc/html/range/reference/extending/method_3/method_3_1.html @@ -90,12 +90,6 @@ const detail::reverse_forwarder reversed = detail::reverse_forwarder(); } - Note that this last step only applies if the adaptor takes no arguments. - If the adaptor takes arguments (such as indexed), - then there is no need for a variable. Instead, give the tag type - a constructor that takes the arguments (and store those arguments - in the tag object). A usage of the adaptor will then involve construction - of an object of the tag type. diff --git a/doc/html/range/reference/extending/method_3/method_3_2.html b/doc/html/range/reference/extending/method_3/method_3_2.html index 829179a..e21cd5f 100644 --- a/doc/html/range/reference/extending/method_3/method_3_2.html +++ b/doc/html/range/reference/extending/method_3/method_3_2.html @@ -80,15 +80,8 @@
  • Implement a holder class to hold the arguments required to construct - the RangeAdaptor. -
    • - -

    - The holder combines multiple parameters into one that can be passed as - the right operand of operator|(). -

    -

    -

    + the RangeAdaptor. The holder combines multiple parameters into one + that can be passed as the right operand of operator|(). 
    template<typename T>
 class replace_holder : public boost::range_detail::holder2<T>
 {
@@ -100,23 +93,15 @@
     void operator=(const replace_holder&);
 };
    -

    -

    -
    1. +
      2. +
    2. Define an instance of the holder with the name of the adaptor -
    -

    -

    static boost::range_detail::forwarder2<replace_holder>
 replaced = boost::range_detail::forwarder2<replace_holder>();
    -

    -

    -
    1. +
      2. +
    2. Define operator| -
    -

    -

    template<typename SinglePassRange>
 inline replace_range<SinglePassRange>
 operator|(SinglePassRange& rng,
@@ -133,8 +118,8 @@
     return replace_range<const SinglePassRange>(rng, f.val1, f.val2);
 }
    -

    -

    + + diff --git a/doc/reference/extending.qbk b/doc/reference/extending.qbk index a4657a2..46652d6 100644 --- a/doc/reference/extending.qbk +++ b/doc/reference/extending.qbk @@ -225,10 +225,6 @@ namespace const detail::reverse_forwarder reversed = detail::reverse_forwarder(); } `` -Note that this last step only applies if the adaptor takes no arguments. If the adaptor takes arguments -(such as `indexed`), then there is no need for a variable. Instead, give the tag type a -constructor that takes the arguments (and store those arguments in the tag object). A usage -of the adaptor will then involve construction of an object of the tag type. [endsect] @@ -285,9 +281,7 @@ public: `` # Implement a holder class to hold the arguments required to construct the RangeAdaptor. - The holder combines multiple parameters into one that can be passed as the right operand of `operator|()`. - `` template class replace_holder : public boost::range_detail::holder2 @@ -302,14 +296,12 @@ private: `` # Define an instance of the holder with the name of the adaptor - `` static boost::range_detail::forwarder2 replaced = boost::range_detail::forwarder2(); `` # Define `operator|` - `` template inline replace_range