)]}'
{"src/main/java/net/spy/memcached/MemcachedClient.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":"35d261aaca4b72043f7d415249c5ed34a1bfceba","unresolved":false,"context_lines":[{"line_number":1613,"context_line":"            latch.countDown();"},{"line_number":1614,"context_line":"          }"},{"line_number":1615,"context_line":"        });"},{"line_number":1616,"context_line":"    mconn.addOperation(key, op);"},{"line_number":1617,"context_line":"    rv.setOperation(op);"},{"line_number":1618,"context_line":"    return rv;"},{"line_number":1619,"context_line":"  }"}],"source_content_type":"text/x-java","patch_set":2,"id":"AAABkn%2F%2F3N4%3D","line":1616,"updated":"2012-01-08 05:25:42.000000000","message":"prefer this code style anyway, so nice change here","commit_id":"fe9d50bd54bd3a1319308b8c9e148d871eb8c2b0"}],"src/main/java/net/spy/memcached/MemcachedConnection.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":"35d261aaca4b72043f7d415249c5ed34a1bfceba","unresolved":false,"context_lines":[{"line_number":641,"context_line":"   */"},{"line_number":642,"context_line":"  public void addOperation(final String key, final Operation o) {"},{"line_number":643,"context_line":"    StringUtils.validateKey(key);"},{"line_number":644,"context_line":"    checkState();"},{"line_number":645,"context_line":""},{"line_number":646,"context_line":"    MemcachedNode placeIn \u003d null;"},{"line_number":647,"context_line":"    MemcachedNode primary \u003d locator.getPrimary(key);"}],"source_content_type":"text/x-java","patch_set":2,"id":"AAABkn%2F%2F3Nw%3D","line":644,"updated":"2012-01-08 05:25:42.000000000","message":"I\u0027m not sure why this is here.  It would seem that if this is needed, there\u0027s otherwise a race in this method.\n\nI see it came in from the refactoring.  Just not sure why it\u0027s needed.","commit_id":"fe9d50bd54bd3a1319308b8c9e148d871eb8c2b0"},{"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":"478bdb363cde3a2bd26ca4448f78619bf8413d0c","unresolved":false,"context_lines":[{"line_number":633,"context_line":"    return locator;"},{"line_number":634,"context_line":"  }"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":"  public void enqueueOperation(final String key, final Operation o) {"},{"line_number":637,"context_line":"    StringUtils.validateKey(key);"},{"line_number":638,"context_line":"    checkState();"},{"line_number":639,"context_line":"    addOperation(key, o);"}],"source_content_type":"text/x-java","patch_set":5,"id":"AAABkn%2F%2F3NM%3D","line":636,"updated":"2012-01-08 06:01:03.000000000","message":"finals aren\u0027t really needed here.  it only protects it during this method, and I can visually see it won\u0027t be mutated in this method.\n\nI used to make this mistake a lot until recently.  One of our reviewers pointed out my sillyness.","commit_id":"7ab44fe71e2a7e827160f088a18857d2e35387cc"}]}