)]}'
{"keystoneclient/utils.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"d1facc3bf7ff68b201878d06d89e30cdffc9702f","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    print pt.get_string(sortby\u003dfields[0])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"def _chunk(string, length\u003d72):"},{"line_number":43,"context_line":"    return \u0027\\n\u0027.join([string[i:i + length] for i in"},{"line_number":44,"context_line":"            range(0, len(string), length)])"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAALH%2F%2FTFg%3D","line":42,"updated":"2012-09-25 00:10:28.000000000","message":"would suggest renaming to:\n\n    def word_wrap(string, max_length\u003d79):\n\nAnd it needs a brief docstring, since the implementation will be fairly cryptic to python newbies.","commit_id":"e30a59ff82a8fd0c89b36b317fd7f4e76887a6e8"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"d1facc3bf7ff68b201878d06d89e30cdffc9702f","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    for (prop, value) in d.iteritems():"},{"line_number":52,"context_line":"        if value is None:"},{"line_number":53,"context_line":"            value \u003d \u0027\u0027"},{"line_number":54,"context_line":"        #If wrap\u003dTrue, wrap values longer then 72"},{"line_number":55,"context_line":"        if wrap is True:"},{"line_number":56,"context_line":"            value \u003d _chunk(value)"},{"line_number":57,"context_line":"        pt.add_row([prop, value])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAALH%2F%2FTFw%3D","line":54,"updated":"2012-09-25 00:10:28.000000000","message":"- make this part of a docstring for print_dict(), to give it more visibility\n\n- the default line length shouldn\u0027t be mentioned here, just say that values may be word-wrapped where applicable","commit_id":"e30a59ff82a8fd0c89b36b317fd7f4e76887a6e8"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"0cd12096d18f26c73193ab37cc9dc847bd64d050","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    print pt.get_string(sortby\u003dfields[0])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"def _word_wrap(string, max_length\u003d0):"},{"line_number":43,"context_line":"    \"\"\"wrap long strings to be no longer then max_length\"\"\""},{"line_number":44,"context_line":"    return \u0027\\n\u0027.join([string[i:i + max_length] for i in"},{"line_number":45,"context_line":"            range(0, len(string), max_length)])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALH%2F%2FTDU%3D","line":42,"updated":"2012-09-25 00:58:20.000000000","message":"I\u0027m not sure a default value of 0 makes any sense here? \n\nrange(0, len(string), 0) will probably throw a stack trace","commit_id":"a607500592bdefb64d692f907ec09a98c1f43f89"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"16abda88bc1c6d9655fba5659f4b8372ad6f30de","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        if value is None:"},{"line_number":59,"context_line":"            value \u003d \u0027\u0027"},{"line_number":60,"context_line":"        # Wrap long words if wrap \u003e 0"},{"line_number":61,"context_line":"        if wrap \u003e 0:"},{"line_number":62,"context_line":"            value \u003d _word_wrap(value, max_length\u003dwrap)"},{"line_number":63,"context_line":"        pt.add_row([prop, value])"},{"line_number":64,"context_line":"    print pt.get_string(sortby\u003d\u0027Property\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAALH%2F%2FS8o%3D","line":61,"updated":"2012-09-25 06:32:41.000000000","message":"you don\u0027t need this conditional or comment anymore, just pass the string to _word_wrap()","commit_id":"6401eff0473dce1fdd74c215c4702cb2a64844d5"}],"keystoneclient/v2_0/shell.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"9a1e64678f745679111f172003861ebefc87c38e","unresolved":false,"context_lines":[{"line_number":416,"context_line":"    utils.print_dict(d)"},{"line_number":417,"context_line":""},{"line_number":418,"context_line":""},{"line_number":419,"context_line":"def _chunk(string, length\u003d72):"},{"line_number":420,"context_line":"    return \u0027\\n\u0027.join([string[i:i + length] for i in"},{"line_number":421,"context_line":"            range(0, len(string), length)])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2FU1Q%3D","line":419,"updated":"2012-09-21 02:03:18.000000000","message":"why 72?","commit_id":"77a617a826ebc3656e96b352c9e8e4f32b0bf058"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"0cd12096d18f26c73193ab37cc9dc847bd64d050","unresolved":false,"context_lines":[{"line_number":407,"context_line":""},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"@utils.arg(\u0027--wrap\u0027, metavar\u003d\u0027\u003cinteger\u003e\u0027, default\u003d79,"},{"line_number":410,"context_line":"           help\u003d\u0027wrap PKI tokens\u0027)"},{"line_number":411,"context_line":"def do_token_get(kc, args):"},{"line_number":412,"context_line":"    \"\"\"Display the current user token\"\"\""},{"line_number":413,"context_line":"    utils.print_dict(kc.service_catalog.get_token(),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALH%2F%2FTDI%3D","line":410,"updated":"2012-09-25 00:58:20.000000000","message":"\u0027wrap PKI tokens to a specified length, or 0 to disable\u0027","commit_id":"a607500592bdefb64d692f907ec09a98c1f43f89"}]}