Remove a result_pool argument from a new in 1.8 function that doesn't have
output arguments. Values passed to a callback shouldn't be allocated in a
global result pool, but in some iterpool. Storing everything in a result
hash breaks the streamy api pattern.

* subversion/include/svn_client.h
  (svn_client_proplist4): Remove result_pool.

* subversion/libsvn_client/deprecated.c
  (svn_client_proplist3): Update caller.

* subversion/libsvn_client/prop_commands.c
  (pristine_or_working_props): Remove function. Fold in only caller.
  (remote_proplist): Add cancel_func. Remove result pool. Don't duplicate the
    entire hash to remove 3 or 4 entry props. Handle cancelation.
    Rename subpool to iterpool.
  (get_remote_props): Remove result pool. Update caller.
  (get_local_props): Fold in pristine_or_working_props.
  (svn_client_proplist4): Remove argument. Fix typo.

* subversion/svn/proplist-cmd.c
  (svn_cl__proplist): Update caller.


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