)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"90a7671f7ec6d901bcbca01e7387b9aaa9c2d452","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Replace \"xrange\" with \"range\"."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Don\u0027t add the import for ranges of 1024 items or less."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: If618b4e810e444f7eb6592bb2398805e9d14d548"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a016987_3a03bb78","line":15,"updated":"2015-05-26 09:07:42.000000000","message":"Details please. Why not? What about files with mixed ranges in them? How much of a downside is it?","commit_id":"5b347b4c2d0666d0878424a8c6833a8684e4bad3"}],"cinder/backup/drivers/ceph.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"99e2d6db8b73544922ac13890d6d19feebe0f4db","unresolved":false,"context_lines":[{"line_number":54,"context_line":"from oslo_utils import encodeutils"},{"line_number":55,"context_line":"from oslo_utils import excutils"},{"line_number":56,"context_line":"from oslo_utils import units"},{"line_number":57,"context_line":"from six.moves import range"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"from cinder.backup import driver"},{"line_number":60,"context_line":"from cinder import exception"}],"source_content_type":"text/x-python","patch_set":3,"id":"3afb71cf_2424f55b","line":57,"updated":"2015-06-10 19:50:15.000000000","message":"We don\u0027t need and IMHO shouldn\u0027t use six\u0027s range method.  Just use the built in here which is an almost exact replacement for xrange back in python 2","commit_id":"f01def68dc48df1789fdc7f6c3a5f92efecb873b"}]}