)]}'
{"nova/virt/vmwareapi/vim_util.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"b4a18c2da242d23cf3ce9bf2194990f1b8b9f15c","unresolved":false,"context_lines":[{"line_number":309,"context_line":"    if properties is None:"},{"line_number":310,"context_line":"        properties \u003d [\u0027name\u0027]"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"    if \u0027vmFolder\u0027 not in properties:"},{"line_number":313,"context_line":"        properties.append(\u0027vmFolder\u0027)"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"    #TODO(hartsocks): use traversal spec here."}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAASX%2F%2F9uU%3D","line":312,"updated":"2013-08-23 05:46:51.000000000","message":"Consider:\nif not properties.get(\u0027vmFolder\u0027):","commit_id":"8bc3710f9e50893e96bfa40064b9a9e564da34bd"},{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"730f8f718ca065ad4f94efd79c7e483e7582705f","unresolved":false,"context_lines":[{"line_number":309,"context_line":"    if properties is None:"},{"line_number":310,"context_line":"        properties \u003d [\u0027name\u0027]"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"    if \u0027vmFolder\u0027 not in properties:"},{"line_number":313,"context_line":"        properties.append(\u0027vmFolder\u0027)"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"    #TODO(hartsocks): use traversal spec here."}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2Fwd8%3D","line":312,"in_reply_to":"AAAASX%2F%2F9uU%3D","updated":"2013-09-23 18:41:41.000000000","message":"properties is a list.","commit_id":"8bc3710f9e50893e96bfa40064b9a9e564da34bd"}]}