From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16143 invoked by alias); 1 Jan 2014 17:42:33 -0000 Mailing-List: contact libc-ports-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: libc-ports-owner@sourceware.org Received: (qmail 16134 invoked by uid 89); 1 Jan 2014 17:42:32 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-vb0-f45.google.com Received: from mail-vb0-f45.google.com (HELO mail-vb0-f45.google.com) (209.85.212.45) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Wed, 01 Jan 2014 17:42:32 +0000 Received: by mail-vb0-f45.google.com with SMTP id i12so6876565vbh.32 for ; Wed, 01 Jan 2014 09:42:29 -0800 (PST) MIME-Version: 1.0 X-Received: by 10.58.235.129 with SMTP id um1mr10094370vec.17.1388598149737; Wed, 01 Jan 2014 09:42:29 -0800 (PST) Received: by 10.52.254.168 with HTTP; Wed, 1 Jan 2014 09:42:29 -0800 (PST) In-Reply-To: References: <52BC8489.9040307@twiddle.net> <52BD983C.5050006@twiddle.net> Date: Wed, 01 Jan 2014 17:42:00 -0000 Message-ID: Subject: Re: [PATCH] [AARCH64]: Pointer mangling support for Aarch64 From: Marcus Shawcroft To: Venkataramanan Kumar Cc: libc-ports@sourceware.org, Patch Tracking Content-Type: text/plain; charset=ISO-8859-1 X-IsSubscribed: yes X-SW-Source: 2014-01/txt/msg00005.txt.bz2 On 31 December 2013 05:15, Venkataramanan Kumar wrote: > Hi Marcus, > > Thank you for reviewing the patch. > I have incorporated your review comments. > > (Snip) > +#ifdef PTR_DEMANGLE > + PTR_DEMANGLE (sp); > +#endif > Nested #if should have indentation between # and if. > (Snip) > > In ports/sysdeps/aarch64/jmpbuf-offsets.h, should I need to add space > between # and if? > Reading the glibc coding style, I understood it as outer #ifndef > __ASSEMBLER__ will not increase indentation level. > > if it is Ok, can you please commit on my behalf since I don't have write access. You are right. I'll commit this for you shortly. /Marcus