Skip to content

[k2] fix TlRpcError::try_fetch#1518

Open
Shamzik wants to merge 18 commits intomasterfrom
kshamazov/rpc_error_fix
Open

[k2] fix TlRpcError::try_fetch#1518
Shamzik wants to merge 18 commits intomasterfrom
kshamazov/rpc_error_fix

Conversation

@Shamzik
Copy link
Contributor

@Shamzik Shamzik commented Feb 5, 2026

This PR fixes stats flag handling: fetch strings intead of ints

Also this PR rewrites whole function to use tl:: types directly. So instance state is got only once per call

@Shamzik Shamzik self-assigned this Feb 5, 2026
@Shamzik Shamzik added small fix When it's not an huge enhancement k2 k2 related labels Feb 5, 2026
@Shamzik Shamzik requested a review from apolyakov February 5, 2026 10:49
@Shamzik Shamzik force-pushed the kshamazov/rpc_error_fix branch 3 times, most recently from 61e52f7 to 667de14 Compare February 9, 2026 15:22
@Shamzik Shamzik changed the title [k2] fix TlRpcError::fetch_and_skip_header [k2] fix TlRpcError::try_fetch Feb 10, 2026
Copy link
Contributor Author

@Shamzik Shamzik left a comment

Choose a reason for hiding this comment

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

LGTM

@Shamzik Shamzik requested a review from apolyakov February 11, 2026 11:01
@Shamzik Shamzik force-pushed the kshamazov/rpc_error_fix branch from 734fcac to 6b1fd72 Compare February 11, 2026 11:06
@Shamzik Shamzik force-pushed the kshamazov/rpc_error_fix branch from 6b1fd72 to 4583ffd Compare February 12, 2026 08:29
@Shamzik Shamzik requested a review from apolyakov February 13, 2026 12:02
apolyakov
apolyakov previously approved these changes Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

k2 k2 related small fix When it's not an huge enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants