From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Wiles, Keith" Subject: Re: tools brainstorming Date: Wed, 8 Apr 2015 16:25:44 +0000 Message-ID: References: <3571725.20GtF5MAnU@xps13> <0C5AFCA4B3408848ADF2A3073F7D8CC86D58F9C2@IRSMSX109.ger.corp.intel.com> <2232884.6IKBPajdgE@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "dev-VfR2kkLFssw@public.gmane.org" To: Thomas Monjalon , "Butler, Siobhan A" Return-path: In-Reply-To: <2232884.6IKBPajdgE@xps13> Content-Language: en-US Content-ID: <2E16B6FC64E20A4C9CE5BE9F54802359-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On 4/8/15, 11:16 AM, "Thomas Monjalon" wrote: >2015-04-08 15:53, Wiles, Keith: >> One of the biggest problems with any style is helping the developer >> maintain the style. Using some tool does help and I have used astyle >> before, not bad code formatter. Here is a few that seem to be >>reasonable. >>=20 >> http://astyle.sourceforge.net/ >>=20 >> http://uncrustify.sourceforge.net/ >>=20 >> http://sourceforge.net/projects/gcgreatcode/ > >I'm not sure it's a good idea to convert the codebase automatically. >The coding style must be a reference for new patches and they must be >automatically checked with a dedicated checkpatch tool. I was not suggesting these tools be used automatically only to get code to comply as it is being imported or patched. If we product a config file for one of the above tools then everyone can review his code before sending the patch. >By forbidding patches which don't comply, the codebase will be naturally >converted over time. It is difficult to view patches to determine if they comply, which is the only problem. Seeing the patched file is easier to see code format problems IMO. > >I didn't review this proposal yet. >My first comment is that it's too long to read :) >When a consensus is done, it must be added with a patch with custom >checkpatch addition. >