Re: [viewvc-dev] Re: [patch] supporting cvsnt kopt

Michael Haggerty wrote:
> I thought several times about making changes like this to the rcsparser
> code, but I'm not sure how serious it would be to break backwards
> compatibility in the Sink interface.

[Just want to comment on this bit of your mail, Michael.]

My opinion:  break it.

ViewVC and cvs2svn are the two foremost consumers of rcsparse, and you're a
committer for both of them.  rcsparse has not yet "grown up" into a project
of its own, and I don't predict that it ever will.  Let's not allow that to
hold back progress where it is most needed.  I would imagine that updating
existing code to expect another parameter on those functions would be
super-trivial effort, right?

-- 
C. Michael Pilato <[email protected]>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

------------------------------------------------------
http://viewvc.tigris.org/ds/viewMessage.do?dsForumId=4251&dsMessageId=2057574

To unsubscribe from this discussion, e-mail: [[email protected]].

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkn/DKYACgkQokEGqRcG/W5cnACdHKaWVtjJaaWh6wJQ1ER24Ynb
Fj8An3dPj8ypmZReKEkyXi6YsdPM5jTL
=rAvD
-----END PGP SIGNATURE-----