[PATCH] Various svnbook fixes
Øyvind A. Holm
sunny at sunbase.org
Wed Aug 17 18:09:41 CDT 2005
On 2005-08-17 23:46:36 Max Bowsher wrote:
> Øyvind A. Holm wrote:
> > On 2005-08-17 08:36:52 C. Michael Pilato wrote:
> > > If not commented on below, I agree with Max's opinions on the
> > > patch.
> > > [...]
> > Thank you for the thorough review of the patch, cmpilato and maxb.
> > And mthelen for digging it up, good catch. :)
> > Now that I know what the general view is on the different changes, I
> > can clean up the patch, possibly splitting it up to keep related
> > changes together. There are also some new additions to it which I
> > also can include. The new patch (with log messages) arrives in a
> > little while.
> Attached is the cleaned patch, containing the results of the reviews,
> plus re-wrapping of lines made overly long by the changes.
> I didn't commit it yet because I haven't yet got around to writing a
> log message. If you'd like to write the log and then commit it, please
Ah, thanks for cleaning it up. I was sitting here splitting and
tweaking, but now I can get these changes out of the way first. I can
take care of the log message and perform the commit. There are some more
changes along the same scheme which I’ve found, some quite identical
(periods outside parentheses and such). Shall I create a new patch for
those, or is it OK to include them in the same commit?
More information about the svnbook-dev