On Wed, Oct 06, 2010 at 04:27:44PM +0200, Maurizio Boriani wrote:
> I've successfully parsed a cvs repo of about 196000 files and obtained the
> same (good) result with old opencvs, patched opencvs and gnu cvs

Thank you for testing, good feedback is even better than bugreports. :)


I have improved rcsparse.c in some more areas:

- comment @@; and expand @@; are optional, set it to OPTIONAL.
- added dummy support for commitid ...;, reported by Joerg Sonnenberger for
  OpenCVS and it was in for OpenRCS already.
- rewrote switch() into more readable if() statement.
- plugged a memory leak
- adjusted some comments

And best part:

- generalized rcsparse enough to get it into OpenRCS and OpenCVS with only
  two lines diff (cvs_vlog instead of vwarnx and its include)!


Our trees (ports, src, www, xenocara) work fine as of today, 10:00 o'clock.

I don't want to put these two diffs (57K/~2500 lines each) in this mail,
please download here:

http://www.stoeckmann.org/opencvs-rcsparse.diff
http://www.stoeckmann.org/openrcs-rcsparse.diff


You can also download an OpenCVS export of mine. I have used it to identify
any possible differences between current and new parser.  It parses a
supplied RCS file fully with current and new parser and compares the
data structures in memory.  If you see ANY output, PLEASE send me the RCS
file!  Usage: opencvs /path/to/RCS/file,v

http://www.stoeckmann.org/cvs.tar.gz


With kind regards,

Tobias Stoeckmann

Reply via email to