Skip to content

Add stub.php and generate arginfo headers#52

Open
arshidkv12 wants to merge 4 commits into
php:masterfrom
arshidkv12:return-val
Open

Add stub.php and generate arginfo headers#52
arshidkv12 wants to merge 4 commits into
php:masterfrom
arshidkv12:return-val

Conversation

@arshidkv12

Copy link
Copy Markdown
Contributor

No description provided.

@remicollet

Copy link
Copy Markdown
Member

From a quick look, this add type hinting and return type

This breaks compatibility with PHP 7.x (still supported from package.xml)

So this need a @generate-legacy-arginfo
Alternatively, we may choose to raise minimal supported version

Missing file list change in package.xml

Worth a CHANGELOG entry (+ package.xml)

@arshidkv12

Copy link
Copy Markdown
Contributor Author

Please check it

@remicollet remicollet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1/ need a rebase

2/ package.xml

  • don't update version
  • add the changelog entry (same than in Changelog)
  • remove deleted file
  • add new files

3/ mailparse_arginfo.h defines ext_functions and class_mimemessage_methods which should be used (instead of mailparse_functions and mimemessage_methods)

4/ constants can also be moved in stub file

@remicollet

Copy link
Copy Markdown
Member

Hmmm... the rebase make commit list unreadable :(
41 commits.... 21 changed files

Can you please cleanup this PR ?

@arshidkv12

Copy link
Copy Markdown
Contributor Author

Hmmm... the rebase make commit list unreadable :( 41 commits.... 21 changed files

Can you please cleanup this PR ?

Okay, I'll clean it up.

Add legacy_arginfo

Add legacy_arginfo

constants moved to stub file

constants moved to stub file
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.

2 participants