automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[FYI] {master} refactor: in Automake::Options (no semantic change)


From: Stefano Lattarini
Subject: [FYI] {master} refactor: in Automake::Options (no semantic change)
Date: Fri, 24 Feb 2012 17:15:06 +0100

* lib/Automake/Options.pm: Prefer leading spaces to leading tabs
throughout.  Minor whitespace and comment changes.
(_process_option_list): Simple refactoring to make the code more
pleasant to read and easier to modify in the future.  This
refactoring also reduces code duplication, with the help of ...
(_option_must_be_from_configure, _is_valid_easy_option): ... these
new internal subroutines.
* tests/tar3.test: Enhance.
* tests/silent-amopts.test: New, checks that automake complains if
the 'silent-rules' option is used in AUTOMAKE_OPTIONS.
* tests/list-of-tests.mk: Add it.

 lib/Automake/Options.pm  |  174 ++++++++++++++++++++++++++--------------------
 tests/list-of-tests.mk   |    1 +
 tests/silent-amopts.test |   28 ++++++++
 tests/tar3.test          |   10 ++-
 4 files changed, 136 insertions(+), 77 deletions(-)

The patch is attached both in the proper, "complete" format and in a
format that ignores differences in whitespace only (obtained with
"git format-patch -w").

Regards,
  Stefano

Attachment: no-whitespace.patch
Description: Text Data

Attachment: complete.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]