)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"aa03ff09a169371c963b48d4c6d3c4c6be5e6983","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: remove redundant check for block devices"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"block_device_info_get_mapping will return None, [] or"},{"line_number":10,"context_line":"a list of block devices. There is no need to check the size"},{"line_number":11,"context_line":"of the array."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"ba3cc151_7a66f9ef","line":9,"range":{"start_line":9,"start_character":42,"end_line":9,"end_character":46},"updated":"2015-07-10 02:45:23.000000000","message":"if block_device_info_get_mapping can return None, there appears to be a bug since \n\nlen(None) does not work.\n\nMaybe the bug goes deeper then this single line.","commit_id":"d0fe92c644933f3f10ab1ee1891e5bd0f3365964"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"203fb6a97253a2fad9416c614add23507ab2cb95","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: remove redundant check for block devices"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"block_device_info_get_mapping will return None, [] or"},{"line_number":10,"context_line":"a list of block devices. There is no need to check the size"},{"line_number":11,"context_line":"of the array."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3a50d1a3_3e55654f","line":9,"range":{"start_line":9,"start_character":42,"end_line":9,"end_character":46},"in_reply_to":"ba3cc151_7a66f9ef","updated":"2015-07-27 09:41:23.000000000","message":"Correct the method will only return a list","commit_id":"d0fe92c644933f3f10ab1ee1891e5bd0f3365964"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5ae8f7f33ba3efbd78a5d04c1b1aa153652abcc7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: remove redundant check for block devices"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"block_device_info_get_mapping will return None, [] or"},{"line_number":10,"context_line":"a list of block devices. There is no need to check the size"},{"line_number":11,"context_line":"of the array."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9a41bdd9_7ad49446","line":9,"range":{"start_line":9,"start_character":42,"end_line":9,"end_character":46},"in_reply_to":"ba3cc151_7a66f9ef","updated":"2015-07-17 08:25:40.000000000","message":"The code seems quite different to this description:\nhttps://github.com/openstack/nova/blob/master/nova/virt/driver.py#L132\n\nI don\u0027t think it can ever return None, with explains why the previous check worked fine.","commit_id":"d0fe92c644933f3f10ab1ee1891e5bd0f3365964"}]}