From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19203 invoked by alias); 6 Nov 2013 21:14:38 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 19188 invoked by uid 89); 6 Nov 2013 21:14:37 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.0 required=5.0 tests=AWL,BAYES_50,RDNS_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 X-HELO: mail-vc0-f171.google.com Received: from Unknown (HELO mail-vc0-f171.google.com) (209.85.220.171) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Wed, 06 Nov 2013 21:14:35 +0000 Received: by mail-vc0-f171.google.com with SMTP id ik5so56121vcb.2 for ; Wed, 06 Nov 2013 13:14:27 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=YOW9tkdOHtq0chw47aAomBECMERmeZCeAKtMYNM0bW4=; b=AQvQu20knnL4b/0xTO1UtFovHT/EedpH6U6nWP69I3GIy7nK1uStVVIqbzlhR5kS+y iLOkDswjLqXghypZsW2PvBNawbrJYoyIf62+p/NOpXLGQ8IdOGDrP0+hTAHpea6shmW1 mhlQZzLwITVWUjBcAY8MRUCkePMrMBcrf9UVauC4AV+d+CT8TdJ0f/4kSVdGnmVID8tP bLjCL5hlHQ9G/1UIZn7/D+V2Ih/NrH7NOoJt5YOMtWFnWMu0iqClgI6fAVIKUFFO/NQX 4zpIoO1AAI0kSvWDf+PZT3qWNCQzPDat+/Gw5JnjiGX/6wgKKDmmLldnuStORrXYDba4 qn0w== X-Gm-Message-State: ALoCoQkMCxeIl1/fwvsdHMbhWnI3VTlFF0/uqbNVDKJCxJQ4IRJlf+AqF7g7aikFLNjIbDWuy6NwbxjgTTkhKcIZIdfjLAWtzeTmVarFM5i1961YnbEoLI9BJ8NuGY+VSe7xiLyRi5fAEsGYc9Ups5WAm/h+Yn98UCxYcDPMAKuQx6iKNceWh0tN2bfmUeKvrIW3g3xg8ZjRhQoy1mqV2xB1uIpmG0QmSA== MIME-Version: 1.0 X-Received: by 10.220.253.66 with SMTP id mz2mr4116357vcb.10.1383772467357; Wed, 06 Nov 2013 13:14:27 -0800 (PST) Received: by 10.52.237.232 with HTTP; Wed, 6 Nov 2013 13:14:27 -0800 (PST) In-Reply-To: <20131105180547.GA24004@host2.jankratochvil.net> References: <20131031154957.GA11260@host2.jankratochvil.net> <87li13shk2.fsf@fleche.redhat.com> <20131105172219.GA21529@host2.jankratochvil.net> <20131105180547.GA24004@host2.jankratochvil.net> Date: Wed, 06 Nov 2013 21:16:00 -0000 Message-ID: Subject: Re: [PATCH] Fix Gold/strip discrepancies for PR 11786 From: Doug Evans To: Jan Kratochvil Cc: Tom Tromey , gdb-patches , Pedro Alves Content-Type: text/plain; charset=ISO-8859-1 X-IsSubscribed: yes X-SW-Source: 2013-11/txt/msg00148.txt.bz2 On Tue, Nov 5, 2013 at 10:05 AM, Jan Kratochvil wrote: > On Tue, 05 Nov 2013 18:56:29 +0100, Doug Evans wrote: >> If the decision is to be more strict with the rules for testcases >> that's fine by me. >> Let's write it down, then discussions like these will become a *lot* shorter. > > Adding more and more rules I do not find as a clear win. > When I code GDB I have to think about so many established non-standard coding > style rules my head is going to explode. Switching between multiple projects > each having different coding style makes it worse. > > But sending a patch and getting it corrected here and there due to unwritten > rules one could not find anywhere is also not great, though, I agree. At the end of the day I'm still lacking the clarity I seek. [It's not imperative, but it's more than "IWBN".] I'm not suggesting adding more rules (per se), but I do think there's no downside to writing down existing unwritten rules (for those things that are, indeed, rules). I'm going to propose the following, and if y'all are ok with it then this can be the end of it, this thread is done. If there are no objections, I will add a Testsuite section to the C-Coding-Standards section of the wiki: https://sourceware.org/gdb/wiki/Internals%20GDB-C-Coding-Standards It will basically say tests are in general not required to following the GDB coding standards, but there are a few exceptions, and then have an enumeration of explicit rules, and for now just have the one: (void) is required over (). And for grin's sake, since there's so many of them, and less likely to be a problem, int main () is ok. I'll also mention that "Monkey See Monkey Do hacking should generally Just Work." [There's less need to go into detail if one can say one can just mimic existing code. That will keep it short-and-sweet. I'm ok with people adding more to the wiki, but I like taking "baby steps".] I'll probably also add a link to the new section to: https://sourceware.org/gdb/wiki/GDBTestcaseCookbook