Merge pull request #6 from peachisai/sit

fix comment
apache · Oct 24, 2023 · f5113e3 · f5113e3
2 parents f7d5759 + 304c1ca
commit f5113e3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...ava/org/apache/skywalking/apm/plugin/netty/http/ChannelPipelineRemoveLastInterceptor.java b/...ava/org/apache/skywalking/apm/plugin/netty/http/ChannelPipelineRemoveLastInterceptor.java
@@ -36,7 +36,7 @@ public void beforeMethod(EnhancedInstance objInst, Method method, Object[] allAr
 
     @Override
     public Object afterMethod(EnhancedInstance objInst, Method method, Object[] allArguments, Class<?>[] argumentsTypes, Object ret) throws Throwable {
-        // if the removed handler is a enhanced handler by skywalking, we call the method again to remove and return the user handler
+        // If the removed handler is a enhanced handler by skywalking, we call the method again to remove and return the user handler.
         if (((ChannelHandler) ret).getClass().getName().startsWith(NettyConstants.HANDLER_PACKAGE_NAME)) {
             return ((ChannelPipeline) objInst).removeLast();
         }

diff --git a/...n/src/main/java/org/apache/skywalking/apm/plugin/netty/http/RemoveHandlerInterceptor.java b/...n/src/main/java/org/apache/skywalking/apm/plugin/netty/http/RemoveHandlerInterceptor.java
@@ -44,7 +44,7 @@ public Object afterMethod(EnhancedInstance objInst, Method method, Object[] allA
         Channel channel = ctx.channel();
         ChannelPipeline pipeline = channel.pipeline();
 
-        // If the removal of the handler fails
+        // If the removal of the handler fails.
         if (pipeline.context((ChannelHandler) objInst) != null) {
             return ret;
         }