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/