)]}'
{"java/src/main/java/monasca/api/resource/Links.java":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6bd0effe8b9e409de4122c8ba3e71806dc5397ed","unresolved":false,"context_lines":[{"line_number":153,"context_line":"      selfLink.href \u003d uriInfo.getRequestUri().toString();"},{"line_number":154,"context_line":"      paged.links.add(selfLink);"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"      if (elements !\u003d null) {"},{"line_number":157,"context_line":"        if (elements.size() \u003e\u003d Paged.LIMIT) {"},{"line_number":158,"context_line":"          Link nextLink \u003d new Link();"},{"line_number":159,"context_line":"          nextLink.rel \u003d \"next\";"}],"source_content_type":"text/x-java","patch_set":3,"id":"9a80dd14_34466b24","line":156,"updated":"2015-03-11 14:36:07.000000000","message":"Wouldn\u0027t it make more sense to add the offset !\u003d null check here? Might as well add in the self URL and then it handles elements \u003d\u003d null and offset \u003d\u003d null. Less code duplication that way, too","commit_id":"1fd1d86c46c18dcefcee18f4ddcab2bbe45246de"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6bd0effe8b9e409de4122c8ba3e71806dc5397ed","unresolved":false,"context_lines":[{"line_number":181,"context_line":"    } else {"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"      Paged paged \u003d new Paged();"},{"line_number":184,"context_line":"      paged.elements \u003d elements;"},{"line_number":185,"context_line":"      return paged;"},{"line_number":186,"context_line":"    }"},{"line_number":187,"context_line":"  }"}],"source_content_type":"text/x-java","patch_set":3,"id":"9a80dd14_d45d0fee","line":184,"updated":"2015-03-11 14:36:07.000000000","message":"Doesn\u0027t seem to handle elements \u003d\u003d null like above","commit_id":"1fd1d86c46c18dcefcee18f4ddcab2bbe45246de"}],"java/src/main/java/monasca/api/resource/MeasurementResource.java":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6bd0effe8b9e409de4122c8ba3e71806dc5397ed","unresolved":false,"context_lines":[{"line_number":80,"context_line":"      Paged paged \u003d new Paged();"},{"line_number":81,"context_line":"      paged.elements \u003d pagedList;"},{"line_number":82,"context_line":"      return paged;"},{"line_number":83,"context_line":"    } else {"},{"line_number":84,"context_line":"      Paged paged \u003d new Paged();"},{"line_number":85,"context_line":"      paged.elements \u003d pagedList;"},{"line_number":86,"context_line":"      return paged;"}],"source_content_type":"text/x-java","patch_set":3,"id":"9a80dd14_9402b7b2","line":83,"updated":"2015-03-11 14:36:07.000000000","message":"Both the if and the else have the exact same code. Why?","commit_id":"1fd1d86c46c18dcefcee18f4ddcab2bbe45246de"}]}