From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D18A321E1454F for ; Tue, 15 Aug 2017 09:14:39 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 6DBBD8E3DE; Tue, 15 Aug 2017 16:17:04 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 6DBBD8E3DE Authentication-Results: ext-mx01.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx01.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=lersek@redhat.com Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-68.phx2.redhat.com [10.3.116.68]) by smtp.corp.redhat.com (Postfix) with ESMTP id 0B3A0544F2; Tue, 15 Aug 2017 16:17:02 +0000 (UTC) To: "Kinney, Michael D" , edk2-devel-01 Cc: "Justen, Jordan L" , Leif Lindholm , Ard Biesheuvel References: <20170811164851.9466-1-lersek@redhat.com> <20d2afa8-3bd9-8088-7248-53a656555568@redhat.com> From: Laszlo Ersek Message-ID: <6aeb9789-1a1a-00bf-3df8-ee6e8700a403@redhat.com> Date: Tue, 15 Aug 2017 18:17:02 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Tue, 15 Aug 2017 16:17:04 +0000 (UTC) Subject: Re: [edk2-CCodingStandardsSpecification PATCH 0/2] improvements related to line wrapping X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 15 Aug 2017 16:14:40 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 08/15/17 17:17, Kinney, Michael D wrote: > Laszlo, > > If you only change a couple of words in a sentence, it will > show the differences at the work level instead of the line > level. This crossed my mind, but when I compared the two views, I saw the word-level highlighting (on top of the line-level highlighting) even under the URL without the w=1 parameter. I was confused. > If you add/remove entire paragraphs, then you will > not see any differences. I am just in the habit of always > adding ?w=1 to provide the best possible review format. > > I have not found a URL to auto select that view. When a > single patch changes several files, you get that button for > each file within the patch. I'll try to remember adding such URLs to the blurb that require the least amount of clicks in order to end up with a "rich diff". Thanks! Laszlo >> -----Original Message----- >> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On >> Behalf Of Laszlo Ersek >> Sent: Tuesday, August 15, 2017 4:01 AM >> To: Kinney, Michael D ; edk2-devel- >> 01 >> Cc: Justen, Jordan L ; Leif Lindholm >> ; Ard Biesheuvel >> >> Subject: Re: [edk2] [edk2-CCodingStandardsSpecification PATCH >> 0/2] improvements related to line wrapping >> >> Mike, >> >> On 08/11/17 19:07, Kinney, Michael D wrote: >>> Laszlo, >>> >>> You can also provide links to the GitHub commits with "?w=1" >>> flag appended. If you select the "Display the rich diff" >> button >>> above each changed file, GitHub renders a view with change >> bars. >>> >>> https://github.com/lersek/edk2- >> CCodingStandardsSpecification/commit/2c5534a24b15616fdaa02478858 >> ed1d8908dc653?w=1 >> >> I opened two browser tabs with this link, one without "?w=1" and >> another >> with "?w=1", and compared the contents visually (by switching >> back and >> forth between the tabs), checking each screen-ful. I'm not >> seeing any >> differences. >> >> When I click the button that you mention, I do get a rendered >> diff as >> well. I wonder if this feature is javascript-only, or if it can >> be >> triggered with URL changes. >> >> Thanks >> Laszlo >> >>> >>> https://github.com/lersek/edk2- >> CCodingStandardsSpecification/commit/e3797dc48316052005cefa26246 >> ab2fd32641881?w=1 >>> >>> Best regards, >>> >>> Mike >>> >>>> -----Original Message----- >>>> From: Laszlo Ersek [mailto:lersek@redhat.com] >>>> Sent: Friday, August 11, 2017 9:49 AM >>>> To: edk2-devel-01 >>>> Cc: Ard Biesheuvel ; Justen, >> Jordan L >>>> ; Leif Lindholm >>>> ; Kinney, Michael D >>>> >>>> Subject: [edk2-CCodingStandardsSpecification PATCH 0/2] >>>> improvements related to line wrapping >>>> >>>> We've discussed these ideas repeatedly over time; I'm now >>>> attempting to >>>> formalize them. Also test-driving the documentation >> contribution >>>> process. >>>> >>>> - Repo: https://github.com/lersek/edk2- >>>> CCodingStandardsSpecification.git >>>> - Branch: line_wrapping >>>> - Rendered views of the pages modified: >>>> - https://lersek.gitbooks.io/laszlo-s-fork-of-the-edk-ii-c- >>>> coding-standards-sp/content/v/line_wrapping/#edk-ii-c-coding- >>>> standards-specification >>>> - https://lersek.gitbooks.io/laszlo-s-fork-of-the-edk-ii-c- >>>> coding-standards- >> sp/content/v/line_wrapping/5_source_files/#51- >>>> general-rules >>>> - https://lersek.gitbooks.io/laszlo-s-fork-of-the-edk-ii-c- >>>> coding-standards- >>>> sp/content/v/line_wrapping/5_source_files/52_spacing.html#52- >>>> spacing >>>> >>>> Cc: Ard Biesheuvel >>>> Cc: Jordan Justen >>>> Cc: Leif Lindholm >>>> Cc: Michael D Kinney >>>> >>>> Thanks >>>> Laszlo >>>> >>>> Laszlo Ersek (2): >>>> Source Files / General Rules: limit line lengths to 80 >> columns >>>> Source Files / Spacing / Multi-line func. calls: allow >>>> condensed >>>> arguments >>>> >>>> 5_source_files/52_spacing.md | 29 +++++++++++++++++++- >>>> 5_source_files/README.md | 17 ++++++++---- >>>> README.md | 2 ++ >>>> 3 files changed, 42 insertions(+), 6 deletions(-) >>>> >>>> -- >>>> 2.13.1.3.g8be5a757fa67 >>> >> >> _______________________________________________ >> edk2-devel mailing list >> edk2-devel@lists.01.org >> https://lists.01.org/mailman/listinfo/edk2-devel