)]}'
{"bin/glance":[{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    return wrap"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"def getTerminalSize():"},{"line_number":76,"context_line":"    \"\"\""},{"line_number":77,"context_line":"    determine the number of Rows and Cols of a console"},{"line_number":78,"context_line":"    code inpired by Johannes WeiB (http://tux4u.de/)"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiU%3D","line":75,"updated":"2011-09-08 14:42:05.000000000","message":"Can you rename this to get_terminal_size?","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":74,"context_line":""},{"line_number":75,"context_line":"def getTerminalSize():"},{"line_number":76,"context_line":"    \"\"\""},{"line_number":77,"context_line":"    determine the number of Rows and Cols of a console"},{"line_number":78,"context_line":"    code inpired by Johannes WeiB (http://tux4u.de/)"},{"line_number":79,"context_line":"    \"\"\""},{"line_number":80,"context_line":"    def ioctl_GWINSZ(fd):"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiM%3D","line":77,"updated":"2011-09-08 14:42:05.000000000","message":"Can you document the return values?","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    \"\"\""},{"line_number":80,"context_line":"    def ioctl_GWINSZ(fd):"},{"line_number":81,"context_line":"        try:"},{"line_number":82,"context_line":"            import fcntl"},{"line_number":83,"context_line":"            import termios"},{"line_number":84,"context_line":"            import struct"},{"line_number":85,"context_line":"            import os"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiY%3D","line":82,"updated":"2011-09-08 14:42:05.000000000","message":"I don\u0027t see a reason that these imports need to be embedded in this function. Would you mind moving them to the top of the file?","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":109,"context_line":"    return int(nrow_ncol[0]), int(nrow_ncol[1])"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"def getch():"},{"line_number":113,"context_line":"    \"\"\""},{"line_number":114,"context_line":"    getchar"},{"line_number":115,"context_line":"    \"\"\""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiQ%3D","line":112,"updated":"2011-09-08 14:42:05.000000000","message":"Might as well call this get_char and write a docstring that reflects its functionality.","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":496,"context_line":"    \"\"\"Driver function for images_index\"\"\""},{"line_number":497,"context_line":"    parameters \u003d {"},{"line_number":498,"context_line":"        \"filters\": filters,"},{"line_number":499,"context_line":"        \"limit\": limit or getTerminalSize()[0],"},{"line_number":500,"context_line":"    }"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"    optional_kwargs \u003d [\u0027marker\u0027, \u0027sort_key\u0027, \u0027sort_dir\u0027]"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiI%3D","line":499,"updated":"2011-09-08 14:42:05.000000000","message":"A comment here as to why the [0] is necessary would be nice.","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":527,"context_line":"                                    image[\u0027size\u0027])"},{"line_number":528,"context_line":""},{"line_number":529,"context_line":"    if not options.force and len(images) \u003d\u003d limit and \\"},{"line_number":530,"context_line":"       not user_response(\"press any key to continue, or press \u0027q\u0027 to quit\"):"},{"line_number":531,"context_line":"        return SUCCESS"},{"line_number":532,"context_line":""},{"line_number":533,"context_line":"    parameters[\u0027marker\u0027] \u003d images[-1][\u0027id\u0027]"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiE%3D","line":530,"updated":"2011-09-08 14:42:05.000000000","message":"I don\u0027t think we need to specify this prompt here. Can it just be the default in user_response?","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"8b02f0949d04bfc09a44bac33fc21261577d85a3","unresolved":false,"context_lines":[{"line_number":547,"context_line":"fields are treated as image metadata properties\"\"\""},{"line_number":548,"context_line":"    client \u003d get_client(options)"},{"line_number":549,"context_line":"    filters \u003d get_image_filters_from_args(args)"},{"line_number":550,"context_line":"    limit \u003d options.limit or getTerminalSize()[0] - 3"},{"line_number":551,"context_line":"    marker \u003d options.marker"},{"line_number":552,"context_line":"    sort_key \u003d options.sort_key"},{"line_number":553,"context_line":"    sort_dir \u003d options.sort_dir"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACH%2F%2F%2BiA%3D","line":550,"updated":"2011-09-08 14:42:05.000000000","message":"A comment here on why \u0027[0] - 3\u0027 is necessary would be helpful.","commit_id":"3687323ec8304f7bdddba1dbffd8e5495eaeb8f8"}]}