svn commit: r1542251 - in /subversion/trunk/subversion/bindings/javahl/native/jniwrapper: jni_list.cpp jni_list.hpp

[br...@apache.org]

Author: brane
Date: Fri Nov 15 12:50:57 2013
New Revision: 1542251

URL: http://svn.apache.org/r1542251
Log:
* subversion/bindings/javahl/native/jniwrapper/jni_list.cpp
  (BaseMutableList::length): Use the correct method signature to
   get the list length ...
  ({BaseList, List, BaseMutableList, MutableList}::operator[]):
   Add a note to the docstring about thrown exceptions.

Modified:
    subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.cpp
    subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp

Modified: subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.cpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.cpp?rev=1542251&r1=1542250&r2=1542251&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.cpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.cpp Fri Nov 15 12:50:57 2013
@@ -92,7 +92,7 @@ jint BaseMutableList::length() const
 {
   if (!m_mid_size)
     m_mid_size = m_env.GetMethodID(m_class, "size", "()I");
-  return m_env.CallIntMethod(m_jthis, m_mid_add);
+  return m_env.CallIntMethod(m_jthis, m_mid_size);
 }
 
 } // namespace Java

Modified: subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp?rev=1542251&r1=1542250&r2=1542251&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp Fri Nov 15 12:50:57 2013
@@ -64,6 +64,7 @@ protected:
 
   /**
    * Returns the object reference at @a index.
+   * @throw std::out_of_range if the index value is not valid.
    */
   jobject operator[](jint index) const
     {
@@ -96,6 +97,7 @@ public:
 
   /**
    * Returns a wrapper object for the object reference at @a index.
+   * @throw std::out_of_range if the index value is not valid.
    */
   T operator[](jint index) const
     {
@@ -182,6 +184,7 @@ protected:
 
   /**
    * Returns the object reference at @a index.
+   * @note Throws a Java exception if the index value is not valid.
    */
   jobject operator[](jint index) const;
 
@@ -227,6 +230,7 @@ public:
 
   /**
    * Returns a wrapper object for the object reference at @a index.
+   * @note Throws a Java exception if the index value is not valid.
    */
   T operator[](jint index) const
     {