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

Re: CVS update: subversion/subversion/tests/libsvn_wc commit-test.c

From: Karl Fogel <kfogel_at_galois.collab.net>
Date: 2001-03-13 01:33:46 CET

Greg Stein <gstein@lyra.org> writes:
> > (apply_textdelta, window_handler): return a no-op handler for
> > proper delta composition.
> >...
> > static svn_error_t *
> > +window_handler (svn_txdelta_window_t *window, void *handler_pair)
> > +{
> > + /* No-op, but required for proper editor composition. */
> > + return SVN_NO_ERROR;
> > +}
>
> Doesn't this imply that the default editor is broken? Shouldn't we be able
> to compose as many default editors as we'd like, without worry?

I think it was this situation:

The default editor's apply_textdelta does return a window_handler.
The problem was that Ben has implemented his own apply_textdelta, so
he's not getting the default -- therefore, he has to supply a
window_handler, even if he doesn't plan to do anything with it.

If he weren't implementing apply_textdelta at all, then he wouldn't
have to supply the window_handler either.

-K
Received on Sat Oct 21 14:36:25 2006

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