* automake.in (get_object_extension, handle_languages)

(handle_single_transform_list, handle_compile, handle_compile)
(handle_texinfo_helper, handle_texinfo_helper, handle_man_pages)
(handle_tests, handle_minor_options, check_gnu_standards)
(lang_sub_obj, lang_c_rewrite, lang_c_finish, rule_define):
Simplify "if (defined $options{'X'})" into "if ($options{'X'})".

diff --git a/ChangeLog b/ChangeLog
index bd9748205670ee7b7ee776bf154c1a55eb1c9e1b..24aa2ced0a2fd3793008c438a3206265aa42f774 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2003-07-26  Alexandre Duret-Lutz  <adl@gnu.org>
 
+       * automake.in (get_object_extension, handle_languages)
+       (handle_single_transform_list, handle_compile, handle_compile)
+       (handle_texinfo_helper, handle_texinfo_helper, handle_man_pages)
+       (handle_tests, handle_minor_options, check_gnu_standards)
+       (lang_sub_obj, lang_c_rewrite, lang_c_finish, rule_define):
+       Simplify "if (defined $options{'X'})" into "if ($options{'X'})".
+
        Complete changes from 2003-07-08.
        * automake.in ($cmdline_use_dependencies, $use_dependencies): Remove,
        replaced by $options{'no-dependencies'}.

diff --git a/automake.in b/automake.in
index 30de9d5871b3e030bc26174de634bb1a2bbbd881..3a352fe6e7c5f2a2ef9d3ca317d2c8e3358e89e8 100755 (executable)
--- a/automake.in
+++ b/automake.in
@@ -1265,7 +1265,7 @@ sub get_object_extension
 
     # Check for automatic de-ANSI-fication.
     $extension = '$U' . $extension
-      if defined $options{'ansi2knr'};
+      if $options{'ansi2knr'};
 
     $get_object_extension_was_run = 1;
 
@@ -1366,7 +1366,7 @@ sub handle_languages
            $output_flag = '-o'
              if (! $output_flag
                  && $lang->name eq 'c'
-                 && defined $options{'subdir-objects'});
+                 && $options{'subdir-objects'});
 
            # Compute a possible derived extension.
            # This is not used by depend2.am.
@@ -1828,7 +1828,7 @@ sub handle_single_transform_list ($$$$@)
                 my $obj_sans_ext = substr ($object, 0,
                                           - length ($this_obj_ext));
                my $full_ansi = $full;
-               if ($lang->ansi && defined $options{'ansi2knr'})
+               if ($lang->ansi && $options{'ansi2knr'})
                  {
                    $full_ansi =~ s/$KNOWN_EXTENSIONS_PATTERN$/\$U$&/;
                    $obj_sans_ext .= '$U';
@@ -2297,7 +2297,7 @@ sub handle_compile ()
 
     # Boilerplate.
     my $default_includes = '';
-    if (! defined $options{'nostdinc'})
+    if (! $options{'nostdinc'})
       {
        $default_includes = ' -I. -I$(srcdir)';
 
@@ -2338,7 +2338,7 @@ sub handle_compile ()
     $output_rules .= "$coms$rules";
 
     # Check for automatic de-ANSI-fication.
-    if (defined $options{'ansi2knr'})
+    if ($options{'ansi2knr'})
       {
        my ($ansi2knr_filename, $ansi2knr_where) = @{$options{'ansi2knr'}};
        my $ansi2knr_dir = '';
@@ -3092,7 +3092,7 @@ sub handle_texinfo_helper ($)
 
   push (@dist_targets, 'dist-info');
 
-  if (! defined $options{'no-installinfo'})
+  if (! $options{'no-installinfo'})
     {
       # Make sure documentation is made and installed first.  Use
       # $(INFO_DEPS), not 'info', because otherwise recursive makes
@@ -3114,7 +3114,7 @@ sub handle_texinfo_helper ($)
   # Do some error checking.  Note that this file is not required
   # when in Cygnus mode; instead we defined TEXINFO_TEX explicitly
   # up above.
-  if ($need_texi_file && ! defined $options{'no-texinfo.tex'})
+  if ($need_texi_file && ! $options{'no-texinfo.tex'})
     {
       if ($need_texi_file > 1)
        {
@@ -3218,10 +3218,8 @@ sub handle_man_pages
                                 new Automake::Location,
                                 MANS => "@mans");
 
-  if (! defined $options{'no-installman'})
-    {
-      push (@all, '$(MANS)');
-    }
+  push (@all, '$(MANS)')
+    unless $options{'no-installman'};
 }
 
 # Handle DATA variables.
@@ -4255,7 +4253,7 @@ sub handle_factored_dependencies
   reject_target ('install-info-local',
                 "`install-info-local' target defined but "
                 . "`no-installinfo' option not in use")
-    unless defined $options{'no-installinfo'};
+    unless $options{'no-installinfo'};
 
   # Install the -local hooks.
   foreach (keys %dependencies)
@@ -4330,7 +4328,7 @@ sub handle_tests_dejagnu
 # Handle TESTS variable and other checks.
 sub handle_tests
 {
-  if (defined $options{'dejagnu'})
+  if ($options{'dejagnu'})
     {
       &handle_tests_dejagnu;
     }
@@ -4425,7 +4423,7 @@ sub handle_java
 # Handle some of the minor options.
 sub handle_minor_options
 {
-  if (defined $options{'readme-alpha'})
+  if ($options{'readme-alpha'})
     {
       if ($relative_dir eq '.')
        {
@@ -4784,7 +4782,7 @@ sub check_gnu_standards
     {
       msg ('error-gnu', $options{$opt},
           "option `$opt' disallowed by GNU standards")
-       if (defined $options{$opt});
+       if $options{$opt};
     }
 }
 
@@ -4817,7 +4815,7 @@ sub check_gnits_standards
 # when a subdir object should be used.
 sub lang_sub_obj
 {
-    return defined $options{'subdir-objects'} ? LANG_SUBDIR : LANG_PROCESS;
+    return $options{'subdir-objects'} ? LANG_SUBDIR : LANG_PROCESS;
 }
 
 # Rewrite a single C source file.
@@ -4825,14 +4823,14 @@ sub lang_c_rewrite
 {
   my ($directory, $base, $ext) = @_;
 
-  if (defined $options{'ansi2knr'} && $base =~ /_$/)
+  if ($options{'ansi2knr'} && $base =~ /_$/)
     {
       # FIXME: include line number in error.
       err_am "C source file `$base.c' would be deleted by ansi2knr rules";
     }
 
   my $r = LANG_PROCESS;
-  if (defined $options{'subdir-objects'})
+  if ($options{'subdir-objects'})
     {
       $r = LANG_SUBDIR;
       $base = $directory . '/' . $base
@@ -4966,7 +4964,7 @@ sub lang_c_finish
        }
     }
 
-    if (defined $options{'ansi2knr'} && keys %de_ansi_files)
+    if ($options{'ansi2knr'} && keys %de_ansi_files)
     {
        # Make all _.c files depend on their corresponding .c files.
        my @objects;
@@ -5579,7 +5577,7 @@ sub rule_define ($$$$$)
       && $target_name{$noexe}{$cond} ne $target)
     {
       # The no-exeext option enables this feature.
-      if (! defined $options{'no-exeext'})
+      if (! $options{'no-exeext'})
        {
          msg ('obsolete', $targets{$noexe}{$cond},
               "deprecated feature: target `$noexe' overrides "