]> granicus.if.org Git - php/commitdiff
Remove copy argument from php_glob_stream_get_path/pattern
authorNikita Popov <nikita.ppv@gmail.com>
Wed, 19 Jun 2019 15:02:54 +0000 (17:02 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Wed, 19 Jun 2019 15:27:09 +0000 (17:27 +0200)
This is unused, and shouldn't be part of the API.

ext/spl/spl_directory.c
main/streams/glob_wrapper.c
main/streams/php_stream_glob_wrapper.h

index 7e9289c9026494198d3a8c9bc915e404afff81fd..676f107c3515dff4c880cf2f33ff7ddde10fe592 100644 (file)
@@ -189,7 +189,7 @@ PHPAPI char* spl_filesystem_object_get_path(spl_filesystem_object *intern, size_
 #ifdef HAVE_GLOB
        if (intern->type == SPL_FS_DIR) {
                if (php_stream_is(intern->u.dir.dirp ,&php_glob_stream_ops)) {
-                       return php_glob_stream_get_path(intern->u.dir.dirp, 0, len);
+                       return php_glob_stream_get_path(intern->u.dir.dirp, len);
                }
        }
 #endif
index f0c969a650bf7ed0e637a3d96ee50c50455d3604..0386d7225593f05be24878a95fbda038713d4e4f 100644 (file)
@@ -45,7 +45,7 @@ typedef struct {
        size_t   pattern_len;
 } glob_s_t;
 
-PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *plen STREAMS_DC) /* {{{ */
+PHPAPI char* _php_glob_stream_get_path(php_stream *stream, size_t *plen STREAMS_DC) /* {{{ */
 {
        glob_s_t *pglob = (glob_s_t *)stream->abstract;
 
@@ -53,11 +53,7 @@ PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *ple
                if (plen) {
                        *plen = pglob->path_len;
                }
-               if (copy) {
-                       return estrndup(pglob->path, pglob->path_len);
-               } else {
-                       return pglob->path;
-               }
+               return pglob->path;
        } else {
                if (plen) {
                        *plen = 0;
@@ -67,7 +63,7 @@ PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *ple
 }
 /* }}} */
 
-PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, int copy, size_t *plen STREAMS_DC) /* {{{ */
+PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, size_t *plen STREAMS_DC) /* {{{ */
 {
        glob_s_t *pglob = (glob_s_t *)stream->abstract;
 
@@ -75,11 +71,7 @@ PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, int copy, size_t *
                if (plen) {
                        *plen = pglob->pattern_len;
                }
-               if (copy) {
-                       return estrndup(pglob->pattern, pglob->pattern_len);
-               } else {
-                       return pglob->pattern;
-               }
+               return pglob->pattern;
        } else {
                if (plen) {
                        *plen = 0;
index 31ba5b2ca05fb030ba39f7db8cc23e9bb06b2d5b..644e5ab7e4a51b17be0c7770b46bb49cfae89b21 100644 (file)
@@ -21,11 +21,11 @@ PHPAPI extern const php_stream_ops     php_glob_stream_ops;
 
 BEGIN_EXTERN_C()
 
-PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *plen STREAMS_DC);
-#define php_glob_stream_get_path(stream, copy, plen)   _php_glob_stream_get_path((stream), (copy), (plen) STREAMS_CC)
+PHPAPI char* _php_glob_stream_get_path(php_stream *stream, size_t *plen STREAMS_DC);
+#define php_glob_stream_get_path(stream, plen) _php_glob_stream_get_path((stream), (plen) STREAMS_CC)
 
-PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, int copy, size_t *plen STREAMS_DC);
-#define php_glob_stream_get_pattern(stream, copy, plen)        _php_glob_stream_get_pattern((stream), (copy), (plen) STREAMS_CC)
+PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, size_t *plen STREAMS_DC);
+#define php_glob_stream_get_pattern(stream, plen)      _php_glob_stream_get_pattern((stream), (plen) STREAMS_CC)
 
 PHPAPI int   _php_glob_stream_get_count(php_stream *stream, int *pflags STREAMS_DC);
 #define php_glob_stream_get_count(stream, pflags)      _php_glob_stream_get_count((stream), (pflags) STREAMS_CC)