subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bert Huijben" <b...@qqmail.nl>
Subject RE: svn commit: r1351717 - in /subversion/trunk/subversion/libsvn_ra_serf: ra_serf.h serf.c
Date Tue, 19 Jun 2012 14:21:29 GMT


> -----Original Message-----
> From: rhuijben@apache.org [mailto:rhuijben@apache.org]
> Sent: dinsdag 19 juni 2012 15:55
> To: commits@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



Mime
View raw message