[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [swig-py3][patch] interfacing bytes object instead of str

From: Yasuhito FUTATSUKI <futatuki_at_poem.co.jp>
Date: Fri, 28 Dec 2018 22:19:51 +0900

On 12/28/18 4:09 AM, Troy Curtis wrote:
> LGTM, committed as r1849804, but without the unnecessary gymnastics to continue supporting str() in those two cases mentioned above. I also tweaked the log message to use complete sentences (and removed references to the code that I didn't include).

Thank you for correct my broken log message.

As the second step of follow up r1849784, I've fixed #if blocks usage
in subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c because
it is obvious and it doesn't contain any functional change.

follow up r189784: fix macro usage to distinguish py2 and py3 context

* subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c
   Include "swigutil_py3c.h" file before define SVN_SWIG_BYTES_FMT to use
   IS_PY3 macro to switch py2 and py3 context
   (add_directory, add_file, apply_textdelta, change_file_prop,
    close_file, parse_fn3_set_revision_property,
    parse_fn3_set_node_property, svn_swig_py_notify_func,
    svn_swig_py_notify_func, svn_swig_py_delta_path_driver_cb_func,
    svn_swig_py_repos_authz_func, svn_swig_py_log_receiver,
    ra_callbacks_get_wc_prop, ra_callbacks_push_or_set_wc_prop,
    ra_callbacks_invalidate_wc_props, svn_swig_py_commit_callback,
    reporter_set_path, reporter_link_path,
    wc_diff_callbacks2_file_deleted, wc_diff_callbacks2_dir_added,
     Use IS_PY3 macro instead of comparison of PY_VERSION_HEX
   (add_directory, add_file, apply_textdelta, change_file_prop,
    close_file, parse_fn3_set_revision_property,
    parse_fn3_set_node_property, ra_callbacks_get_wc_prop,
    ra_callbacks_push_or_set_wc_prop, ra_callbacks_invalidate_wc_props):
     Enclose only format argument of Py_CallFunction or Py_CallMethod
     with #if IS_PY3 #else #endif block



Received on 2018-12-28 14:20:51 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.