Re: Transactionless report processing

Greg Hudson <ghudson@MIT.EDU> writes:
> So, fetch those patches and apply them first, then sub in reporter.c.

It's sweet. The code is exceptionally clear & well-commented.

The only unfulfilled desire I have is for a prose description of the
tempfile format. Of course, the format string in write_path_info()
helps, but one has to decode it rather carefully -- a comment
describing how the tempfile is laid out would be great.

In fact, a comment near the top that describes the overall method
would be great. Heck, just take the text from the issue and put it in
an introductory comment at the top of reporter.c:

   /* The report is written out to a temporary file and then read back
      in and processed in a streamy manner. The tempfile's format is:

      blah blah blah. */

Uh, I take it you won't mind that I filed this one in Post-1.0? :-)

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Dec 30 18:08:43 2003