Skip to content
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

fix junit test and javadoc error #251

Merged
merged 2 commits into from
Oct 8, 2022
Merged

fix junit test and javadoc error #251

merged 2 commits into from
Oct 8, 2022

Conversation

Reason94
Copy link
Collaborator

@Reason94 Reason94 commented Oct 8, 2022

No description provided.

* Decode byte of the Protocol to MsgBase. Parsing protocol only, not parsing body. Execute in io thread.
* <1>Determine the data type: Request or Response</1> <2>Decode the byte to MsgBase</2> <3>Throw CodecException</3>
* Decode byte of the Protocol to MsgBase. Parsing protocol only, not parsing body. Execute in io thread. 1.
* Determine the data type: Request or Response 2. Decode the byte to MsgBase 3. Throw CodecException
Copy link
Member

Choose a reason for hiding this comment

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

建议用ul li标签排版下

* {@link #encodeBody(MsgBase)} <1>Determine the data type: Request or Response</1> <2>Encode the MsgBase to
* ByteBuf</2> <3>Throw CodecException</3>
* {@link #encodeBody(MsgBase)} 1.Determine the data type: Request or Response 2.Encode the MsgBase to ByteBuf
* 3.Throw CodecException
Copy link
Member

Choose a reason for hiding this comment

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

建议用ul li标签排版下

@@ -158,10 +158,10 @@ public static InetAddress getLocalAddress() {
}

/**
* check if is a valid address <1> not null </1> <2> not loopbackip </2> <3> not 0.0.0.0 or 127.0.0.1 </3>
* check if is a valid address 1. not null 2. not loopbackip 3. not 0.0.0.0 or 127.0.0.1
Copy link
Member

Choose a reason for hiding this comment

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

ul li排版

* {@link #createMapping(Class, Object)}</1> <2> According to the mapping relationship, map HttpRequest to MethodHandler
* {@link #handler(HttpServletRequest)}</2> <3> Convert HttpRequst to method params
* {@link #resolveArguments(HandlerMethod, HttpServletRequest, HttpServletResponse)} </3> Used in
* {@link RpcRequest}. Step1: Store the mapping relationship between Method and RequestMappingInfo
Copy link
Member

Choose a reason for hiding this comment

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

ul li排版

@tullyliu tullyliu merged commit 8a87a60 into baidu:main Oct 8, 2022
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.

None yet

2 participants