[Boost-commit] svn:boost r72602 - in trunk: boost/python libs/python/doc/v2 libs/python/test

Subject: [Boost-commit] svn:boost r72602 - in trunk: boost/python libs/python/doc/v2 libs/python/test
From: rwgk_at_[hidden]
Date: 2011-06-14 19:23:34

Author: rwgk
Date: 2011-06-14 19:23:33 EDT (Tue, 14 Jun 2011)
New Revision: 72602
URL: http://svn.boost.org/trac/boost/changeset/72602

Log:
boost/python/slice.hpp: correct long-standing spelling error; affects interface; keeping old interface for backward compatibility
Text files modified: 
   trunk/boost/python/slice.hpp        |    16 +++++++++++++---                        
   trunk/libs/python/doc/v2/slice.html |    14 +++++++-------                          
   trunk/libs/python/test/slice.cpp    |     8 ++++----                                
   trunk/libs/python/test/slice.py     |    12 ++++++------                            
   4 files changed, 30 insertions(+), 20 deletions(-)

Modified: trunk/boost/python/slice.hpp
==============================================================================
--- trunk/boost/python/slice.hpp	(original)
+++ trunk/boost/python/slice.hpp	2011-06-14 19:23:33 EDT (Tue, 14 Jun 2011)
@@ -30,7 +30,7 @@
       // that created this slice, than that parameter is None here, and compares 
       // equal to a default-constructed boost::python::object.
       // If a user-defined type wishes to support slicing, then support for the 
-      // special meaning associated with negative indicies is up to the user.
+      // special meaning associated with negative indices is up to the user.
       object start() const;
       object stop() const;
       object step() const;
@@ -63,7 +63,7 @@
         
     // The following algorithm is intended to automate the process of 
     // determining a slice range when you want to fully support negative
-    // indicies and non-singular step sizes.  Its functionallity is simmilar to 
+    // indices and non-singular step sizes.  Its functionallity is simmilar to 
     // PySlice_GetIndicesEx() in the Python/C API, but tailored for C++ users.
     // This template returns a slice::range struct that, when used in the 
     // following iterative loop, will traverse a slice of the function's
@@ -110,7 +110,7 @@
     
     template<typename RandomAccessIterator>
     slice::range<RandomAccessIterator>
