From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sog-mx-3.v43.ch3.sourceforge.com ([172.29.43.193] helo=mx.sourceforge.net) by sfs-ml-4.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1SrUfC-0000GM-60 for ltp-list@lists.sourceforge.net; Wed, 18 Jul 2012 13:51:38 +0000 Received: from mail-pb0-f47.google.com ([209.85.160.47]) by sog-mx-3.v43.ch3.sourceforge.com with esmtps (TLSv1:RC4-SHA:128) (Exim 4.76) id 1SrUfB-0003T9-1g for ltp-list@lists.sourceforge.net; Wed, 18 Jul 2012 13:51:38 +0000 Received: by pbbrq2 with SMTP id rq2so2479517pbb.34 for ; Wed, 18 Jul 2012 06:51:31 -0700 (PDT) Message-ID: <5006BF53.2080607@casparzhang.com> Date: Wed, 18 Jul 2012 21:51:15 +0800 From: Caspar Zhang MIME-Version: 1.0 References: <1342194676-15275-1-git-send-email-marios.makris@gmail.com> <500671C4.5020609@cn.fujitsu.com> In-Reply-To: Subject: Re: [LTP] [PATCH 1/6] /lib/tst_tmpdir indentation and coding style fixes List-Id: Linux Test Project General Discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ltp-list-bounces@lists.sourceforge.net To: "marios.makris" Cc: ltp-list@lists.sourceforge.net On 07/18/2012 09:32 PM, marios.makris wrote: > On 18 July 2012 11:20, Wanlong Gao wrote: >> On 07/13/2012 11:51 PM, Marios Makris wrote: >>> Identation and coding style fixes as well as moved the function descriptions >>> to the header file (/include/test.h) >>> >>> Signed-off-by: Marios Makris >>> --- >>> include/test.h | 26 +++++++++++++++ >>> lib/tst_tmpdir.c | 97 ++++++++++++++++++------------------------------------ >>> 2 files changed, 58 insertions(+), 65 deletions(-) >>> >>> diff --git a/include/test.h b/include/test.h >>> index 033fdbc..934585e 100644 >>> --- a/include/test.h >>> +++ b/include/test.h >>> @@ -201,8 +201,34 @@ extern int Tst_count; >>> void tst_sig(int fork_flag, void (*handler)(), void (*cleanup)()); >>> >>> /* lib/tst_tmpdir.c */ >>> + >>> +/* tst_tmpdir() >>> + * >>> + * Create a unique temporary directory and chdir() to it. It expects the caller >> >> It's better to keep the comments lines within 73 characters. >> >> Thanks, >> Wanlong Gao >> > > Well i thought that the line limit was 80 characters(columns) ? > > Thanks > I think it's 80 characters. (it should be OK if checkpatch.pl doesn't complain...) Caspar ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list