From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.0 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9E34BC3F2D1 for ; Wed, 4 Mar 2020 00:34:52 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 700912146E for ; Wed, 4 Mar 2020 00:34:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="WtNevm5+" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728327AbgCDAev (ORCPT ); Tue, 3 Mar 2020 19:34:51 -0500 Received: from pandora.armlinux.org.uk ([78.32.30.218]:43794 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727725AbgCDAev (ORCPT ); Tue, 3 Mar 2020 19:34:51 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=WG3iPdftyaxe4JF5JBknRpB8gUrE1SSRMPAlrPJsmAw=; b=WtNevm5+K2WjRRR67Ed0jVpB6 2oJaPGiNNJmApUlAltjHdWr9W/NretAGPe9Z6tevcgzgRk9L2yAT0tUbHxpOwAaek/kj+oBUg2DMe JyaA7k+NBSgfftFwLQGsOsS2Q0MpVXffr86yMK07g1Ig/U+ImfanNGrozJLMbwFJNcU9jmwFCB4h1 sgd7H/znFf3SbrihoxKv1/LajzEzRX9qYpazbzDB//g1fE+iCmm7cKKvnfIVCV/itV6RmsQF5gvP6 iGrgNxkcuKyFcnpfeGomwtjYtDu/0/koB1HvTX+Y3fBfsYWqE17Z1P80Tv0ZMyRI5HNBe9zsAQ1Eq lXxz6ZIWA==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:60064) by pandora.armlinux.org.uk with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1j9Hzo-00039p-FC; Wed, 04 Mar 2020 00:34:44 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.92) (envelope-from ) id 1j9Hzm-0006Mf-W2; Wed, 04 Mar 2020 00:34:43 +0000 Date: Wed, 4 Mar 2020 00:34:42 +0000 From: Russell King - ARM Linux admin To: David Miller Cc: kuba@kernel.org, corbet@lwn.net, netdev@vger.kernel.org, linux-doc@vger.kernel.org Subject: Re: [PATCH net-next] doc: sfp-phylink: correct code indentation Message-ID: <20200304003442.GW25745@shell.armlinux.org.uk> References: <20200303.153546.1011655145785464830.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200303.153546.1011655145785464830.davem@davemloft.net> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Tue, Mar 03, 2020 at 03:35:46PM -0800, David Miller wrote: > From: Russell King > Date: Tue, 03 Mar 2020 13:29:42 +0000 > > > Using vim to edit the phylink documentation reveals some mistakes due > > to the "invisible" pythonesque white space indentation that can't be > > seen with other editors. Fix it. > > > > Signed-off-by: Russell King > > I applied this, but you do know that GIT is going to warn about the > trailing whitespace to me: > > .git/rebase-apply/patch:29: trailing whitespace. > > .git/rebase-apply/patch:39: trailing whitespace. > > warning: 2 lines add whitespace errors. > > Do the empty lines really need that leading TAB? If vim's syntax colouring is correct, then it does need the tab for the code sequence to be recognised as a block of code. As kerneldoc is based on python, and white-space indentation defining a block of code is a very (annoying) pythonesque thing, it seems that vim's probably correct. But... unless someone knows how the .rst format really works... It could be that vim's syntax colouring for .rst files is broken. I was hoping that the documentation people would've spoken up about that though, as I explicitly stated in the commit message that the patch was based on vim's behaviour. Not having the tabs causes vim to reverse-bold a lot of the file, making it basically uneditable without sunglasses. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up