postgres.git / summary / log / commit / refs

commit    0cf56f14dd15532fec930b502cb6457023b01ef8
Author:   Stephen Frost <sfrost@snowman.net>
Date:     Sat May 09 15:13:37 2015 +0000

    Improve ParseConfigFp comment wrt head/tail
    
    The head_p and tail_p pointers passed to ParseConfigFp() are actually
    input/output parameters, not strictly output paramaters.  This updates
    the function comment to reflect that.
    
    Per discussion with Tom.


 src/backend/utils/misc/guc-file.l | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/backend/utils/misc/guc-file.l b/src/backend/utils/misc/guc-file.l
index a04d35d3c98..5c0a9655fd8 100644
--- a/src/backend/utils/misc/guc-file.l
+++ b/src/backend/utils/misc/guc-file.l
@@ -604,12 +604,12 @@ GUC_flex_fatal(const char *msg)
  *	config_file: absolute or relative path name of the configuration file
  *	depth: recursion depth (should be 0 in the outermost call)
  *	elevel: error logging level to use
- * Output parameters:
+ * Input/Output parameters:
  *	head_p, tail_p: head and tail of linked list of name/value pairs
  *
- * *head_p and *tail_p must be initialized to NULL before calling the outer
- * recursion level.  On exit, they contain a list of name-value pairs read
- * from the input file(s).
+ * *head_p and *tail_p must either be initialized to NULL or valid pointers
+ * to a ConfigVariable list before calling the outer recursion level.  Any
+ * name-value pairs read from the input file(s) will be added to the list.
  *
  * Returns TRUE if successful, FALSE if an error occurred.  The error has
  * already been ereport'd, it is only necessary for the caller to clean up


[parent: 9a0884176fdf]