)]}'
{"gnocchi/rest/__init__.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"5ad6b759a9c979d1c9ff98ed951001f160186307","unresolved":false,"context_lines":[{"line_number":153,"context_line":"            granularity \u003d round(timespan / float(points))"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        if points is None and timespan is not None:"},{"line_number":156,"context_line":"            points \u003d int(timespan / granularity)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        self.points \u003d points"},{"line_number":159,"context_line":"        self.granularity \u003d granularity"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_36443ab6","line":156,"updated":"2014-11-14 17:12:17.000000000","message":"Instead of intersecting logic in lines 155-156 and 160-163, would it be clearer to combine the if statements, something like:\n\n  if points is None:\n      if timespan is None:\n          self.timespan \u003d None\n      else:\n          points \u003d int(timespan / granularity)\n          self.timespan \u003d granularity * points\n  else:\n      self.timespan \u003d granularity * points\n\n  self.points \u003d points\n  self.granularity \u003d granularity","commit_id":"4877221ee3666331e3ddb9512680bded330443fd"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"59baaa773b5eee0900f1ecb9f5c2bdccd6ecab91","unresolved":false,"context_lines":[{"line_number":153,"context_line":"            granularity \u003d round(timespan / float(points))"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        if points is None and timespan is not None:"},{"line_number":156,"context_line":"            points \u003d int(timespan / granularity)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        self.points \u003d points"},{"line_number":159,"context_line":"        self.granularity \u003d granularity"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_280bce8e","line":156,"in_reply_to":"5a890539_36443ab6","updated":"2014-11-17 13:39:49.000000000","message":"Done","commit_id":"4877221ee3666331e3ddb9512680bded330443fd"}]}
