Doc #67549 [Asn]: $peg argument not documented

From: Date: Mon, 07 Jun 2021 03:09:59 +0000
Subject: Doc #67549 [Asn]: $peg argument not documented
References: 1  Groups: php.doc.bugs 
Request: Send a blank email to doc-bugs+get-18832@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=67549&edit=1

 ID:                 67549
 Updated by:         alan_k@php.net
 Reported by:        giso at connectholland dot nl
 Summary:            $peg argument not documented
 Status:             Assigned
 Type:               Documentation Problem
 Package:            svn
 PHP Version:        5.5.14
-Assigned To:        alan_k
+Assigned To:        cmb
 Block user comment: N
 Private report:     N

 New Comment:

Committing to SVN seems to not work anymore - here's the patch to fix this.


--- trunk/svn.c	(revision 352318)
+++ trunk/svn.c	(working copy)
@@ -961,7 +961,7 @@
 }
 
 
-/* {{{ proto array svn_ls(string repository_url [, int revision [, bool recurse [, bool peg]]])
+/* {{{ proto array svn_ls(string repository_url [, int revision [, bool recurse ]])
 	Returns a list of a directory in a working copy or repository, optionally at revision_no. */
 PHP_FUNCTION(svn_ls)
 {
@@ -968,7 +968,7 @@
 	const char *repos_url = NULL;
 	const char *utf8_repos_url = NULL;
 	long repos_url_len;
-	zend_bool recurse = 0, peg = 0;
+	zend_bool recurse = 0;
 	svn_error_t *err;
 	svn_opt_revision_t revision = { 0 };
 	apr_hash_t *dirents;
@@ -977,8 +977,8 @@
 	const char *true_path;
 	apr_hash_index_t *hi;
 
-	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|lbb",
-			&repos_url, &repos_url_len, &revision.value.number, &recurse, &peg) ==
FAILURE) {
+	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|lb",
+			&repos_url, &repos_url_len, &revision.value.number, &recurse) == FAILURE) {
 		return;
 	}
 	PHP_SVN_INIT_CLIENT();


Previous Comments:
------------------------------------------------------------------------
[2021-06-01 02:27:40] alan at roojs dot com

Yes, 

The peg option was removed here - It's probably only in there for BC - the argument is ignored
http://svn.php.net/viewvc/pecl/svn/trunk/svn.c?r1=315262&r2=317303

------------------------------------------------------------------------
[2021-05-31 13:55:14] cmb@php.net

Alan, can you confirm that this parameter is indeed unused?

------------------------------------------------------------------------
[2018-02-24 17:16:43] cmb@php.net

From looking at the sources, the $peg parameter appears to be
unused.

------------------------------------------------------------------------
[2014-07-01 11:42:47] giso at connectholland dot nl

Description:
------------
---
From manual page: http://www.php.net/function.svn-ls
---

The $peg argument listed in the function call description is not documented



------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=67549&edit=1


Thread (7 messages)

« previous php.doc.bugs (#18832) next »