Vladimir,
I haven't forgotten your patches or the other thread about the editor, but I also haven't had a chance to review them. My connectivity will be spotty over the next few days, so it may be later this week before I have meaningful feedback. (Hopefully others can chime in before that, though.)
-Hyrum
On May 28, 2012, at 10:34 PM, Vladimir Berezniker <vladimir_at_berezniker.com> wrote:
> Hi Hyrum,
>
> I committed JavaHL re-factoring changes in r1343452 and r1343456 thinking that
> while they are generally applicable to JavaHL code, they won't be used by any
> other JavaHL code, so they should go on the branch. But on a second thought
> they are not tied to the new RA code, it just happens to be the only user at
> the moment. Should I submit changes like these as patches against trunk to
> @dev or continue committing them to javahl-ra branch?
>
> Thank you in advance,
>
> Vladimir
>
>
> On Mon, May 28, 2012 at 11:22 PM, <commits-help_at_subversion.apache.org> wrote:
>
> Author: vmpn
> Date: Tue May 29 02:57:05 2012
> New Revision: 1343456
>
> URL: http://svn.apache.org/viewvc?rev=1343456&view=rev
> Log:
> On the javahl-ra branch:
>
> JavaHL: Support returning non const, empty rather than NULL hash as required
> by (svn_ra_get_commit_editor3) apr_hash_t *revprop_table parameter
>
> [ in subversion/bindings/javahl/native ]
>
> * RevpropTable.cpp,
> RevpropTable.h
> (hash): Removed const qualifier and added bool nullIfEmpty parameter to
> specify whether empty hash or NULL should be returned
>
> Modified:
> subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.cpp
> subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.h
>
> Modified: subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.cpp
> URL: http://svn.apache.org/viewvc/subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.cpp?rev=1343456&r1=1343455&r2=1343456&view=diff
> ==============================================================================
> --- subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.cpp (original)
> +++ subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.cpp Tue May 29 02:57:05 2012
> @@ -41,9 +41,9 @@ RevpropTable::~RevpropTable()
> JNIUtil::getEnv()->DeleteLocalRef(m_revpropTable);
> }
>
> -const apr_hash_t *RevpropTable::hash(const SVN::Pool &pool)
> +apr_hash_t *RevpropTable::hash(const SVN::Pool &pool, bool nullIfEmpty)
> {
> - if (m_revprops.size() == 0)
> + if (m_revprops.size() == 0 && nullIfEmpty)
> return NULL;
>
> apr_hash_t *revprop_table = apr_hash_make(pool.getPool());
>
> Modified: subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.h
> URL: http://svn.apache.org/viewvc/subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.h?rev=1343456&r1=1343455&r2=1343456&view=diff
> ==============================================================================
> --- subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.h (original)
> +++ subversion/branches/javahl-ra/subversion/bindings/javahl/native/RevpropTable.h Tue May 29 02:57:05 2012
> @@ -44,7 +44,7 @@ class RevpropTable
> public:
> RevpropTable(jobject jrevpropTable);
> ~RevpropTable();
> - const apr_hash_t *hash(const SVN::Pool &pool);
> + apr_hash_t *hash(const SVN::Pool &pool, bool nullIfEmpty = true);
> };
>
> #endif // REVPROPTABLE_H
>
>
>
>
>
Received on 2012-05-29 17:23:07 CEST