add inline DumpError() to avoid subtle API break

1 files changed, 13 insertions, 1 deletions

diff --git a/apt-pkg/contrib/error.h b/apt-pkg/contrib/error.h
index d39500427..4af0302c0 100644
--- a/apt-pkg/contrib/error.h
+++ b/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