Add merge-err-lines script

Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/9441)
This commit is contained in:
Rich Salz 2019-08-12 15:50:51 -04:00
parent f6aca23e26
commit 8c0e76813a
2 changed files with 31 additions and 6 deletions

View File

@ -10,12 +10,8 @@
# perl -pi util/err-to-error files... # perl -pi util/err-to-error files...
# or # or
# git ls-files | grep '\.c$' | xargs perl -pi util/err-to-raise # git ls-files | grep '\.c$' | xargs perl -pi util/err-to-raise
# Consider running util/merge-err-lines first, to catch most (all?) of the
# There will be some hand-edits necessary, when the second arg was on a # cases where the XXXerr() call is split into two lines.
# separate line. This command will find them:
# git grep -E '[A-Z0-9_]+err\('
# There are about 500 such lines. Another script looks for such things
# and tries to merge lines.
# Also, what to do about the engines files? This includes: # Also, what to do about the engines files? This includes:
# AFALGerr, CAPIerr, DASYNC, OSSLTEST # AFALGerr, CAPIerr, DASYNC, OSSLTEST

29
util/merge-err-lines Executable file
View File

@ -0,0 +1,29 @@
#! /usr/bin/env perl
# Copyright 2019 The OpenSSL Project Authors. All Rights Reserved.
#
# Licensed under the Apache License 2.0 (the "License"). You may not use
# this file except in compliance with the License. You can obtain a copy
# in the file LICENSE in the source distribution or at
# https://www.openssl.org/source/license.html
# Sometimes calls to XXXerr() are split into two lines, because the define'd
# names are very long. This script looks for those lines and merges them.
# It should be run before the "err-to-raise" script.
# Run this program like this:
# perl -pi util/merge-err-lines files...
# or
# git grep -l '[A-Z0-9]err([^)]*$' | xargs perl -pi util/merge-err-lines
use strict;
use warnings;
# Look for "{whitespace}XXXerr(no-close-paren{WHITESPACE}" lines
if ( /^ *[_A-Z0-9]+err\([^)]+ *$/ ) {
my $copy = $_;
chop($copy);
$copy =~ s/ +$//;
my $next = <>;
$next =~ s/^ +//;
$_ = $copy . ' ' . $next;
}