Trustin Lee
|
6f2840193a
|
Fix inspection warnings related with JUnit usage
|
2012-11-12 12:45:06 +09:00 |
|
Trustin Lee
|
aedf8790c3
|
Fix various Javadoc issues / Do not use argN parameter names
|
2012-11-12 12:26:19 +09:00 |
|
Trustin Lee
|
ea4a0e3535
|
Prefer {@code ...} to <code>...</code> / Fix deprecation warnings
|
2012-11-12 11:51:23 +09:00 |
|
Trustin Lee
|
b195190b2a
|
Remove an unused import
|
2012-11-12 11:26:29 +09:00 |
|
Trustin Lee
|
15642f2cd8
|
Do not call static methods via instances
|
2012-11-12 11:23:06 +09:00 |
|
Trustin Lee
|
f1e382c89d
|
Fix compiler warnings related with SCTP detection
|
2012-11-12 09:53:44 +09:00 |
|
Trustin Lee
|
d78f5a4f76
|
Optimize imports / Remove britspace
|
2012-11-12 09:51:59 +09:00 |
|
Trustin Lee
|
9746bb2036
|
Make a member field final wherever possible
|
2012-11-12 09:43:55 +09:00 |
|
Trustin Lee
|
4dce19b814
|
Replace a variable with a constant wherever possible
|
2012-11-12 09:43:14 +09:00 |
|
Trustin Lee
|
1cb589a950
|
Replace a loop that runs only once with a single statement (Second take)
Fix potential NoSuchElementException
|
2012-11-12 09:39:16 +09:00 |
|
Trustin Lee
|
18f1b1f1d0
|
Replace a loop that runs only once with a single statement
|
2012-11-12 09:35:50 +09:00 |
|
Trustin Lee
|
aa7cd691df
|
Remove redundant 'else' branches.
|
2012-11-12 09:31:40 +09:00 |
|
Trustin Lee
|
361703b319
|
Remove unnecessary 'return's / Clean up QueryStringDecoder
|
2012-11-12 09:15:33 +09:00 |
|
Trustin Lee
|
91a61d7f43
|
Remove unnecessary qualifiers
|
2012-11-12 09:11:48 +09:00 |
|
Trustin Lee
|
61d872d6e2
|
Suppress false-positive inspection warnings /
|
2012-11-12 09:05:16 +09:00 |
|
Trustin Lee
|
a07fb94fe7
|
Prefer "str".equals(var) to var.equals("str") / Add proper null checks
|
2012-11-12 08:59:54 +09:00 |
|
Norman Maurer
|
fa805c4c13
|
Allow easier use of AIO transport via bootstrap. Related to [#725]
|
2012-11-10 20:43:53 +01:00 |
|
Trustin Lee
|
1cc104e1c0
|
Fix a compilation error
|
2012-11-10 08:51:59 +09:00 |
|
Trustin Lee
|
5d45880b9e
|
Fix a failing test
There's practically no way to test if the detected localhost is good if the user's environment is broken.
|
2012-11-10 08:50:21 +09:00 |
|
Trustin Lee
|
250c2545e6
|
Fix a compilation error (sorry!)
|
2012-11-10 08:48:00 +09:00 |
|
Trustin Lee
|
0b30bf613d
|
More robust localhost resolution
Ensure the resolved localhost can be bound and connected actually
|
2012-11-10 08:45:07 +09:00 |
|
Trustin Lee
|
b4f796c5e3
|
Use 'x' over "x" wherever possible / String.equals("") -> isEmpty()
|
2012-11-10 08:03:52 +09:00 |
|
Trustin Lee
|
05c416b674
|
Add 'static' modifier to the methods that don't need to be member methods
|
2012-11-10 07:54:33 +09:00 |
|
Trustin Lee
|
5a4b2ec07e
|
Replace keySet() + unnecessary map lookup with entrySet()
|
2012-11-10 07:36:42 +09:00 |
|
Trustin Lee
|
f77f13faf0
|
Make classes static wherever possible
|
2012-11-10 07:32:53 +09:00 |
|
Trustin Lee
|
c77bac44a2
|
Fix unchecked warnings
|
2012-11-10 07:29:14 +09:00 |
|
Trustin Lee
|
9f2c1c4774
|
Use foreach loop wherever possible / Prefer String.contains() to indexOf() >= 0 / Prefer StringUtil.split() to String.split()
|
2012-11-10 07:24:54 +09:00 |
|
Trustin Lee
|
3f4b3314cc
|
DefaultHttpDataFactory.MINSIZE must be final
|
2012-11-10 07:11:31 +09:00 |
|
Trustin Lee
|
4e0f455e69
|
Remove methods overridden but identical with the super implementation / Make constructors of abstract classes protected rather than non-sense public
AbstractWrappedByteBuf.capacity(int) should raise a UnsupportedOperationException rather than ReadOnlyBufferException.
|
2012-11-10 07:10:30 +09:00 |
|
Trustin Lee
|
d6a0fe54fd
|
Remove unnecessary throws clauses for unchecked exceptions
|
2012-11-10 07:07:37 +09:00 |
|
Trustin Lee
|
0d0eb0abfb
|
Remove unused imports
|
2012-11-10 07:05:18 +09:00 |
|
Trustin Lee
|
23883d25ee
|
Remove various unnecessary qualifiers
|
2012-11-10 07:03:07 +09:00 |
|
Trustin Lee
|
5c57dd9f0d
|
Remove redundant field initialization
|
2012-11-10 06:56:39 +09:00 |
|
Trustin Lee
|
b9fbed49bd
|
Suppress false positives related with utility class inspections.
IntelliJ IDEA inspection for private utility classes seems flawed.
|
2012-11-10 06:53:34 +09:00 |
|
Trustin Lee
|
958d04a42b
|
Remove redundant throws clauses / Suppress inspections for some false positives
|
2012-11-10 06:47:59 +09:00 |
|
Norman Maurer
|
211b4059b1
|
[#719] Handle http requests without an absolute path the right way when encoding them, which is adding / to it
|
2012-11-09 20:09:07 +01:00 |
|
Trustin Lee
|
a5a19efb4b
|
Remove unnecessary this, parenthesis, and semicolons
|
2012-11-10 02:27:33 +09:00 |
|
Trustin Lee
|
a9cfde97f0
|
Add missing final modifiers
|
2012-11-10 02:19:11 +09:00 |
|
Trustin Lee
|
8ad50a3e9c
|
Remove recundant type casting
|
2012-11-10 02:13:15 +09:00 |
|
Trustin Lee
|
27dc582e2b
|
Fix a regression in HttpMessageDecoder due to a mistake during yak shaving
|
2012-11-10 01:55:33 +09:00 |
|
Trustin Lee
|
b8f5ef0423
|
Remove unnecessary null check before equals()
|
2012-11-10 01:34:39 +09:00 |
|
Trustin Lee
|
58ba0de659
|
Remove unnecessarily qualified static access
|
2012-11-10 01:32:21 +09:00 |
|
Trustin Lee
|
bbcb035246
|
Prefer isEmpty() over size() == 0 or length() == 0
|
2012-11-10 01:24:04 +09:00 |
|
Trustin Lee
|
02a43804d4
|
Upgrade netty-build to 11
|
2012-11-10 01:13:38 +09:00 |
|
Trustin Lee
|
957154c005
|
Remove redundant no-arg constructors
|
2012-11-10 01:08:18 +09:00 |
|
Trustin Lee
|
3d364c7f75
|
Enum should not have a non-final value
|
2012-11-10 00:53:37 +09:00 |
|
Trustin Lee
|
e21dc5925d
|
Replace dynamic regular expressions with precompiled Patterns or new StringUtil.split()
|
2012-11-10 00:41:22 +09:00 |
|
Trustin Lee
|
8c0e5626c2
|
Remove concatenation with empty string
|
2012-11-09 17:55:28 +09:00 |
|
Trustin Lee
|
96a769805b
|
Fix utility classes - missing final modifiers etc
|
2012-11-09 17:44:40 +09:00 |
|
Trustin Lee
|
8842f8ef90
|
Add missing @Override annotation
|
2012-11-09 17:34:34 +09:00 |
|