andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove todo in StringEncoder (#9719)

Browse Source 
Motivation:

StringEncoder is marked @Shareable and CharsetEncoder is not thread-safe.

Modifications:

Remove TODO.

Result:

Less technical debt.
This commit is contained in:
Johno Crawford 2019-10-29 19:47:28 +01:00 committed by Norman Maurer
parent ff2a792923
commit cf74acfce0
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
codec/src/main/java/io/netty/handler/codec/string/StringEncoder.java
View File

@ -51,7 +51,6 @@ import java.util.List;
@Sharable @Sharable
public class StringEncoder extends MessageToMessageEncoder<CharSequence> { public class StringEncoder extends MessageToMessageEncoder<CharSequence> {
// TODO Use CharsetEncoder instead.
private final Charset charset; private final Charset charset;
/** /**