On Wed, Sep 7, 2011 at 6:16 AM, Bert Huijben <b...@qqmail.nl> wrote:
>
>
>> -----Original Message-----
>> From: danie...@apache.org [mailto:danie...@apache.org]
>> Sent: woensdag 7 september 2011 12:20
>> To: comm...@subversion.apache.org
>> Subject: svn commit: r1166096 - in /subversion/trunk/subversion:
>> include/svn_repos.h svnadmin/main.c
>>
>> Author: danielsh
>> Date: Wed Sep  7 10:19:32 2011
>> New Revision: 1166096
>>
>> URL: http://svn.apache.org/viewvc?rev=1166096&view=rev
>> Log:
>> Teach the svn_repos_notify_* interface to say "I won't repeat this warning
>> in the future".
>>
>> * subversion/include/svn_repos.h
>>   (svn_repos_notify_t): New member LAST_WARNING.
>>
>> * subversion/svnadmin/main.c
>>   (repos_notify_handler): Consume new member.
>
> Shouldn't the user interface (in this case svnadmin) be responsible for the 
> suppressing of duplicate messages?
>
> Maybe a different client would want a different way of suppressing the 
> messages.
>
> I can't think of a specific use case here,  but maybe something like counting 
> the number of occurrences like we do with conflicts and skips in svn update.

+1

There is little harm in the libraries spitting out verbose warnings
unconditionally.  Consumers should be able to filter them as desired.

-Hyrum



-- 

uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/

Reply via email to