[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r36156 - trunk

From: Greg Stein <gstein_at_gmail.com>
Date: Thu, 26 Feb 2009 15:56:43 +0100

Um... that was something like 3 or 4 changes to that ONE line over a
series of a few commits. Are you not testing your changes at all
before you commit? Or doing a pre-commit diff inspection?

It worries me when these kinds of trivial/obvious errors get into the
code as an apparent result of no testing. What will be next? What is
going to get *missed*?

-g

On Thu, Feb 26, 2009 at 14:16, Bert Huijben <rhuijben_at_sharpsvn.net> wrote:
> Author: rhuijben
> Date: Thu Feb 26 05:16:48 2009
> New Revision: 36156
>
> Log:
> * win-tests.py
>  (Httpd._start_daemon): Remove an invalid ')' introduced yesterday.
>
> Modified:
>   trunk/win-tests.py
>
> Modified: trunk/win-tests.py
> URL: http://svn.collab.net/viewvc/svn/trunk/win-tests.py?pathrev=36156&r1=36155&r2=36156
> ==============================================================================
> --- trunk/win-tests.py  Thu Feb 26 05:01:32 2009        (r36155)
> +++ trunk/win-tests.py  Thu Feb 26 05:16:48 2009        (r36156)
> @@ -524,7 +524,7 @@ class Httpd:
>     try:
>       import win32process
>       import win32con
> -      args = ' '.join([self._quote(x) for x in self.httpd_args)])
> +      args = ' '.join([self._quote(x) for x in self.httpd_args])
>       self.proc_handle = (
>         win32process.CreateProcess(self._quote(self.path), args,
>                                    None, None, 0,
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=495&dsMessageId=1232916
>

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1233320
Received on 2009-02-26 15:57:03 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.