Skip to content

Preparation step for You can now write TL serializers in PHP#1403

Merged
DrDet merged 1 commit into
VKCOM:masterfrom
hrissan:gbuteyko/tl_php_serializers_tl2php
Sep 19, 2025
Merged

Preparation step for You can now write TL serializers in PHP#1403
DrDet merged 1 commit into
VKCOM:masterfrom
hrissan:gbuteyko/tl_php_serializers_tl2php

Conversation

@hrissan
Copy link
Copy Markdown
Contributor

@hrissan hrissan commented Sep 2, 2025

This MR changes tl2php only (and generated code) to add stubs to RpcFunction descendants (and RpcFunction.php, but only inside #ifndef KPHP).

This MR adds no changes to runtime at all.

#1402

@DrDet DrDet self-requested a review September 4, 2025 17:23
Comment thread Makefile Outdated
Comment thread common/tl-files/tests.tl Outdated
Comment thread runtime/tl/rpc_function.h
Comment thread runtime-light/stdlib/rpc/rpc-tl-function.h
Comment thread runtime-light/stdlib/rpc/rpc-tl-function.h Outdated
Copy link
Copy Markdown
Contributor

@DrDet DrDet left a comment

Choose a reason for hiding this comment

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

Let's extract tl2php changes to separate PR and update it first, before merging changes with new abstract RpcFunction methods. Otherwise we can't compile code with old RpcFunction inheritors without new abstract methods implementation.

@DrDet DrDet merged commit 0d13f96 into VKCOM:master Sep 19, 2025
7 checks passed
@Danil42Russia Danil42Russia added this to the next milestone Sep 20, 2025
@Danil42Russia Danil42Russia added refactoring Logic and code style improvements runtime Feature related to runtime labels Sep 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Logic and code style improvements runtime Feature related to runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants