Fixed a silly thinko in the design of the JavahL PropertyTable wrapper:
whether it generates a NULL hash table should depend on the value
of the original Java property set, and the semantics should be defined
when the wrapper is created, not when it's converted to a hash table.

* subversion/bindings/javahl/native/PropertyTable.h
  (PropertyTable::m_empty_if_null): New private member.
  (PropertyTable::PropertyTable): Remove default value from
   the bytearray_values parameter and add empty_if_null parameter.
  (PropertyTable::hash): Remove nullIfEmpty parameter.
* subversion/bindings/javahl/native/PropertyTable.cpp
  (PropertyTable::hash): Update signature and change the logic
   that decides whether we should return a NULL hash value.
  (PropertyTable::PropertyTable): Update signature and
   initialize the new private member.

* subversion/bindings/javahl/native/CommitEditor.cpp,
  subversion/bindings/javahl/native/org_apache_subversion_javahl_SVNClient.cpp:
   Update all uses of PropertyTable.


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1537549 13f79535-47bb-0310-9956-ffa450edef68