On Fri, 2006-06-23 at 10:55 -0400, Garrett Rooney wrote:
> On 6/23/06, Jelmer Vernooij <jelmer@samba.org> wrote:
> > Hi,
> >
> > [[[
> > Fix Python bindings to svn.wc.crawl_revisions2().
> >
> > [in subversion/bindings/swig]
> >
> > * svn_ra.i: Add Python typemap for svn_ra_reporter2_t.
> >
> > * svn_wc.i: Add Python typemap for svn_wc_notify_func2_t.
> >
> > * tests/wc.py
> > (setUp): Open working copy subdirectories.
> > (test_crawl_revisions2): Add test for svn.wc.crawl_revisions2() and
> > svn_ra_reporter2_t implementations.
> >
> > * python/libsvn_swig_py/swigutil_py.c
> > (make_ob_wc_notify): Add function.
> > (svn_swig_py_notify_func2): Add Python wrapper for
> > svn_wc_notify_func2_t.
> > (reporter_set_path): Add function.
> > (reporter_delete_path): Add function.
> > (reporter_link_path): Add function.
> > (reporter_abort_report): Add function.
> > (reporter_finish_report): Add function.
> > (swig_py_ra_reporter2): Add Python wrapper for svn_ra_reporter2_t.
> >
> > * python/libsvn_swig_py/swigutil_py.h
> > (swig_py_ra_reporter2): Add extern.
> > (svn_swig_py_notify_func2): Add prototype.
> > ]]]
>
> I'm totally unqualified to evaluate this patch, but I do notice that
> it's full of tabs, and appears to have been written assuming that tab
> == 4 spaces, which totally screws up the indenting in 'svn diff'
> output, that'll have to be fixed before it can be applied.
Oops - Fixed in the attached version.
Cheers,
Jelmer
--
Jelmer Vernooij <jelmer_at_samba.org> - http://samba.org/~jelmer/
Received on Fri Jun 23 16:59:28 2006