)]}'
{"nova/compat/flagfile.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d7cc1baa1c3a0932ad933e5fc01a0da12068c406","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        return argp"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    args \u003d _read_lines(flagfile)"},{"line_number":99,"context_line":"    args \u003d _iterate_args(args, _read_flagfile, tempdir\u003dtempdir)"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    config_file_args \u003d []"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F%2FIs%3D","line":99,"updated":"2012-02-03 16:34:38.000000000","message":"This code is a little confusing to me; it looks like this would cause _read_flagfile() to be called recursively.  Is there anything we can do to make this a little less opaque?  Perhaps an explanatory comment that explains what\u0027s going on?","commit_id":"bced6fbaeb9f6513ba95cf228ff1d2db762dee6b"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"89ff9a114b7ff10ff5be44ad420ebd0e80f41b26","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        return argp"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    args \u003d _read_lines(flagfile)"},{"line_number":99,"context_line":"    args \u003d _iterate_args(args, _read_flagfile, tempdir\u003dtempdir)"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    config_file_args \u003d []"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F%2FDE%3D","line":99,"in_reply_to":"AAAAEn%2F%2F%2FIs%3D","updated":"2012-02-03 19:24:49.000000000","message":"Yeah, it is confusing. I futzed about the code to try and make it somewhat less confusing and failed. I hoped the docstring would help a bit. I\u0027ll add a comment at least","commit_id":"bced6fbaeb9f6513ba95cf228ff1d2db762dee6b"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"8c2d80f90daf77d42e71d1392522d4b8c3deae53","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        return argp"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    args \u003d _read_lines(flagfile)"},{"line_number":99,"context_line":"    args \u003d _iterate_args(args, _read_flagfile, tempdir\u003dtempdir)"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    config_file_args \u003d []"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAEn%2F%2F%2FDM%3D","line":99,"updated":"2012-02-03 18:53:04.000000000","message":"This code is a little confusing to me; it looks like this would cause _read_flagfile() to be called recursively. Is there anything we can do to make this a little less opaque? Perhaps an explanatory comment that explains what\u0027s going on?","commit_id":"994c3b409aed1d02c117fb9905676865bbe37f3a"}]}