-    get_indicies( const RandomAccessIterator& begin, 
+    get_indices( const RandomAccessIterator& begin, 
         const RandomAccessIterator& end) const
     {
         // This is based loosely on PySlice_GetIndicesEx(), but it has been 
@@ -240,6 +240,16 @@
         
         return ret;
     }
+
+    // Incorrect spelling. DO NOT USE. Only here for backward compatibility.
+    // Corrected 2011-06-14.
+    template<typename RandomAccessIterator>
+    slice::range<RandomAccessIterator>
+    get_indicies( const RandomAccessIterator& begin, 
+        const RandomAccessIterator& end) const
+    {
+        get_indices(begin, end);
+    }
         
  public:
     // This declaration, in conjunction with the specialization of 

Modified: trunk/libs/python/doc/v2/slice.html
==============================================================================
--- trunk/libs/python/doc/v2/slice.html	(original)
+++ trunk/libs/python/doc/v2/slice.html	2011-06-14 19:23:33 EDT (Tue, 14 Jun 2011)
@@ -85,7 +85,7 @@
       object stop();
       object step();
 
-      // The return type of slice::get_indicies()
+      // The return type of slice::get_indices()
       template &lt;typename RandomAccessIterator&gt;
       struct range
       {
@@ -96,7 +96,7 @@
 
       template &lt;typename RandomAccessIterator&gt;
       range&lt;RandomAccessIterator&gt;
-      get_indicies(
+      get_indices(
           RandomAccessIterator const&amp; begin, 
           RandomAccessIterator const&amp; end);
   };
@@ -164,7 +164,7 @@
 <pre>
 template &lt;typename RandomAccessIterator&gt;
 slice::range&lt;RandomAccessIterator&gt;
-slice::get_indicies( 
+slice::get_indices( 
     RandomAccessIterator const&amp; begin, 
     RandomAccessIterator const&amp; end) const;
 </pre>
@@ -173,8 +173,8 @@
 Iterators that form a half-open range.</dt>
   <dt><b>Effects:</b> Create a RandomAccessIterator pair that defines a
 fully-closed range within the [begin,end) range of its arguments.&nbsp;
-This function translates this slice's indicies while accounting for the
-effects of any PyNone or negative indicies, and non-singular step sizes.</dt>
+This function translates this slice's indices while accounting for the
+effects of any PyNone or negative indices, and non-singular step sizes.</dt>
   <dt><b>Returns:</b> a slice::range
 that has been initialized with a non-zero value of step and a pair of
 RandomAccessIterators that point within the range of this functions
@@ -182,7 +182,7 @@
   <dt><b>Throws:</b> Raises a Python <code>TypeError</code> exception if any of this slice's arguments
 are neither references to <code>PyNone</code> nor convertible to <code>int</code>.&nbsp; Throws
 <code>std::invalid_argument</code> if the resulting range would be empty.&nbsp; You
-should always wrap calls to <code>slice::get_indicies()</code>
+should always wrap calls to <code>slice::get_indices()</code>
 within <code>try { ...; } catch (std::invalid_argument) {}</code> to
 handle this case and take appropriate action.</dt>
   <dt><b>Rationale</b>: closed-interval: If
@@ -221,7 +221,7 @@
 {
     slice::range&lt;std::vector&lt;double&gt;::const_iterator&gt; bounds;
     try {
-        bounds = index.get_indicies&lt;&gt;(Foo.begin(), Foo.end());
+        bounds = index.get_indices&lt;&gt;(Foo.begin(), Foo.end());
     }
     catch (std::invalid_argument) {
         return 0.0;

Modified: trunk/libs/python/test/slice.cpp
==============================================================================
--- trunk/libs/python/test/slice.cpp	(original)
+++ trunk/libs/python/test/slice.cpp	2011-06-14 19:23:33 EDT (Tue, 14 Jun 2011)
@@ -98,9 +98,9 @@
 
 #if BOOST_WORKAROUND( BOOST_MSVC, BOOST_TESTED_AT(1400)) \
     || BOOST_WORKAROUND( BOOST_INTEL_WIN, == 710)
-int check_slice_get_indicies(slice index);
+int check_slice_get_indices(slice index);
 #endif
-int check_slice_get_indicies(
+int check_slice_get_indices(
 #if !BOOST_WORKAROUND(__SUNPRO_CC, BOOST_TESTED_AT(0x590))
     const
 #endif 
@@ -116,7 +116,7 @@
     
     slice::range<std::vector<int>::iterator> bounds;
     try {
-        bounds = index.get_indicies(coll.begin(), coll.end());
+        bounds = index.get_indices(coll.begin(), coll.end());
     }
     catch (std::invalid_argument) {
         return 0;
@@ -136,5 +136,5 @@
     def( "accept_slice", accept_slice);
     def( "check_numeric_array_rich_slice", check_numeric_array_rich_slice);
     def( "check_string_rich_slice", check_string_rich_slice);
-    def( "check_slice_get_indicies", check_slice_get_indicies);
+    def( "check_slice_get_indices", check_slice_get_indices);
 }

Modified: trunk/libs/python/test/slice.py
==============================================================================
--- trunk/libs/python/test/slice.py	(original)
+++ trunk/libs/python/test/slice.py	2011-06-14 19:23:33 EDT (Tue, 14 Jun 2011)
@@ -37,17 +37,17 @@
 ...     print 1
 ...
 1
->>> check_slice_get_indicies( slice(None))
+>>> check_slice_get_indices( slice(None))
 0
->>> check_slice_get_indicies( slice(2,-2))
+>>> check_slice_get_indices( slice(2,-2))
 0
->>> check_slice_get_indicies( slice(2, None, 2))
+>>> check_slice_get_indices( slice(2, None, 2))
 5
->>> check_slice_get_indicies( slice(2, None, -1))
+>>> check_slice_get_indices( slice(2, None, -1))
 -12
->>> check_slice_get_indicies( slice( 20, None))
+>>> check_slice_get_indices( slice( 20, None))
 0
->>> check_slice_get_indicies( slice( -2, -5, -2))
+>>> check_slice_get_indices( slice( -2, -5, -2))
 6
 """