RE: svn commit: r1351717 - in /subversion/trunk/subversion/libsvn_ra_serf: ra_serf.h serf.c

> -----Original Message-----
> From: rhuijben_at_apache.org [mailto:rhuijben_at_apache.org]
> Sent: dinsdag 19 juni 2012 15:55
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1351717 - in
> /subversion/trunk/subversion/libsvn_ra_serf: ra_serf.h serf.c
>
> Author: rhuijben
> Date: Tue Jun 19 13:54:45 2012
> New Revision: 1351717
>
> URL: http://svn.apache.org/viewvc?rev=1351717&view=rev
> Log:
> * subversion/libsvn_ra_serf/ra_serf.h
> (svn_ra_serf__session_t): Make timeout type match the argument type of
> serf_context_run, its only user.
>
> * subversion/libsvn_ra_serf/serf.c
> (load_config): Verify that we don't set timeout to a negative value.

[Not sure when this message gets through. My ADSL is still broken]

Unless the user configured a different timeout value in his/her configuration we used to set timeout to a negative value on any platform with 32 bit longs. This patch should make the behavior the same on all platforms.

This timeout is probably still too long, but setting it negative made it infinite.

The code should be fixed to use a proper default.
Waiting for 10 minutes until checking for ^C via the cancel function is still broken behavior for GUI clients like AnkhSVN.

I see no problem in waiting for more than 10 minutes, but it should be user cancelable within at least something like 30 seconds, preferably sooner or the user will just kill his application.
(Which in case of AnkhSVN will include losing work in open editors)

        Bert
Received on 2012-06-19 16:22:52 CEST