From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753417AbbGNLIH (ORCPT ); Tue, 14 Jul 2015 07:08:07 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:17204 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752818AbbGNLIF (ORCPT ); Tue, 14 Jul 2015 07:08:05 -0400 Date: Tue, 14 Jul 2015 14:07:36 +0300 From: Dan Carpenter To: Andy Whitcroft Cc: Joe Perches , linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org, Jes.Sorensen@redhat.com, sparmaintainer@unisys.com, driverdev-devel@linuxdriverproject.org, Tim Sell Subject: [patch] checkpatch: Fixes: tag lines are allowed to be long Message-ID: <20150714110736.GA10856@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.23 (2014-03-12) X-Source-IP: userv0022.oracle.com [156.151.31.74] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A lot of the Fixes: tags go over the 75 character limit and that's ok. Fixes: 2a076f40d8c9 ('checkpatch, SubmittingPatches: suggest line wrapping commit messages at 75 columns') Signed-off-by: Dan Carpenter --- The other common cause of false positives it Oops output but I don't have an idea how to filter for that. diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 90e1edc..537973b 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2310,7 +2310,7 @@ sub process { # Check for line lengths > 75 in commit log, warn once if ($in_commit_log && !$commit_log_long_line && - length($line) > 75) { + length($line) > 75 && !($line =~ /^Fixes:/)) { WARN("COMMIT_LOG_LONG_LINE", "Possible unwrapped commit description (prefer a maximum 75 chars per line)\n" . $herecurr); $commit_log_long_line = 1;