From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 106619 invoked by alias); 6 Oct 2015 14:04:36 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 106609 invoked by uid 89); 6 Oct 2015 14:04:35 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 06 Oct 2015 14:04:34 +0000 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (Postfix) with ESMTPS id 610402F9129 for ; Tue, 6 Oct 2015 14:04:33 +0000 (UTC) Received: from [10.10.50.40] (unused [10.10.50.40] (may be forged)) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t96E4WO9009883; Tue, 6 Oct 2015 10:04:32 -0400 Subject: Re: [patch 4/3] Header file reduction - Tools for contrib To: gcc-patches@gcc.gnu.org, Bernd Schmidt References: <560DEA79.8050709@redhat.com> <56127AA4.9090707@redhat.com> <5612E939.9000701@redhat.com> <5613A1F9.3030407@codesourcery.com> <5613B846.2090107@redhat.com> From: Andrew MacLeod Message-ID: <5613D4F0.7070204@redhat.com> Date: Tue, 06 Oct 2015 14:04:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <5613B846.2090107@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2015-10/txt/msg00572.txt.bz2 On 10/06/2015 08:02 AM, Bernd Schmidt wrote: > >>> There are 9 tools I used over the run of the project. They were >>> developed in various stages and iterations, but I tried to at least >>> have >>> some common interface things, and I tried some cleaning up and >>> documentation. > > I'll probably have to make multiple passes over this. A disclaimer > first, I have done enough Python programming to develop a dislike for > the language, but not enough to call myself an expert. > > General comments first. Where applicable, I think we should apply the > same coding standards to Python as we do for C/C++. That means things > like function comments documenting parameters. They are absent for the > most part in this patch, and I won't point out individual instances. > Also, I think the documentation should follow our usual rules. There > are spelling and grammar problems. I will point out what I find (only > the first instance for recurring problems), but please proofread the > whole thing for the next submission. The Thunderbird spellchecker > actually is pointing out a lot of these. Capitalize starts of > sentences, write full sentences and terminate with punctuation. > I primarily submitted it early because you wanted to look at the tools before the code patch, which is the one I care about since the longer it goes, the more effort it is to update the patch to mainline. I apologize for not proofreading it as much as I usually do. My longer term intention was to polish the readme stuff and put it into each tool as well. however, none of the other tools or scripts in contrib subscribe to commenting every function the same as we do for c/c++. I did put comments in many places where it wasn't obvious what was going on to help with readability, but other cases it seemed obvious enough not to bother. I don't mind adding missing ones that are important, but I do not see why every function needs to have the full c/c++ coding standard applied to it when no other tool does. These certainly appear as good to me if not better than the existing scripts... >> No commenting on the quality of python code... :-) I was >> learning python on the fly. Im sure some things are QUITE awful., > > Yeah, the general impression is of fairly ad-hoc code. Not sure how > much can be done about this. they were never intended as general purpose tools, they were developed over multiple iterations and bugfixing and never properly designed.. they were never originally intended for public submission, so they suffer... and I'm not interested in rewriting them yet again Andrew