From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46946) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WSoA1-0000Dh-T6 for qemu-devel@nongnu.org; Wed, 26 Mar 2014 09:46:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WSo9x-0004QY-15 for qemu-devel@nongnu.org; Wed, 26 Mar 2014 09:46:29 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60951) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WSo9w-0004QO-Oq for qemu-devel@nongnu.org; Wed, 26 Mar 2014 09:46:24 -0400 From: Markus Armbruster References: <150e59818a7f8a0c5928f9f08d31c2248029ab57.1394579440.git.crobinso@redhat.com> <532DED51.6040706@suse.de> <20140324090916.543e00ea@redhat.com> <53303730.4090301@suse.de> Date: Wed, 26 Mar 2014 14:46:21 +0100 In-Reply-To: <53303730.4090301@suse.de> ("Andreas =?utf-8?Q?F=C3=A4rber=22?= =?utf-8?Q?'s?= message of "Mon, 24 Mar 2014 14:46:24 +0100") Message-ID: <8738i55az6.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 4/6] monitor: Remove unused monitor_print_filename List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?utf-8?Q?F=C3=A4rber?= Cc: Kevin Wolf , Cole Robinson , qemu-devel@nongnu.org, Luiz Capitulino Andreas F=C3=A4rber writes: > Am 24.03.2014 14:09, schrieb Luiz Capitulino: >> On Sat, 22 Mar 2014 21:06:41 +0100 >> Andreas F=C3=A4rber wrote: >>=20 >>> Am 12.03.2014 00:15, schrieb Cole Robinson: >>>> Cc: Luiz Capitulino >>>> Cc: Markus Armbruster >>>> Signed-off-by: Cole Robinson >>> >>> Given that it even had its own stub file, the commit message should >>> please explain why it is no longer needed and thus okay to drop. >>=20 >> Because it's unused? > > That's no real answer. Why did it need to be escaped before the commit > Cole pointed to and now no longer is? Valid question, but it applies to the commits that dropped uses, not to Cole's patch. The function goes back to 2006, commit fef3074 "Escape filname printout properly". This made "info block" print some but not all funny characters in filenames more legibly. Dropped in Kevin's commit fbe2e26 "hmp: Make "info block" output more readable" from last June. Could be regarded as regression. Not sure we care. A second use was introduced a couple of months later, in commit a9ce859 "info vnc command". Reason for escaping isn't obvious to me. Dropped in commit 1ff7df1 "Enhance 'info vnc' monitor output", which prints something else entirely. > My point was that someone went throught the hassle of creating a stub > file just for this functionality, so it's not some random unused > function in some file. (But I'm not particularly attached to it, just > cautioning about removing too much in this non-verbose way, making it > hard to follow for outsiders.) Sounds like you'd like a more vebose commit message. What information would you like to see?