Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp619738pxu; Sun, 22 Nov 2020 22:27:07 -0800 (PST) X-Google-Smtp-Source: ABdhPJyLusCY+RDOUw40M98crwrGSq+gagy3+X6a9LqIj8YV6EM6Go4FPRoiRlt+BLz2qCa4EOyl X-Received: by 2002:a17:906:ca5a:: with SMTP id jx26mr10570004ejb.361.1606112827349; Sun, 22 Nov 2020 22:27:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606112827; cv=none; d=google.com; s=arc-20160816; b=CaY43zZa+a8yoR4WnWZsbPX2Et9rineBuh8uLibrA50F64HlMiQ8ZqqUwSAgthG/Kn +eJa1xwQIeZRuFxhhiCXy1pMn9wtO5R0EJjfK2hqa9O3u6+wK3FOm5tg0jY3tqUbe2U0 LpJTQyv2NpPxotW3fvyO/iDky7v70Nf3rEa0IeP8GKnjjcVWJPkixil838f7OKhAHno6 9jAinymeos3eMSUG9EXVj5r4f/Cgba0LwwhhEgzcYS+8hNjbERr0zcWh2i95KabMtoJW oNmwnmo6Lc/0C1bAtTmm6Kl2Wh1M+YNJ6uOA1XjpaYI63AGnMpwJ6KaylqdTN2SEGRfq R4aQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:date:cc:to:from:subject :message-id; bh=Q7ArPIwaUOkWZJo3p3imMwm6XSIx5X0JH/+UQDkqQKs=; b=oNb4+PqvefHGTdAgmoPzKUx8Zq3ocrZKh4rAw+7B4MbHFnoemp7HXfO74wHt8A6K31 QZsJQZcRFEr9o+uMwC0Y3uIq4cY4xkqHqf2AkyahIlRHMyiI3mix6feQXAaS9Z6oc4wE /OKVROXtg2Qp7tfZHmSWXWzMbR2HdOT/kB65mfFq2/wh0P86Bk8Y2wcsuti1ner85+Tf Nt8eZTKhPrF7QtOj7sjaDqc0bPjca3C2qm09+5203p4heHdMUjMuQwHYSxE2h0wFGg3F qfWuYQxMT/fGryZJ0+kdun+s2wJjUdaAYZath1gSlZwSdY9OkS0M8iSkmaxB/YqD8+5b 6SHA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i2si4549708edd.383.2020.11.22.22.26.44; Sun, 22 Nov 2020 22:27:07 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727909AbgKWFrS (ORCPT + 99 others); Mon, 23 Nov 2020 00:47:18 -0500 Received: from smtprelay0166.hostedemail.com ([216.40.44.166]:51520 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726802AbgKWFrR (ORCPT ); Mon, 23 Nov 2020 00:47:17 -0500 Received: from filter.hostedemail.com (clb03-v110.bra.tucows.net [216.40.38.60]) by smtprelay08.hostedemail.com (Postfix) with ESMTP id 0BAAB182CF665; Mon, 23 Nov 2020 05:47:16 +0000 (UTC) X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,,RULES_HIT:41:355:379:599:800:960:973:982:988:989:1260:1277:1311:1313:1314:1345:1359:1437:1515:1516:1518:1534:1541:1593:1594:1711:1730:1747:1777:1792:2197:2198:2199:2200:2393:2559:2562:2828:3138:3139:3140:3141:3142:3353:3622:3653:3865:3866:3867:3868:3873:4321:5007:7514:7903:10004:10400:10848:11232:11658:11914:12297:12438:12555:12740:12895:13069:13311:13357:13439:13894:14181:14659:14721:21080:21221:21451:21627:21740:21774:30030:30054:30070:30091,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:2,LUA_SUMMARY:none X-HE-Tag: room91_1714d1927363 X-Filterd-Recvd-Size: 2627 Received: from XPS-9350.home (unknown [47.151.128.180]) (Authenticated sender: joe@perches.com) by omf19.hostedemail.com (Postfix) with ESMTPA; Mon, 23 Nov 2020 05:47:15 +0000 (UTC) Message-ID: Subject: Re: [PATCH v6] checkpatch: add fix option for LOGICAL_CONTINUATIONS From: Joe Perches To: Aditya Srivastava Cc: lukas.bulwahn@gmail.com, linux-kernel-mentees@lists.linuxfoundation.org, linux-kernel@vger.kernel.org Date: Sun, 22 Nov 2020 21:47:13 -0800 In-Reply-To: <20201122125908.22212-1-yashsri421@gmail.com> References: <368300090d93ad66f7fb19076d076d4321a04c42.camel@perches.com> <20201122125908.22212-1-yashsri421@gmail.com> Content-Type: text/plain; charset="ISO-8859-1" User-Agent: Evolution 3.38.1-1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 2020-11-22 at 18:29 +0530, Aditya Srivastava wrote: > Currently, checkpatch warns if logical continuations are placed at the > start of a line and not at the end of previous line. > > E.g., running checkpatch on commit 3485507fc272 ("staging: > bcm2835-camera: Reduce length of enum names") reports: > > CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the > previous line > + if (!ret > + && camera_port == > > Provide a simple fix by inserting logical operator at the last > non-comment, non-whitespace char of the previous line and removing from > current line, if both the lines are additions(ie start with '+') > > Signed-off-by: Aditya Srivastava [] > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl [] > @@ -3553,8 +3553,16 @@ sub process { > ? > > ?# check for && or || at the start of a line > ? if ($rawline =~ /^\+\s*(&&|\|\|)/) { > - CHK("LOGICAL_CONTINUATIONS", > - "Logical continuations should be on the previous line\n" . $hereprev); > + my $operator = $1; > + if (CHK("LOGICAL_CONTINUATIONS", > + "Logical continuations should be on the previous line\n" . $hereprev) && > + $fix && $prevrawline =~ /^\+/) { > + # insert logical operator at last non-comment, non-whitepsace char on previous line > + $prevline =~ /[\s$;]*$/; > + my $line_end = substr($prevrawline, $-[0]); > + $fixed[$fixlinenr - 1] =~ s/\Q$line_end\E/ $operator$line_end/; This doesn't work when the same leading whitespace and trailing whitespace characters exist. You need to use something like: $fixed[$fixlinenr - 1] =~ s/\Q$line_end\E$/ $operator$line_end/; (note the $ after \E to use EOL)