Skip to content

Commit 668fd57

Browse files
huanghantaohuanghantao
committed
Rename zend_compile_string_position to zend_compile_position
1 parent 2b207f8 commit 668fd57

13 files changed

+32
-32
lines changed

‎Zend/zend_compile.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ static inline uint32_t zend_alloc_cache_slot(void) {
8080
}
8181

8282
ZEND_API zend_op_array *(*zend_compile_file)(zend_file_handle *file_handle, int type);
83-
ZEND_API zend_op_array *(*zend_compile_string)(zend_string *source_string, const char *filename, zend_lex_begin_state begin_state);
83+
ZEND_API zend_op_array *(*zend_compile_string)(zend_string *source_string, const char *filename, zend_compile_position position);
8484

8585
#ifndef ZTS
8686
ZEND_API zend_compiler_globals compiler_globals;

‎Zend/zend_compile.h

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -743,11 +743,11 @@ struct _zend_execute_data {
743743

744744
#include "zend_globals.h"
745745

746-
typedef enum _zend_lex_begin_state {
747-
ZEND_LEX_BEGIN_STATE_SHEBANG = 0,
748-
ZEND_LEX_BEGIN_STATE_INITIAL,
749-
ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING
750-
} zend_lex_begin_state;
746+
typedef enum _zend_compile_position {
747+
ZEND_COMPILE_POSITION_AT_SHEBANG = 0,
748+
ZEND_COMPILE_POSITION_AT_OPEN_TAG,
749+
ZEND_COMPILE_POSITION_AFTER_OPEN_TAG
750+
} zend_compile_position;
751751

752752
BEGIN_EXTERN_C()
753753

@@ -761,7 +761,7 @@ void zend_file_context_begin(zend_file_context *prev_context);
761761
void zend_file_context_end(zend_file_context *prev_context);
762762

763763
extern ZEND_API zend_op_array *(*zend_compile_file)(zend_file_handle *file_handle, int type);
764-
extern ZEND_API zend_op_array *(*zend_compile_string)(zend_string *source_string, const char *filename, zend_lex_begin_state begin_state);
764+
extern ZEND_API zend_op_array *(*zend_compile_string)(zend_string *source_string, const char *filename, zend_compile_position begin_state);
765765

766766
ZEND_API int ZEND_FASTCALL lex_scan(zval *zendlval, zend_parser_stack_elem *elem);
767767
void startup_scanner(void);
@@ -816,7 +816,7 @@ zend_string *zval_make_interned_string(zval *zv);
816816
struct _zend_arena;
817817

818818
ZEND_API zend_op_array *compile_file(zend_file_handle *file_handle, int type);
819-
ZEND_API zend_op_array *compile_string(zend_string *source_string, const char *filename, zend_lex_begin_state begin_state);
819+
ZEND_API zend_op_array *compile_string(zend_string *source_string, const char *filename, zend_compile_position begin_state);
820820
ZEND_API zend_op_array *compile_filename(int type, zend_string *filename);
821821
ZEND_API zend_ast *zend_compile_string_to_ast(
822822
zend_string *code, struct _zend_arena **ast_arena, zend_string *filename);

‎Zend/zend_execute.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4394,7 +4394,7 @@ static zend_never_inline zend_op_array* ZEND_FASTCALL zend_include_or_eval(zval
43944394
break;
43954395
case ZEND_EVAL: {
43964396
char *eval_desc = zend_make_compiled_string_description("eval()'d code");
4397-
new_op_array = zend_compile_string(inc_filename, eval_desc, ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING);
4397+
new_op_array = zend_compile_string(inc_filename, eval_desc, ZEND_COMPILE_POSITION_AFTER_OPEN_TAG);
43984398
efree(eval_desc);
43994399
}
44004400
break;

‎Zend/zend_execute_API.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1223,7 +1223,7 @@ ZEND_API zend_result zend_eval_stringl(const char *str, size_t str_len, zval *re
12231223

12241224
original_compiler_options = CG(compiler_options);
12251225
CG(compiler_options) = ZEND_COMPILE_DEFAULT_FOR_EVAL;
1226-
new_op_array = zend_compile_string(code_str, string_name, ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING);
1226+
new_op_array = zend_compile_string(code_str, string_name, ZEND_COMPILE_POSITION_AFTER_OPEN_TAG);
12271227
CG(compiler_options) = original_compiler_options;
12281228

12291229
if (new_op_array) {

‎Zend/zend_language_scanner.l

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -782,7 +782,7 @@ ZEND_API size_t zend_get_scanned_file_offset(void)
782782
return offset;
783783
}
784784

785-
zend_op_array *compile_string(zend_string *source_string, const char *filename, zend_lex_begin_state begin_state)
785+
zend_op_array *compile_string(zend_string *source_string, const char *filename, zend_compile_position position)
786786
{
787787
zend_lex_state original_lex_state;
788788
zend_op_array *op_array = NULL;
@@ -800,14 +800,14 @@ zend_op_array *compile_string(zend_string *source_string, const char *filename,
800800
zend_prepare_string_for_scanning(&tmp, filename_str);
801801
zend_string_release(filename_str);
802802

803-
switch (begin_state) {
804-
case ZEND_LEX_BEGIN_STATE_SHEBANG:
803+
switch (position) {
804+
case ZEND_COMPILE_POSITION_AT_SHEBANG:
805805
BEGIN(SHEBANG);
806806
break;
807-
case ZEND_LEX_BEGIN_STATE_INITIAL:
807+
case ZEND_COMPILE_POSITION_AT_OPEN_TAG:
808808
BEGIN(INITIAL);
809809
break;
810-
case ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING:
810+
case ZEND_COMPILE_POSITION_AFTER_OPEN_TAG:
811811
BEGIN(ST_IN_SCRIPTING);
812812
break;
813813
}

‎ext/zend_test/test.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ static ZEND_FUNCTION(zend_test_compile_string)
203203
{
204204
zend_string *source_string = NULL;
205205
zend_string *filename = NULL;
206-
zend_long begin_state = ZEND_LEX_BEGIN_STATE_INITIAL;
206+
zend_long begin_state = ZEND_COMPILE_POSITION_AT_OPEN_TAG;
207207

208208
ZEND_PARSE_PARAMETERS_START(3, 3)
209209
Z_PARAM_STR(source_string)

‎ext/zend_test/test.stub.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ function zend_test_nullable_array_return(): ?array {}
6262

6363
function zend_test_void_return(): void {}
6464

65-
function zend_test_compile_string(string $source_string, string $filename, int $begin_state): void {}
65+
function zend_test_compile_string(string $source_string, string $filename, int $position): void {}
6666

6767
/** @deprecated */
6868
function zend_test_deprecated(mixed $arg = null): void {}

‎ext/zend_test/test_arginfo.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ ZEND_END_ARG_INFO()
1313
ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_zend_test_compile_string, 0, 3, IS_VOID, 0)
1414
ZEND_ARG_TYPE_INFO(0, source_string, IS_STRING, 0)
1515
ZEND_ARG_TYPE_INFO(0, filename, IS_STRING, 0)
16-
ZEND_ARG_TYPE_INFO(0, begin_state, IS_LONG, 0)
16+
ZEND_ARG_TYPE_INFO(0, position, IS_LONG, 0)
1717
ZEND_END_ARG_INFO()
1818

1919
ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_zend_test_deprecated, 0, 0, IS_VOID, 0)

‎ext/zend_test/tests/zend_test_compile_string.phpt

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,45 +5,45 @@ zend_test
55
--FILE--
66
<?php
77

8-
define('ZEND_LEX_BEGIN_STATE_SHEBANG', 0);
9-
define('ZEND_LEX_BEGIN_STATE_INITIAL', 1);
10-
define('ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING', 2);
8+
define('ZEND_COMPILE_POSITION_AT_SHEBANG', 0);
9+
define('ZEND_COMPILE_POSITION_AT_OPEN_TAG', 1);
10+
define('ZEND_COMPILE_POSITION_AFTER_OPEN_TAG', 2);
1111

1212
$source_string = <<<EOF
1313
#!/path/to/php
1414
<?php
1515
var_dump('php');
1616
EOF;
1717

18-
zend_test_compile_string($source_string, 'Source string', ZEND_LEX_BEGIN_STATE_SHEBANG);
18+
zend_test_compile_string($source_string, 'Source string', ZEND_COMPILE_POSITION_AT_SHEBANG);
1919

2020
$source_string = <<<EOF
2121
#!/path/to/php
2222
<?php
2323
var_dump('php');
2424
EOF;
2525

26-
zend_test_compile_string($source_string, 'Source string', ZEND_LEX_BEGIN_STATE_INITIAL);
26+
zend_test_compile_string($source_string, 'Source string', ZEND_COMPILE_POSITION_AT_OPEN_TAG);
2727

2828
$source_string = <<<EOF
2929
<?php
3030
var_dump('php');
3131
EOF;
3232

33-
zend_test_compile_string($source_string, 'Source string', ZEND_LEX_BEGIN_STATE_INITIAL);
33+
zend_test_compile_string($source_string, 'Source string', ZEND_COMPILE_POSITION_AT_OPEN_TAG);
3434

3535
$source_string = <<<EOF
3636
var_dump('php');
3737
EOF;
3838

39-
zend_test_compile_string($source_string, 'Source string', ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING);
39+
zend_test_compile_string($source_string, 'Source string', ZEND_COMPILE_POSITION_AFTER_OPEN_TAG);
4040

4141
$source_string = <<<EOF
4242
<?php
4343
var_dump('php');
4444
EOF;
4545

46-
zend_test_compile_string($source_string, 'Source string', ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING);
46+
zend_test_compile_string($source_string, 'Source string', ZEND_COMPILE_POSITION_AFTER_OPEN_TAG);
4747
?>
4848
--EXPECT--
4949
string(3) "php"

‎sapi/phpdbg/phpdbg.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ ZEND_BEGIN_MODULE_GLOBALS(phpdbg)
270270

271271
zend_op_array *(*compile_file)(zend_file_handle *file_handle, int type);
272272
zend_op_array *(*init_compile_file)(zend_file_handle *file_handle, int type);
273-
zend_op_array *(*compile_string)(zend_string *source_string, const char *filename, zend_lex_begin_state begin_state);
273+
zend_op_array *(*compile_string)(zend_string *source_string, const char *filename, zend_compile_position position);
274274
HashTable file_sources;
275275

276276
zend_arena *oplog_arena; /* arena for storing oplog */

‎sapi/phpdbg/phpdbg_bp.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -878,7 +878,7 @@ static inline void phpdbg_create_conditional_break(phpdbg_breakcond_t *brake, co
878878

879879
bp_code = zend_string_concat3(
880880
"return ", sizeof("return ")-1, expr, expr_len, ";", sizeof(";")-1);
881-
new_break.ops = zend_compile_string(bp_code, "Conditional Breakpoint Code", ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING);
881+
new_break.ops = zend_compile_string(bp_code, "Conditional Breakpoint Code", ZEND_COMPILE_POSITION_AFTER_OPEN_TAG);
882882
zend_string_release(bp_code);
883883

884884
if (new_break.ops) {

‎sapi/phpdbg/phpdbg_list.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -309,15 +309,15 @@ zend_op_array *phpdbg_init_compile_file(zend_file_handle *file, int type) {
309309
return op_array;
310310
}
311311

312-
zend_op_array *phpdbg_compile_string(zend_string *source_string, const char *filename, zend_lex_begin_state begin_state) {
312+
zend_op_array *phpdbg_compile_string(zend_string *source_string, const char *filename, zend_compile_position position) {
313313
zend_string *fake_name;
314314
zend_op_array *op_array;
315315
phpdbg_file_source *dataptr;
316316
uint32_t line;
317317
char *bufptr, *endptr;
318318

319319
if (PHPDBG_G(flags) & PHPDBG_IN_EVAL) {
320-
return PHPDBG_G(compile_string)(source_string, filename, begin_state);
320+
return PHPDBG_G(compile_string)(source_string, filename, position);
321321
}
322322

323323
dataptr = emalloc(sizeof(phpdbg_file_source) + sizeof(uint32_t) * ZSTR_LEN(source_string));
@@ -332,7 +332,7 @@ zend_op_array *phpdbg_compile_string(zend_string *source_string, const char *fil
332332
dataptr->lines = ++line;
333333
dataptr->line[line] = endptr - dataptr->buf;
334334

335-
op_array = PHPDBG_G(compile_string)(source_string, filename, begin_state);
335+
op_array = PHPDBG_G(compile_string)(source_string, filename, position);
336336

337337
if (op_array == NULL) {
338338
efree(dataptr->buf);

‎sapi/phpdbg/phpdbg_prompt.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -514,7 +514,7 @@ PHPDBG_COMMAND(stdin)
514514
} /* }}} */
515515

516516
int phpdbg_compile_stdin(zend_string *code) {
517-
PHPDBG_G(ops) = zend_compile_string(code, "Standard input code", ZEND_LEX_BEGIN_STATE_ST_IN_SCRIPTING);
517+
PHPDBG_G(ops) = zend_compile_string(code, "Standard input code", ZEND_COMPILE_POSITION_AFTER_OPEN_TAG);
518518
zend_string_release(code);
519519

520520
if (EG(exception)) {

0 commit comments

Comments
 (0)