Skip to content

Adding Block Manager To Allow Modifications of Headers In Expressions#2455

Closed
Fenikkusu wants to merge 1 commit intozephir-lang:developmentfrom
twistersfury:feature/block-manager
Closed

Adding Block Manager To Allow Modifications of Headers In Expressions#2455
Fenikkusu wants to merge 1 commit intozephir-lang:developmentfrom
twistersfury:feature/block-manager

Conversation

@Fenikkusu
Copy link
Copy Markdown
Contributor

Hello!

  • Type: new feature
  • Link to issue: N/A

In raising this pull request, I confirm the following:

  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I updated the CHANGELOG

Small description of change:

As part of the changes I was "attempting" with #2450, at one point, I added these changes. This was originally done so I could have something like the below inside Closures.php. Eventually, my changes went another path, and these additions were no longer required. However, I still see them as having some value, so I wanted to go ahead and create a separate PR for these changes.

$compilationContext->blockManager->add(
                <<<EOF
typedef struct _zend_closure {
	zend_object       std;
	zend_function     func;
	zval              this_ptr;
	zend_class_entry *called_scope;
	zif_handler       orig_internal_handler;
} zend_closure;
EOF
            );

@Fenikkusu Fenikkusu changed the title Feature/block manager Adding Block Manager To Allow Modifications of Headers In Expressions Jun 3, 2025
@Fenikkusu Fenikkusu force-pushed the feature/block-manager branch from 4d73746 to a6cefef Compare June 3, 2025 18:12
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 3, 2025

@Fenikkusu
Copy link
Copy Markdown
Contributor Author

This was more a sample (I think), so closing.

@Fenikkusu Fenikkusu closed this Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant