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

Re: ra vtable do_merge function?

From: Kevin Pilch-Bisson <kevin_at_pilch-bisson.net>
Date: 2002-05-24 18:13:00 CEST

On Fri, May 24, 2002 at 11:07:40AM -0500, cmpilato@collab.net wrote:
> Kevin Pilch-Bisson <kevin@pilch-bisson.net> writes:
> > In continuing to work on ra_pipe, I went to take a look at the do_merge
> > functions in ra_local and ra_dav, and found that there weren't any. Both r=
> > a's
> > just pass null as the vtable pointer, and no code in subversion ever calls
> > do_merge. Should prototype be removed from the vtable interface, or is the=
> > re
> > a reason to keep it?
> I *think* the prototype can go. I believe I added that when we first
> started talking about doing merge, but before sussman settled on a
> design for it. If `svn log' shows that indeed, that was my addition,
> then you have my blessing on removing it.

Consider it gone.

$ svn log -r 1021 subversion/include/svn_ra.h
rev 1021: cmpilato | Tue 22 Jan 2002 13:18:05 | 16 lines

* subversion/include/svn_ra.h

  (svn_ra_plugin_t): Make docstring style for 'get_log' and
  'check_path' consistent with others in this file. Added 'do_merge'
  vtable item.

* subversion/libsvn_ra_dav/session.c

  (dav_plugin): Added NULL placeholder for soon-to-arrive
  svn_ra_dav__do_merge function.

* subversion/libsvn_ra_local/ra_plugin.c

  (ra_local_plugin): Added NULL placeholder for soon-to-arrive
  do_merge function.

Kevin Pilch-Bisson                    http://www.pilch-bisson.net
     "Historically speaking, the presences of wheels in Unix
     has never precluded their reinvention." - Larry Wall

  • application/pgp-signature attachment: stored
Received on Fri May 24 18:15:07 2002

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.