On Wed, Jun 18, 2008 at 07:50:04PM +0300, Daniel Shahaf wrote:
> Daniel L. Rall wrote on Wed, 18 Jun 2008 at 08:24 -0700:
> > On Wed, 18 Jun 2008, firemeteor_at_tigris.org wrote:
> >
> > > Author: firemeteor
> > > Revision: 31763
> > > Property Name: svn:log
> > > Action: modified
> > >
> > > Property diff:
> > > --- old property value
> > > +++ new property value
> > > @@ -1,6 +1,8 @@
> > > -A small readability tweak suggested and approved by kfogel (Karl Fogel).
> > > -I mistakenly did it in my branch. Now, I revert that change and do it in trunk
> > > -first.
> > > +Approved by: kfogel
> > > +
> > > +A small readability tweak suggested and approved by Karl Fogel. I mistakenly
> > > +did it in my branch (issue-2843-dev, r31741). Now, I revert that change and do
> > > +it in trunk first.
> > >
> > > * subversion/libsvn_wc/adm_ops.c
> > > (svn_wc_remove_from_revision_control): Improve readability by introduce
> >
> > Mind listing the tags like "Approved by" last? When looking through the
> > history, they're the last things I typically care about.
> >
> > Thanks, Dan
> >
>
> Already reordered it (before I saw your mail).
>
> Daniel
Sorry for the inconvenience. I subscribed the dev list only and thus didn't
aware that the approved line should come at last. I won't let this happen
again. Thank you very much.
Rui
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-19 07:30:47 CEST