Skip to content

Commit 7cbd4f3

Browse files
committed
Remove copy argument from php_glob_stream_get_path/pattern
This is unused, and shouldn't be part of the API.
1 parent 9b3b07c commit 7cbd4f3

File tree

3 files changed

+9
-17
lines changed

3 files changed

+9
-17
lines changed

ext/spl/spl_directory.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ PHPAPI char* spl_filesystem_object_get_path(spl_filesystem_object *intern, size_
189189
#ifdef HAVE_GLOB
190190
if (intern->type == SPL_FS_DIR) {
191191
if (php_stream_is(intern->u.dir.dirp ,&php_glob_stream_ops)) {
192-
return php_glob_stream_get_path(intern->u.dir.dirp, 0, len);
192+
return php_glob_stream_get_path(intern->u.dir.dirp, len);
193193
}
194194
}
195195
#endif

main/streams/glob_wrapper.c

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -45,19 +45,15 @@ typedef struct {
4545
size_t pattern_len;
4646
} glob_s_t;
4747

48-
PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *plen STREAMS_DC) /* {{{ */
48+
PHPAPI char* _php_glob_stream_get_path(php_stream *stream, size_t *plen STREAMS_DC) /* {{{ */
4949
{
5050
glob_s_t *pglob = (glob_s_t *)stream->abstract;
5151

5252
if (pglob && pglob->path) {
5353
if (plen) {
5454
*plen = pglob->path_len;
5555
}
56-
if (copy) {
57-
return estrndup(pglob->path, pglob->path_len);
58-
} else {
59-
return pglob->path;
60-
}
56+
return pglob->path;
6157
} else {
6258
if (plen) {
6359
*plen = 0;
@@ -67,19 +63,15 @@ PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *ple
6763
}
6864
/* }}} */
6965

70-
PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, int copy, size_t *plen STREAMS_DC) /* {{{ */
66+
PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, size_t *plen STREAMS_DC) /* {{{ */
7167
{
7268
glob_s_t *pglob = (glob_s_t *)stream->abstract;
7369

7470
if (pglob && pglob->pattern) {
7571
if (plen) {
7672
*plen = pglob->pattern_len;
7773
}
78-
if (copy) {
79-
return estrndup(pglob->pattern, pglob->pattern_len);
80-
} else {
81-
return pglob->pattern;
82-
}
74+
return pglob->pattern;
8375
} else {
8476
if (plen) {
8577
*plen = 0;

main/streams/php_stream_glob_wrapper.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,11 @@ PHPAPI extern const php_stream_ops php_glob_stream_ops;
2121

2222
BEGIN_EXTERN_C()
2323

24-
PHPAPI char* _php_glob_stream_get_path(php_stream *stream, int copy, size_t *plen STREAMS_DC);
25-
#define php_glob_stream_get_path(stream, copy, plen) _php_glob_stream_get_path((stream), (copy), (plen) STREAMS_CC)
24+
PHPAPI char* _php_glob_stream_get_path(php_stream *stream, size_t *plen STREAMS_DC);
25+
#define php_glob_stream_get_path(stream, plen) _php_glob_stream_get_path((stream), (plen) STREAMS_CC)
2626

27-
PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, int copy, size_t *plen STREAMS_DC);
28-
#define php_glob_stream_get_pattern(stream, copy, plen) _php_glob_stream_get_pattern((stream), (copy), (plen) STREAMS_CC)
27+
PHPAPI char* _php_glob_stream_get_pattern(php_stream *stream, size_t *plen STREAMS_DC);
28+
#define php_glob_stream_get_pattern(stream, plen) _php_glob_stream_get_pattern((stream), (plen) STREAMS_CC)
2929

3030
PHPAPI int _php_glob_stream_get_count(php_stream *stream, int *pflags STREAMS_DC);
3131
#define php_glob_stream_get_count(stream, pflags) _php_glob_stream_get_count((stream), (pflags) STREAMS_CC)

0 commit comments

Comments
 (0)