From d5bfb95654d001bb5ac299ef9d20876585b833ad Mon Sep 17 00:00:00 2001
From: Matthias Paulmier <matthias.paulmier@etu.u-bordeaux.fr>
Date: Wed, 13 Jun 2018 09:32:31 +0200
Subject: [PATCH] Variable.pm: Standardize the docs

The docs was mainly pod but the newly added methods were not documented that
way.  This commit standardize that.  Of course the private methods stay
documented with simple comments.
---
 lib/Automake/Variable.pm | 88 +++++++++++++++++++++++-----------------
 1 file changed, 50 insertions(+), 38 deletions(-)

diff --git a/lib/Automake/Variable.pm b/lib/Automake/Variable.pm
index 39573d4f6..40868a787 100644
--- a/lib/Automake/Variable.pm
+++ b/lib/Automake/Variable.pm
@@ -1674,17 +1674,6 @@ sub transform_variable_recursively ($$$$$&;%)
   return $res;
 }
 
-
-=back
-
-=head1 SEE ALSO
-
-L<Automake::VarDef>, L<Automake::Condition>,
-L<Automake::DisjConditions>, L<Automake::Location>.
-
-=cut
-
-
 # _verbose_private_var (NAME)
 # --------------------------
 # The naming policy for the private variables for silent rules.
@@ -1694,10 +1683,12 @@ sub _verbose_private_var ($)
     return 'am__v_' . $name;
 }
 
+=item C<verbose_var ($name)>
+
+The public variable stem used to implement silent rules.
+
+=cut
 
-# verbose_var (NAME)
-# ------------------
-# The public variable stem used to implement silent rules.
 sub verbose_var ($)
 {
     my ($name) = @_;
@@ -1705,11 +1696,14 @@ sub verbose_var ($)
 }
 
 
-# define_verbose_var (NAME, VAL-IF-SILENT, [VAL-IF-VERBOSE])
-# ----------------------------------------------------------
-# For  silent rules, setup VAR and dispatcher, to expand to
-# VAL-IF-SILENT if silent, to VAL-IF-VERBOSE (defaulting to
-# empty) if not.
+=item C<define_verbose_var ($name, $var-if-silent, [$var-if-verbose]>
+
+For silent rules, setup C<$var> and dispatcher, to expand to
+C<$val-if-silent> if silent, to C<$val-if-verbose> (defaulting to empty)
+if not.
+
+=cut
+
 sub define_verbose_var ($$;$)
 {
     my ($name, $silent_val, $verbose_val) = @_;
@@ -1734,24 +1728,29 @@ sub define_verbose_var ($$;$)
 }
 
 
-# define_verbose_tagvar (NAME)
-# ----------------------------
-# Engage the needed silent rules machinery for tag NAME.
+=item C<define_verbose_tagvar ($name)
+
+Engage the needed silent rules machinery for tag C<$name>.
+
+=cut
+
 sub define_verbose_tagvar ($)
 {
   my ($name) = @_;
   define_verbose_var ($name, '@echo "  '. $name . ' ' x (8 - length ($name)) . '" $@;');
 }
 
+=item C<define_pretty_variable ($var, $cond, $where, @value)>
+
+Like C<define_variable>, but the C<$value> is a list, and the variable
+may be defined conditionally.  The second argument is the condition
+under which the value should be defined; this should be the empty string
+to define the variable unconditionally.  The third argument is a list
+holding the values to use for the variable.  The C<$value> is pretty
+printed in the output file.
+
+=cut
 
-# define_pretty_variable ($VAR, $COND, $WHERE, @VALUE)
-# ----------------------------------------------------
-# Like define_variable, but the value is a list, and the variable may
-# be defined conditionally.  The second argument is the condition
-# under which the value should be defined; this should be the empty
-# string to define the variable unconditionally.  The third argument
-# is a list holding the values to use for the variable.  The value is
-# pretty printed in the output file.
 sub define_pretty_variable ($$$@)
 {
     my ($var, $cond, $where, @value) = @_;
@@ -1764,22 +1763,26 @@ sub define_pretty_variable ($$$@)
     }
 }
 
+=item C<define_variable ($var, $value, $where)
+
+Define a new Automake Makefile variable C<$var> to C<$value>, but only
+if not already defined.
+
+=cut
 
-# define_variable ($VAR, $VALUE, $WHERE)
-# --------------------------------------
-# Define a new Automake Makefile variable VAR to VALUE, but only if
-# not already defined.
 sub define_variable ($$$)
 {
     my ($var, $value, $where) = @_;
     define_pretty_variable ($var, TRUE, $where, $value);
 }
 
+=item C<define_files_variable ($var, \@basename, $extension, $where)>
+
+Define the $VAR which content is the list of file names composed of a
+C<@basename> and the C<$extension>.
+
+=cut
 
-# define_files_variable ($VAR, \@BASENAME, $EXTENSION, $WHERE)
-# ------------------------------------------------------------
-# Define the $VAR which content is the list of file names composed of
-# a @BASENAME and the $EXTENSION.
 sub define_files_variable ($\@$$)
 {
   my ($var, $basename, $extension, $where) = @_;
@@ -1789,4 +1792,13 @@ sub define_files_variable ($\@$$)
 }
 
 
+=back
+
+=head1 SEE ALSO
+
+L<Automake::VarDef>, L<Automake::Condition>,
+L<Automake::DisjConditions>, L<Automake::Location>.
+
+=cut
+
 1;
-- 
2.47.2