Merge pull request #697 from CruzBishop/more-checkstyle

Small checkstyle fixes
This commit is contained in:
Norman Maurer 2012-10-30 05:04:27 -07:00
commit a9814fbf93
2 changed files with 10 additions and 10 deletions

View File

@ -41,14 +41,14 @@ public class SpdyHttpResponseStreamIdHandler extends
@Override @Override
public Object encode(ChannelHandlerContext ctx, Object msg) throws Exception { public Object encode(ChannelHandlerContext ctx, Object msg) throws Exception {
if (msg instanceof HttpMessage) { if (msg instanceof HttpMessage) {
boolean contains = ((HttpMessage)msg).containsHeader(SpdyHttpHeaders.Names.STREAM_ID); boolean contains = ((HttpMessage) msg).containsHeader(SpdyHttpHeaders.Names.STREAM_ID);
if (!contains) { if (!contains) {
ids.add(NO_ID); ids.add(NO_ID);
} else { } else {
ids.add(SpdyHttpHeaders.getStreamId((HttpMessage) msg)); ids.add(SpdyHttpHeaders.getStreamId((HttpMessage) msg));
} }
} else if (msg instanceof SpdyRstStreamFrame) { } else if (msg instanceof SpdyRstStreamFrame) {
ids.remove(((SpdyRstStreamFrame)msg).getStreamId()); ids.remove(((SpdyRstStreamFrame) msg).getStreamId());
} }
return msg; return msg;

View File

@ -17,21 +17,21 @@
/** /**
* This package is intended for use with testing against the Python * This package is intended for use with testing against the Python
* <a href="http://www.tavendo.de/autobahn/testsuite.html">AutoBahn test suite</a>. * <a href="http://www.tavendo.de/autobahn/testsuite.html">AutoBahn test suite</a>.
* *
* Autobahn installation documentation can be found <a href="http://autobahn.ws/testsuite/installation">here</a>. * Autobahn installation documentation can be found <a href="http://autobahn.ws/testsuite/installation">here</a>.
* *
* <h3>How to run the tests on Ubuntu.</h3> * <h3>How to run the tests on Ubuntu.</h3>
* *
* <p>01. Install <a href="http://python.org/">python</a> (if not already installed). * <p>01. Install <a href="http://python.org/">python</a> (if not already installed).
* *
* <p>02. Install <a href="http://pypi.python.org/pypi/setuptools">Python Setup Tools</a> if not already * <p>02. Install <a href="http://pypi.python.org/pypi/setuptools">Python Setup Tools</a> if not already
* installed. <tt>sudo apt-get install python-setuptools</tt> * installed. <tt>sudo apt-get install python-setuptools</tt>
* *
* <p>03. Add <tt>ppa:twisted-dev/ppa</tt> to your system's Software Sources * <p>03. Add <tt>ppa:twisted-dev/ppa</tt> to your system's Software Sources
* *
* <p>04. Install Twisted: <tt>sudo apt-get install python-twisted</tt> * <p>04. Install Twisted: <tt>sudo apt-get install python-twisted</tt>
* *
* <p>05. Install AutoBahn: <tt>sudo easy_install autobahntestsuite</tt>. Test using <tt>wstest --help<tt>. * <p>05. Install AutoBahn: <tt>sudo easy_install autobahntestsuite</tt>. Test using <tt>wstest --help</tt>.
* *
* <p>06. Create a directory for test configuration and results: <tt>mkdir autobahn</tt> <tt>cd autobahn</tt>. * <p>06. Create a directory for test configuration and results: <tt>mkdir autobahn</tt> <tt>cd autobahn</tt>.
* *
@ -40,13 +40,13 @@
* { * {
* "options": {"failByDrop": false}, * "options": {"failByDrop": false},
* "outdir": "./reports/servers", * "outdir": "./reports/servers",
* *
* "servers": [ * "servers": [
* {"agent": "Netty4", * {"agent": "Netty4",
* "url": "ws://localhost:9000", * "url": "ws://localhost:9000",
* "options": {"version": 18}} * "options": {"version": 18}}
* ], * ],
* *
* "cases": ["*"], * "cases": ["*"],
* "exclude-cases": [], * "exclude-cases": [],
* "exclude-agent-cases": {} * "exclude-agent-cases": {}