-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
重写消息协议层与发送消息流程 #1997
Merged
Merged
重写消息协议层与发送消息流程 #1997
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Him188
force-pushed
the
send-message-rewrite
branch
3 times, most recently
from
April 30, 2022 18:01
3a4691c
to
673ed3b
Compare
Him188
force-pushed
the
send-message-rewrite
branch
from
May 9, 2022 11:00
4e6da24
to
673ed3b
Compare
Him188
force-pushed
the
send-message-rewrite
branch
from
May 16, 2022 18:36
673ed3b
to
bc66d75
Compare
Him188
force-pushed
the
send-message-rewrite
branch
from
May 16, 2022 20:28
e158cf0
to
47f1ba2
Compare
Closed
Him188
force-pushed
the
send-message-rewrite
branch
6 times, most recently
from
May 23, 2022 21:26
7027949
to
60f309b
Compare
Him188
force-pushed
the
send-message-rewrite
branch
3 times, most recently
from
May 23, 2022 22:35
6d0e86d
to
88acc59
Compare
Him188
force-pushed
the
send-message-rewrite
branch
from
May 25, 2022 20:28
88acc59
to
9940808
Compare
…ge.acceptChildren` as internal API
Him188
force-pushed
the
send-message-rewrite
branch
from
May 25, 2022 20:49
9940808
to
9930ab8
Compare
Change attributes carrying helper objects to components Make ClockHolder open Use originalMessage for MessageReceipt
Add fragmented test Fix MusicShareProtocol and implement sending tests for MusicShareProtocolTest OutgoingMessagePipelineImpl: set stopWhenConsumed Fix `currentMessageChain` not updated in `processAlso`, add strong message packets checks, add tests for converting messages failed to send at FIRST step to LongMessageInternal Add notes for receiving ForwardMessage Add facade parameters to `download ForwardMessage*` Add ForwardMessageProtocolTest Add QuoteReplyProtocolTest Add CONTAINING_MSG for MessageDecoderContext.attributes, for information-use only Fix MessageReceiptTest Fix QuoteReplyProtocolTest
Him188
force-pushed
the
send-message-rewrite
branch
from
May 25, 2022 21:15
9930ab8
to
6790445
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
重写 MessageChain,优化性能。
重写消息解析和编码,重写收发消息流程。
允许模块化,可扩展,易于测试。为将来支持频道等做准备。
测试进度:
特别测试:
TODO: