From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9159 invoked by alias); 26 Jan 2004 11:55:12 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 9149 invoked from network); 26 Jan 2004 11:55:11 -0000 Received: from unknown (HELO walton.kettenis.dyndns.org) (213.93.115.144) by sources.redhat.com with SMTP; 26 Jan 2004 11:55:11 -0000 Received: from elgar.kettenis.dyndns.org (elgar.kettenis.dyndns.org [192.168.0.2]) by walton.kettenis.dyndns.org (8.12.6p3/8.12.6) with ESMTP id i0QBt7IY001963; Mon, 26 Jan 2004 12:55:07 +0100 (CET) (envelope-from kettenis@elgar.kettenis.dyndns.org) Received: from elgar.kettenis.dyndns.org (localhost [127.0.0.1]) by elgar.kettenis.dyndns.org (8.12.6p3/8.12.6) with ESMTP id i0QBt6ZM026466; Mon, 26 Jan 2004 12:55:06 +0100 (CET) (envelope-from kettenis@elgar.kettenis.dyndns.org) Received: (from kettenis@localhost) by elgar.kettenis.dyndns.org (8.12.6p3/8.12.6/Submit) id i0QBt5uJ026463; Mon, 26 Jan 2004 12:55:05 +0100 (CET) Date: Mon, 26 Jan 2004 11:55:00 -0000 Message-Id: <200401261155.i0QBt5uJ026463@elgar.kettenis.dyndns.org> From: Mark Kettenis To: eliz@elta.co.il CC: gdb-patches@sources.redhat.com In-reply-to: (message from Eli Zaretskii on 26 Jan 2004 08:19:17 +0200) Subject: Re: [PATCH] Partial infcmd.c cleanup References: <200401251732.i0PHWmRq007000@elgar.kettenis.dyndns.org> X-SW-Source: 2004-01/txt/msg00668.txt.bz2 Date: 26 Jan 2004 08:19:17 +0200 From: Eli Zaretskii > Date: Sun, 25 Jan 2004 18:32:48 +0100 (CET) > From: Mark Kettenis > > Since I'll be changing this bit of code I though I'd give it a cleanup > first. Mostly wrapping long lines and adding some missing whitespace. > > Committed as obvious, I think changing finish_command_continuation from a void function into a static void function does not really qualify as obvious. I'm talking about this part of the patch: But I didn't. It already was static, since the earlier on in the file we have the prototype: static void finish_command_continuation (struct continuation_arg *); I should have mentioned it in the ChangeLog. Mark