)]}'
{"src/main/java/net/spy/memcached/ObserveResponse.java":[{"author":{"_account_id":1000013,"name":"Matt Ingenthron","email":"ingenthr@gmail.com","username":"ingenthr","avatars":[{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"b6a8055720071c75a77a6acc5cce0c9ad1b5fa4f","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"  ObserveResponse(byte b) {"},{"line_number":61,"context_line":"    value \u003d b;"},{"line_number":62,"context_line":"  }"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"}"}],"source_content_type":"text/x-java","patch_set":1,"id":"AAABu3%2F%2F36E%3D","side":"PARENT","line":62,"updated":"2012-08-21 00:41:18.000000000","message":"Why not skip a blank line here?  I usually do between methods.  I\u0027ve noticed this in some of your code.","commit_id":"e7c3c28e82befed6a7e2cee222952b3c85c3d431"}],"src/main/java/net/spy/memcached/protocol/binary/ObserveOperationImpl.java":[{"author":{"_account_id":1000013,"name":"Matt Ingenthron","email":"ingenthr@gmail.com","username":"ingenthr","avatars":[{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"b6a8055720071c75a77a6acc5cce0c9ad1b5fa4f","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    final short  keylen \u003d (short) decodeShort(pl, 2);"},{"line_number":62,"context_line":"    keystate \u003d (byte) decodeByte(pl, keylen+4);"},{"line_number":63,"context_line":"    retCas \u003d (long) decodeLong(pl, keylen+5);"},{"line_number":64,"context_line":"    ObserveResponse r \u003d ObserveResponse.UNINITIALIZED;"},{"line_number":65,"context_line":"    r \u003d ObserveResponse.valueOf(keystate);"},{"line_number":66,"context_line":"    ((ObserveOperation.Callback) getCallback()).gotData(key, retCas, r);"},{"line_number":67,"context_line":"    getCallback().receivedStatus(STATUS_OK);"}],"source_content_type":"text/x-java","patch_set":1,"id":"AAABu3%2F%2F36A%3D","line":64,"updated":"2012-08-21 00:41:18.000000000","message":"Line 64 isn\u0027t needed, since the switch will return it.  You can just declare it in line 65.","commit_id":"dbd53814d2a03fa1d15f6b8d8f3d032848ec62d4"},{"author":{"_account_id":1000013,"name":"Matt Ingenthron","email":"ingenthr@gmail.com","username":"ingenthr","avatars":[{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/dface45e4481ef6bd571492fa3b4ef8c.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"571bdf0cdd058334020e106e507155e6ef505aa7","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    keystate \u003d (byte) decodeByte(pl, keylen+4);"},{"line_number":63,"context_line":"    retCas \u003d (long) decodeLong(pl, keylen+5);"},{"line_number":64,"context_line":"    ObserveResponse r;"},{"line_number":65,"context_line":"    r \u003d ObserveResponse.valueOf(keystate);"},{"line_number":66,"context_line":"    ((ObserveOperation.Callback) getCallback()).gotData(key, retCas, r);"},{"line_number":67,"context_line":"    getCallback().receivedStatus(STATUS_OK);"},{"line_number":68,"context_line":"  }"}],"source_content_type":"text/x-java","patch_set":2,"id":"AAABu3%2F%2F35I%3D","line":65,"updated":"2012-08-21 00:54:14.000000000","message":"why not on one line?","commit_id":"bf8e999528db436e4e627823cb4dbeca3a8526f4"}]}