)]}'
{"bin/swift-ring-builder":[{"author":{"_account_id":1009,"name":"Greg Lange","email":"greglange@gmail.com","username":"greglange"},"change_message_id":"40c2a912e6d7b69bd991bc66039c8c0a9bdb662e","unresolved":false,"context_lines":[{"line_number":46,"context_line":"            wrap_dict[key] \u003d \u0027[\u0027 + dev[key] + \u0027]\u0027"},{"line_number":47,"context_line":"        else:"},{"line_number":48,"context_line":"            wrap_dict[key] \u003d dev[key]"},{"line_number":49,"context_line":"    pattern \u003d (\u0027d%(id)sr%(region)sz%(zone)s-%%(ip)s:%(port)s/%(device)s_\u0027"},{"line_number":50,"context_line":"               \u0027\"%(meta)s\"\u0027 % dev)"},{"line_number":51,"context_line":"    return pattern % wrap_dict"},{"line_number":52,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAOn%2F%2F6g4%3D","line":49,"updated":"2013-04-11 19:15:57.000000000","message":"That extra \"%\" in front of the \"ip\" field is a mistake?","commit_id":"79db99b8b48eb96105855501e41ad1b6d370feb7"},{"author":{"_account_id":995,"name":"David Goetz","email":"david.goetz@rackspace.com","username":"david-goetz"},"change_message_id":"4c7f5a8ac1b274c501dd57bcfa892e1ee070bee3","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            wrap_dict[key] \u003d dev[key]"},{"line_number":49,"context_line":"    pattern \u003d (\u0027d%(id)sr%(region)sz%(zone)s-%%(ip)s:%(port)s/%(device)s_\u0027"},{"line_number":50,"context_line":"               \u0027\"%(meta)s\"\u0027 % dev)"},{"line_number":51,"context_line":"    return pattern % wrap_dict"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"class Commands:"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAOn%2F%2F6NE%3D","line":51,"updated":"2013-04-11 21:23:52.000000000","message":"please refactor this so its easier to understand/read- is double string replacing really needed here? its seems overly complicated","commit_id":"79db99b8b48eb96105855501e41ad1b6d370feb7"},{"author":{"_account_id":1009,"name":"Greg Lange","email":"greglange@gmail.com","username":"greglange"},"change_message_id":"4e5ff64688b6e5a69f3a0b73b52a4fc4d6328368","unresolved":false,"context_lines":[{"line_number":36,"context_line":"EXIT_ERROR \u003d 2"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def format_device(dev):"},{"line_number":40,"context_line":"    \"\"\""},{"line_number":41,"context_line":"    Format a device for display."},{"line_number":42,"context_line":"    \"\"\""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAOn%2F%2F5S8%3D","line":39,"updated":"2013-04-12 16:18:13.000000000","message":"This is much easier to understand now.  Thanks!","commit_id":"7c4ec6f076fb1c47a9848786c91f3c97c66d4606"},{"author":{"_account_id":6577,"name":"Sergey Kraynev","email":"sergejyit@gmail.com","username":"skraynev"},"change_message_id":"4eff3c8ec30de6aff54dbd6f9167a0f3390d6fbc","unresolved":false,"context_lines":[{"line_number":36,"context_line":"EXIT_ERROR \u003d 2"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def format_device(dev):"},{"line_number":40,"context_line":"    \"\"\""},{"line_number":41,"context_line":"    Format a device for display."},{"line_number":42,"context_line":"    \"\"\""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAOn%2F%2F5SM%3D","line":39,"in_reply_to":"AAAAOn%2F%2F5S8%3D","updated":"2013-04-12 16:37:10.000000000","message":"I glad read it. If you think, that it is much easier, I can back previous version)","commit_id":"7c4ec6f076fb1c47a9848786c91f3c97c66d4606"},{"author":{"_account_id":1009,"name":"Greg Lange","email":"greglange@gmail.com","username":"greglange"},"change_message_id":"4e5ff64688b6e5a69f3a0b73b52a4fc4d6328368","unresolved":false,"context_lines":[{"line_number":339,"context_line":"                    exit(EXIT_ERROR)"},{"line_number":340,"context_line":"            for dev in devs:"},{"line_number":341,"context_line":"                builder.set_dev_weight(dev[\u0027id\u0027], weight)"},{"line_number":342,"context_line":"                print \u0027%s weight set to %(weight)s\u0027 % (format_device(dev), dev)"},{"line_number":343,"context_line":"        pickle.dump(builder.to_dict(), open(argv[1], \u0027wb\u0027), protocol\u003d2)"},{"line_number":344,"context_line":"        exit(EXIT_SUCCESS)"},{"line_number":345,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAOn%2F%2F5TA%3D","line":342,"updated":"2013-04-12 16:18:13.000000000","message":"%(weight)s needs to be changed to %s and dev to dev[\u0027weight\u0027].  Sorry, I didn\u0027t notice this before.","commit_id":"7c4ec6f076fb1c47a9848786c91f3c97c66d4606"},{"author":{"_account_id":6577,"name":"Sergey Kraynev","email":"sergejyit@gmail.com","username":"skraynev"},"change_message_id":"8fc080f644a4f1e76bdecb2125a3216b718945f6","unresolved":false,"context_lines":[{"line_number":339,"context_line":"                    exit(EXIT_ERROR)"},{"line_number":340,"context_line":"            for dev in devs:"},{"line_number":341,"context_line":"                builder.set_dev_weight(dev[\u0027id\u0027], weight)"},{"line_number":342,"context_line":"                print \u0027%s weight set to %(weight)s\u0027 % (format_device(dev), dev)"},{"line_number":343,"context_line":"        pickle.dump(builder.to_dict(), open(argv[1], \u0027wb\u0027), protocol\u003d2)"},{"line_number":344,"context_line":"        exit(EXIT_SUCCESS)"},{"line_number":345,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAOn%2F%2F5RE%3D","line":342,"in_reply_to":"AAAAOn%2F%2F5TA%3D","updated":"2013-04-12 16:40:49.000000000","message":"Done","commit_id":"7c4ec6f076fb1c47a9848786c91f3c97c66d4606"}]}