David James <james82@gmail.com> writes:
> Do you think it's good to include a link to a mailing list thread in a
> commit message? I've included an example below.
Sure.
By the way, you don't have to write "Patch by:" when you committed it :-).
-K
> On 7/19/05, djames@tigris.org <djames@tigris.org> wrote:
> > Author: djames
> > Date: Tue Jul 19 21:20:28 2005
> > New Revision: 15371
> >
> > Modified:
> > branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c
> >
> > Log:
> > Convert poolAttribute, assertValid, and emptyTuple from casted
> > "char *" variables into "char[]" variables so as to avoid
> > unsafe casts.
> >
> > Patch by: David James <james@cs.toronto.edu>
> > Thread: http://svn.haxx.se/dev/archive-2005-07/0796.shtml
> > Suggested By: Philip Martin <philip@codematters.co.uk>
> > (Thanks Philip! Good catch!)
> >
> > subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c:
> > (Automatic Pool Management Functions): Convert poolAttribute,
> > assertValid, and emptyTuple from (char *) variables into char[]
> > variables.
> >
> >
> >
> > Modified: branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c
> > Url: http://svn.collab.net/viewcvs/svn/branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c?rev=15371&p1=branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c&p2=branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c&r1=15370&r2=15371
> > ==============================================================================
> > --- branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c (original)
> > +++ branches/python-bindings-improvements/subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c Tue Jul 19 21:20:28 2005
> > @@ -113,9 +113,9 @@
> >
> > /* The application pool */
> > apr_pool_t *_global_pool = NULL;
> > -static char *poolAttribute = (char *) "_pool";
> > -static char *assertValid = (char *) "assert_valid";
> > -static char *emptyTuple = (char *) "()";
> > +static char poolAttribute[] = "_pool";
> > +static char assertValid[] = "assert_valid";
> > +static char emptyTuple[] = "()";
> >
> >
> > /* Set the application pool */
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: svn-unsubscribe@subversion.tigris.org
> > For additional commands, e-mail: svn-help@subversion.tigris.org
> >
> >
>
>
> --
> David James -- http://www.cs.toronto.edu/~james
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jul 20 23:06:34 2005