From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48534) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dTMpi-0004nF-5Q for qemu-devel@nongnu.org; Fri, 07 Jul 2017 02:33:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dTMpe-0003xH-7l for qemu-devel@nongnu.org; Fri, 07 Jul 2017 02:33:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44494) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dTMpd-0003x0-VW for qemu-devel@nongnu.org; Fri, 07 Jul 2017 02:33:38 -0400 References: <0a1d5638543965d532284bdc6fce391cf9f509d0.1499381754.git.alistair.francis@xilinx.com> From: Thomas Huth Message-ID: Date: Fri, 7 Jul 2017 08:33:19 +0200 MIME-Version: 1.0 In-Reply-To: <0a1d5638543965d532284bdc6fce391cf9f509d0.1499381754.git.alistair.francis@xilinx.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alistair Francis , qemu-devel@nongnu.org Cc: Peter Maydell , Cornelia Huck , Stefan Hajnoczi , "Michael S. Tsirkin" , Jeff Cody , Alexander Graf , Gerd Hoffmann , Eduardo Habkost , Rob Herring , Josh Durgin , armbru@redhat.com, Christian Borntraeger , Marcel Apfelbaum , David Gibson , Jason Wang , philippe@mathieu-daude.net, Peter Lieven , Greg Kurz , Peter Chubb , Ronnie Sahlberg , Igor Mammedov , alistair23@gmail.com, Richard Henderson , Kevin Wolf , Peter Crosthwaite , Marcelo Tosatti , "Richard W.M. Jones" , Max Reitz , "Aneesh Kumar K.V" , Paolo Bonzini On 07.07.2017 01:49, Alistair Francis wrote: > Convert all uses of error_report("[Ww]arning:"... to use warn_report() > instead. This helps standardise on a single method of printing warnings > to the user. > > All of the warnings were found using this regex expression: > error_report.*[Ww]arning: > and replaced with: > warn_report(" [...] > diff --git a/tests/test-qdev-global-props.c b/tests/test-qdev-global-props.c > index 48e5b7315f..b25fe892ed 100644 > --- a/tests/test-qdev-global-props.c > +++ b/tests/test-qdev-global-props.c > @@ -232,10 +232,10 @@ static void test_dynamic_globalprop(void) > g_test_trap_assert_passed(); > g_test_trap_assert_stderr_unmatched("*prop1*"); > g_test_trap_assert_stderr_unmatched("*prop2*"); > - g_test_trap_assert_stderr("*Warning: global dynamic-prop-type-bad.prop3 has invalid class name\n*"); > + g_test_trap_assert_stderr("*warning: global dynamic-prop-type-bad.prop3 has invalid class name\n*"); > g_test_trap_assert_stderr_unmatched("*prop4*"); > - g_test_trap_assert_stderr("*Warning: global nohotplug-type.prop5=105 not used\n*"); > - g_test_trap_assert_stderr("*Warning: global nondevice-type.prop6 has invalid class name\n*"); > + g_test_trap_assert_stderr("*warning: global nohotplug-type.prop5=105 not used\n*"); > + g_test_trap_assert_stderr("*warning: global nondevice-type.prop6 has invalid class name\n*"); > g_test_trap_assert_stdout(""); > } These changes are unrelated ... please drop them from your patch. Thomas