From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11643 invoked by alias); 26 Jul 2009 19:10:50 -0000 Received: (qmail 11631 invoked by uid 22791); 26 Jul 2009 19:10:49 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from smtp-outbound-1.vmware.com (HELO smtp-outbound-1.vmware.com) (65.115.85.69) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 26 Jul 2009 19:10:37 +0000 Received: from mailhost2.vmware.com (mailhost2.vmware.com [10.16.67.167]) by smtp-outbound-1.vmware.com (Postfix) with ESMTP id 0A0D439014; Sun, 26 Jul 2009 12:10:34 -0700 (PDT) Received: from [10.20.94.141] (msnyder-server.eng.vmware.com [10.20.94.141]) by mailhost2.vmware.com (Postfix) with ESMTP id DDF178E98A; Sun, 26 Jul 2009 12:10:33 -0700 (PDT) Message-ID: <4A6CA889.1050003@vmware.com> Date: Sun, 26 Jul 2009 22:51:00 -0000 From: Michael Snyder User-Agent: Thunderbird 1.5.0.12 (X11/20080411) MIME-Version: 1.0 To: paawan oza CC: Hui Zhu , Mark Kettenis , "pedro@codesourcery.com" , "gdb-patches@sourceware.org" Subject: Re: i386.record.floating.point.patch : with more testing and assurity References: <223451.13359.qm@web112517.mail.gq1.yahoo.com> In-Reply-To: <223451.13359.qm@web112517.mail.gq1.yahoo.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes 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 X-SW-Source: 2009-07/txt/msg00641.txt.bz2 paawan oza wrote: > Hi All, > > I have worked on formatting part of the patch as per Michael and Hui's comments. > > > there is one problem I have been observing. > when I edit with emac and make everything correct.... > still the command 'diff -bwEurN' gives some unaligned formatting at some lines. Unfortunately, the formatting (indent) issues are confusing: 1) Some are artifacts of the combination of spaces and tabs. These can be ignored. 2) But some are still mis-indentations. I'll point some out. > but now things are much improved as far as far as formatting is concerned. > > please find the files attached. > > 1) i386-tdep.c : this looks perfectly okay as per comments and formatting as per open source standard. Well, much closer anyway. ;-) > 2) i386-tdep.orig.c : original file. And here's another issue. Your "orig" file is only up to date as of July 9. There have been several cvs changes since then. You need to always update and merge your files before submitting a diff. > 3) patch : when you edit this patch it has still some formatting problem > , but when you apply on linux, the resulting file looks okay. > > please extract the attached package, you will find above contents > > when I take the patch on windows and try to mail, > again format changes....and I have no clue (I tried to remove tabs, dos2unix etc....) > but still it is the same.... > > please help. > (please try take difference with attached files and see the 'diff' result, it is strange, if you take the patch on windows it gets even more worse with formatting) > > please let me know where and how can I take the difference between new and original file (attached with mail).... Have you tried doing your diffs and your email from a linux machine? I'll send diff comments in a separate email. Michael