add inline DumpError() to avoid subtle API break

apt-pkg/contrib/error.h

@@ -217,10 +217,22 @@ public:									/*{{{*/
 	 *
 	 *  \param threshold minimum level printed
 	 */
-	void inline DumpErrors(MsgType const &threshold = WARNING) {
+	void inline DumpErrors(MsgType const &threshold) {
 		DumpErrors(std::cerr, threshold);
 	}
 
+        // mvo: we do this instead of using a default parameter in the
+        //      previous declaration to avoid a (subtle) API break for
+        //      e.g. sigc++ and mem_fun0
+	/** \brief dumps the messages of type WARNING or higher to std::cerr
+	 *
+	 *  Note that all messages are discarded, displayed or not.
+	 *
+	 */
+	void inline DumpErrors() {
+                DumpErrors(WARNING);
+	}
+
 	/** \brief put the current Messages into the stack
 	 *
 	 *  All "old" messages will be pushed into a stack to

debian/changelog

@@ -35,6 +35,7 @@ apt (0.7.26~exp11) experimental; urgency=low
   [ Michael Vogt ]
   * apt-pkg/contrib/error.{cc,h}
     - docstring cleanup
+    - add inline DumpError() to avoid subtle API break
 
  -- David Kalnischkies <kalnischkies@gmail.com>  Mon, 26 Jul 2010 12:40:44 +0200